Skip to content
This repository was archived by the owner on Nov 24, 2025. It is now read-only.

Add t3cutil for more Go tests#7717

Merged
shamrickus merged 1 commit intoapache:masterfrom
zrhoffman:t3cutil-more
Aug 15, 2023
Merged

Add t3cutil for more Go tests#7717
shamrickus merged 1 commit intoapache:masterfrom
zrhoffman:t3cutil-more

Conversation

@zrhoffman
Copy link
Copy Markdown
Member

As follow-up PR to #7715, this adds the t3cutil package to more Docker images that run unit tests around the project, which is necessary since #7669.


Which Traffic Control components are affected by this PR?

  • Traffic Ops API tests docker image
    • Traffic Monitor unit tests docker image

What is the best way to verify this PR?

Run the tests

cd traffic_monitor/tests
docker-compose up --build

PR submission checklist

@zrhoffman zrhoffman added regression bug a bug in existing functionality introduced by a new version tests related to tests and/or testing infrastructure dependencies Pull requests that update a dependency file labels Aug 15, 2023
Copy link
Copy Markdown
Member

@shamrickus shamrickus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

  • t3cutil tests are added to relevant docker images and pass.

@shamrickus shamrickus merged commit 8f08779 into apache:master Aug 15, 2023
@zrhoffman zrhoffman deleted the t3cutil-more branch August 15, 2023 15:03
cybertunnel pushed a commit to cybertunnel/trafficcontrol that referenced this pull request Aug 16, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

dependencies Pull requests that update a dependency file regression bug a bug in existing functionality introduced by a new version tests related to tests and/or testing infrastructure

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants