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

Initial push of CLI command line tests #4234

Merged
merged 5 commits into from
Sep 27, 2022

Conversation

JimSuplizio
Copy link
Member

  1. Update Start.cs to deal with a CommandLineParser issue. If a default verb is set, CommandLineParser doesn't verify whether or not the verb passed in is actually a supported verb. It assumes it to be the default verb. Something like "test-proxy foo" would end up running test-proxy start instead of erroring. The VerifyVerb function checks for this and outputs an error message that would be output by CommandLineParser if there were no default verb and an incorrect verb was passed in.
  2. GitStoreIntegrationPushTests.cs - Just updated the description here. These tests changed purpose when we went from branches to Tag. They're now similar tests but acting on different Tags that have different files and different file versions.
  3. assets.Tests.Helpers.ps1 - Support functions for the CLI integration tests.
  4. CLIIntegration.Tests.ps1 - Pester tests for testing the test-proxy CLI commands. These mirror the functionality in the test-proxy Integration test suite but with a reduced number of tests with the primary goal to test things from the using the CLI. This does require the test-proxy to be installed on the machine and in the path.

@JimSuplizio JimSuplizio self-assigned this Sep 27, 2022
@JimSuplizio JimSuplizio added Test-Proxy Anything relating to test-proxy requests or issues. Central-EngSys This issue is owned by the Engineering System team. Asset-Sync Any issues relating to moving test assets out of repos. labels Sep 27, 2022
@scbedd
Copy link
Member

scbedd commented Sep 27, 2022

/azp run tools - test-proxy - tests

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@JimSuplizio JimSuplizio marked this pull request as ready for review September 27, 2022 19:53
Copy link
Member

@scbedd scbedd left a comment

Choose a reason for hiding this comment

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

These test scripts are everything I was hoping for. Absolutely stellar tyvm.

@JimSuplizio JimSuplizio merged commit 0938643 into Azure:main Sep 27, 2022
@JimSuplizio JimSuplizio deleted the CLICommandLineTests branch September 27, 2022 20:03
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. Central-EngSys This issue is owned by the Engineering System team. Test-Proxy Anything relating to test-proxy requests or issues.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants