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 test: Make sure FsharpSuite test failure stops and fails whole CI job #14743

Merged
merged 10 commits into from
Feb 16, 2023

Conversation

T-Gro
Copy link
Member

@T-Gro T-Gro commented Feb 13, 2023

No description provided.

@T-Gro
Copy link
Member Author

T-Gro commented Feb 13, 2023

Do not update, do not merge.
I want it to see fail first.

@T-Gro
Copy link
Member Author

T-Gro commented Feb 13, 2023

/azp run

@azure-pipelines
Copy link

Pull request contains merge conflicts.

@T-Gro
Copy link
Member Author

T-Gro commented Feb 14, 2023

Now it fails as it should:

image

Next step, I am going to re-ignore the failing test, so that CI is green and this can get merged.

@T-Gro T-Gro marked this pull request as ready for review February 14, 2023 09:04
@T-Gro T-Gro requested a review from a team as a code owner February 14, 2023 09:04
This is the suite of tests which I am migrating to componenttests with another (in process) mode of execution.

Will make sure those are run there.
@KevinRansom KevinRansom merged commit 979fa51 into dotnet:main Feb 16, 2023
@T-Gro T-Gro deleted the fsharpsuite-fial-CI-if-fails branch February 16, 2023 19:22
KevinRansom added a commit to KevinRansom/fsharp that referenced this pull request Feb 20, 2023
KevinRansom added a commit to KevinRansom/fsharp that referenced this pull request Feb 20, 2023
kant2002 pushed a commit to kant2002/fsharp that referenced this pull request Apr 1, 2023
…job (dotnet#14743)

* If background test run fails, stop entire run

* Background job for running tests passed "Exit" on

* TESTONLY:Let this test fail

* Make test ignored again

* Ignore tests in this PR (to enable FshapSuite failiure)

This is the suite of tests which I am migrating to componenttests with another (in process) mode of execution.

Will make sure those are run there.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

4 participants