Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Traffic Router integration tests depend on an external Traffic Monitor #5992

Closed
zrhoffman opened this issue Jul 2, 2021 · 0 comments · Fixed by #6140
Closed

Traffic Router integration tests depend on an external Traffic Monitor #5992

zrhoffman opened this issue Jul 2, 2021 · 0 comments · Fixed by #6140
Labels
improvement The functionality exists but it could be improved in some way. tech debt rework due to choosing easy/limited solution Traffic Router related to Traffic Router

Comments

@zrhoffman
Copy link
Member

zrhoffman commented Jul 2, 2021

I'm submitting a ...

  • improvement request (usability, performance, tech debt, etc.)

Traffic Control components affected ...

  • Traffic Router

Current behavior:

In order to avoid Comcast/traffic_control#1665, Traffic Router integration tests require an actual Traffic Monitor in order to run:

New behavior:

The TR integration tests should use a mock Traffic Monitor server and mock Traffic Ops server like the external tests use:

httpDataServer = new HttpDataServer(testHttpServerPort);
httpDataServer.start(testHttpServerPort);

Minimal reproduction of the problem with instructions:

  1. Remove traffic_router/core/src/test/conf/traffic_monitor.properties if it exists
  2. Run the integration tests, watch them fail:
    mvn -Djava.library.path=/usr/share/java:/usr/lib64 verify
@zrhoffman zrhoffman added Traffic Router related to Traffic Router improvement The functionality exists but it could be improved in some way. tech debt rework due to choosing easy/limited solution labels Jul 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
improvement The functionality exists but it could be improved in some way. tech debt rework due to choosing easy/limited solution Traffic Router related to Traffic Router
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant