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

The CmdLine step within CI AzDo pipelines should be called something like ctest to indicate it is running unit tests #3082

Closed
ahsonkhan opened this issue Nov 9, 2021 · 2 comments
Labels
Central-EngSys This issue is owned by the Engineering System team. EngSys This issue is impacting the engineering system.

Comments

@ahsonkhan
Copy link
Member

https://dev.azure.com/azure-sdk/public/_build/results?buildId=1184446&view=logs&j=e109b754-e5a4-58aa-96d5-f28daf32bc3c&t=b284fc23-d415-5680-f742-99295c9bb1ab

image

ctest -C Debug -V --tests-regex azure-core. --no-compress-output -T Test

It took some digging to realize this is the step that was running unit tests, and the leg name wasn't self-descriptive.

For example, in Embedded C, we call this step ctest:
https://dev.azure.com/azure-sdk/internal/_build/results?buildId=1184185&view=logs&j=bd7cfc7a-9e05-507b-5103-65c9a747e93d&t=fc133eb2-b3d7-538b-7035-e429cbd6b68c

cc @danieljurek

@ahsonkhan ahsonkhan added EngSys This issue is impacting the engineering system. Central-EngSys This issue is owned by the Engineering System team. labels Nov 9, 2021
@ahsonkhan ahsonkhan added this to the [2021] December milestone Nov 9, 2021
@ahsonkhan
Copy link
Member Author

For some reason the display name is not being picked up:

- script: ctest -C Debug --tests-regex ${{ parameters.CtestRegex }} --no-compress-output -T Test
workingDirectory: build
displayName: ctest

@ahsonkhan
Copy link
Member Author

Re-running the legs in CI, now I do start seeing Test from the display name:
https://dev.azure.com/azure-sdk/public/_build/results?buildId=1185097&view=logs&j=117dcd10-cb90-5257-bbaf-0e8ca4e59cfe&t=dcc291b6-090f-5e15-b262-6c0eab2f9bdc

image

Looks like this PR fixed the issue, by adding the missing displayName field :)
#3079

Feel free to close, if fixed 😊

@github-actions github-actions bot locked and limited conversation to collaborators Apr 11, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Central-EngSys This issue is owned by the Engineering System team. EngSys This issue is impacting the engineering system.
Projects
None yet
Development

No branches or pull requests

2 participants