Skip to content

Updates to build scipt to exclude long running test cases#114

Closed
Suraiya-Hameed wants to merge 1 commit intomicrosoft:devfrom
Suraiya-Hameed:buildScriptTags
Closed

Updates to build scipt to exclude long running test cases#114
Suraiya-Hameed wants to merge 1 commit intomicrosoft:devfrom
Suraiya-Hameed:buildScriptTags

Conversation

@Suraiya-Hameed
Copy link
Copy Markdown
Contributor

Build script will now exclude long running testcases that are marked with 'slow' tag. User needs to comment out the exclude property to run entire test suite.

filters {
tags {
// comment out the exclude property to run all tests
exclude 'slow'
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

by default, does it run all the tests or skip the slow tests?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

by default it will skip all the testcases with tag 'slow'. As discussed, we can come up with better alternative after we finalize on build automation system.

<configuration>
<properties>
<!-- Comment out the excludeTags to run all tests -->
<excludeTags>slow</excludeTags>
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

by default, does it run all the tests or skip the slow tests?

@v-nisidh v-nisidh modified the milestones: 6.1.4, 6.1.5 Feb 5, 2017
@v-nisidh v-nisidh modified the milestones: 6.1.7, 6.1.5 Mar 9, 2017
Copy link
Copy Markdown
Contributor

@v-nisidh v-nisidh left a comment

Choose a reason for hiding this comment

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

May we can add wiki page for how to use it for end users.

@xiangyushawn
Copy link
Copy Markdown
Contributor

closing this PR since new PR is created #425 . Thank you @v-suhame for the brilliant idea 👍

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.

6 participants