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

Add java local invocation to test-proxy livetests #6084

Closed
scbedd opened this issue May 2, 2023 · 0 comments · Fixed by #6093
Closed

Add java local invocation to test-proxy livetests #6084

scbedd opened this issue May 2, 2023 · 0 comments · Fixed by #6093
Assignees
Labels
Asset-Sync Any issues relating to moving test assets out of repos. Test-Proxy Anything relating to test-proxy requests or issues.

Comments

@scbedd
Copy link
Member

scbedd commented May 2, 2023

Very straightforward repro, grab one of the standalone executables.

C:\Users\billwert\AppData\Local\Temp\test-proxy\Azure.Sdk.Tools.TestProxy.exe --storage-location . -- --urls http://localhost:5000
'--urls' was not matched. Did you mean one of the following?
--help
Unrecognized command or argument '--urls'.
Unrecognized command or argument 'http://localhost:5000'.

Aha, this is because the --urls additional argument is only honored by the start verb now. A default server start (verbless) is assumed to be default startup only.

This broke the Java folks local invocation, as @billwert notified me.

To avoid this going forward, I have retitled this issue from -- --urls not being honored by standalone test proxy to add java local invocation to test-proxy livetests.

@scbedd scbedd added Asset-Sync Any issues relating to moving test assets out of repos. Test-Proxy Anything relating to test-proxy requests or issues. labels May 2, 2023
@scbedd scbedd self-assigned this May 2, 2023
@scbedd scbedd moved this from 🤔 Triage to 🐝 Dev in Azure SDK EngSys 🤖🧠 May 2, 2023
@scbedd scbedd changed the title -- --urls not being honored by standalone test proxy Add java local invocation to test-proxy livetests May 2, 2023
@github-project-automation github-project-automation bot moved this from 🐝 Dev to 🎊 Closed in Azure SDK EngSys 🤖🧠 May 4, 2023
@scbedd scbedd moved this from 🎊 Closed to 🔬 Dev in PR in Azure SDK EngSys 🤖🧠 May 4, 2023
@kurtzeborn kurtzeborn moved this from 🔬 Dev in PR to 🎊 Closed in Azure SDK EngSys 🤖🧠 May 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Asset-Sync Any issues relating to moving test assets out of repos. Test-Proxy Anything relating to test-proxy requests or issues.
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

1 participant