Skip to content

Travis CI: Run each failing pytest in allow_failures mode - #1087

Merged
cclauss merged 4 commits into
TheAlgorithms:masterfrom
cclauss:travis-add-failing-pytests
Aug 6, 2019
Merged

Travis CI: Run each failing pytest in allow_failures mode#1087
cclauss merged 4 commits into
TheAlgorithms:masterfrom
cclauss:travis-add-failing-pytests

Conversation

@cclauss

@cclausscclauss commented Jul 30, 2019

Copy link
Copy Markdown
Member

This PR allows us to see which of our files are still fail pytests. As we fix each file so that it passes pytest, we can remove the custom configuration to simplify our testing script. This PR replaces #1055 by running each of our currently failing pytests in allow_failures mode so that we can see them fail without failing the entire Travis CI run.

Complex now but simple once all files pass pytest.

@cclauss
cclaussforce-pushed the travis-add-failing-pytests branch from 343ebb4 to 429909bCompareJuly 30, 2019 15:45
@cclausscclauss changed the title Travis CI: Run each failing pytest in || true modeTravis CI: Run each failing pytest in allow_failures modeJul 30, 2019
@cclauss
cclauss requested a review from AnshulMalikJuly 30, 2019 15:51
@cclausscclauss added the awaiting reviews This PR is ready to be reviewed label Jul 30, 2019

@harshildarjiharshildarji left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good to me!

@harshildarjiharshildarji added awaiting merge This PR is approved and ready to be merged and removed awaiting reviews This PR is ready to be reviewed labels Aug 6, 2019
@cclauss
cclauss merged commit 561a414 into TheAlgorithms:masterAug 6, 2019
stokhos pushed a commit to stokhos/Python that referenced this pull request Jan 3, 2021
…hms#1087)
* Travis CI: Run failing pytest in allow_failures mode
* Sync with master
* Sync with master
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting mergeThis PR is approved and ready to be merged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@cclauss@harshildarji