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

{CI} Use azdev test for core and telemetry #17646

Merged
merged 2 commits into from
Apr 16, 2021
Merged

{CI} Use azdev test for core and telemetry #17646

merged 2 commits into from
Apr 16, 2021

Conversation

jiasli
Copy link
Member

@jiasli jiasli commented Apr 12, 2021

Description

The current Unit Test for Core and Telemetry uses tox to run pytest, making that internal pytest uncontrollable.

This PR replaces tox with azdev test so that core and telemetry use the same test framework as other modules.

@jiasli jiasli changed the title {CI} Add test for core {CI} Add dedicated test for core Apr 12, 2021
if [[ "$(Build.Reason)" == "PullRequest" && "${{ parameters.fullTest }}" == 'False' ]]; then
echo "Running incremental test"
azdev test --no-exitfirst --repo=./ --src=HEAD --tgt=origin/$(System.PullRequest.TargetBranch) --cli-ci --profile ${{ parameters.profile }} --verbose --series
else
echo "Running full test"
azdev test --no-exitfirst --profile ${{ parameters.profile }} --verbose --series
fi
displayName: "Test on Profile ${{ parameters.profile }}"
displayName: "azdev test"
Copy link
Member Author

@jiasli jiasli Apr 12, 2021

Choose a reason for hiding this comment

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

Change the displayName to better reflect what is done inside.

Showing Profile xxx is unnecessary as it is already shown in parent job.

image

@jiasli jiasli changed the title {CI} Add dedicated test for core {CI} Use azdev test for core and telemetry Apr 12, 2021
@yonzhan yonzhan requested review from kairu-ms and msyyc April 12, 2021 05:15
@yonzhan yonzhan added this to the S186 milestone Apr 12, 2021
@yonzhan
Copy link
Collaborator

yonzhan commented Apr 12, 2021

CI

@jiasli jiasli requested a review from fengzhou-msft April 14, 2021 03:45
@jiasli jiasli merged commit a130dfe into Azure:dev Apr 16, 2021
@jiasli jiasli deleted the test-core branch April 16, 2021 09:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants