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

Add t3cutil for Go unit tests#7715

Merged
srijeet0406 merged 1 commit intoapache:masterfrom
zrhoffman:go-unit-tests-t3cutil
Aug 14, 2023
Merged

Add t3cutil for Go unit tests#7715
srijeet0406 merged 1 commit intoapache:masterfrom
zrhoffman:go-unit-tests-t3cutil

Conversation

@zrhoffman
Copy link
Copy Markdown
Member

This PR fixes an issue encountered when running the unit tests from traffic_ops/app/bin/tests:

unit_golang_1  | github.com/apache/trafficcontrol/lib/varnishcfg imports
unit_golang_1  | 	github.com/apache/trafficcontrol/cache-config/t3cutil: no required module provides package github.com/apache/trafficcontrol/cache-config/t3cutil; to add it:
unit_golang_1  | 	go get github.com/apache/trafficcontrol/cache-config/t3cutil
atcn-tounit-1_unit_golang_1 exited with code 1

Which Traffic Control components are affected by this PR?

  • Go unit tests

What is the best way to verify this PR?

Make sure the go unit tests pass when run from traffic_ops/app/bin/tests:

cd traffic_ops/app/bin/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 14, 2023
Copy link
Copy Markdown
Contributor

@kdamichie kdamichie left a comment

Choose a reason for hiding this comment

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

Fixes build. Approved

@srijeet0406 srijeet0406 merged commit 3b32fbc into apache:master Aug 14, 2023
@zrhoffman zrhoffman deleted the go-unit-tests-t3cutil branch August 14, 2023 21:00
@zrhoffman zrhoffman mentioned this pull request Aug 15, 2023
4 tasks
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.

3 participants