Skip to content

Commit

Permalink
Exclude newly introduced tests to check if they are causing the perfo…
Browse files Browse the repository at this point in the history
…rmance problems when the integration tests are run after pushing to the PR
  • Loading branch information
FrankRay78 committed May 22, 2023
1 parent ba4e83f commit 3b51ae7
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion src/Cake.Frosting.Tests/CakeHostTests.cs
Original file line number Diff line number Diff line change
Expand Up @@ -384,7 +384,7 @@ public void The_Version_Option_Should_Call_Version_Feature()
Assert.Collection(fixture.Console.Messages, s => s.Equals("FakeVersion"));
}

[Fact]
[Fact(Skip = "Excluded to see if the performance implication of this test is causing the integration tests to break on push to PR")]
public void Should_Pass_Cake_Runner_Argument_And_Value_To_Build_Script()
{
// Given
Expand Down
2 changes: 1 addition & 1 deletion src/Cake.Tests/Unit/ProgramTests.cs
Original file line number Diff line number Diff line change
Expand Up @@ -132,7 +132,7 @@ public async Task The_Info_Option_Should_Call_Info_Feature(params string[] args)
feature.Received(1).Run(fixture.Console);
}

[Fact]
[Fact(Skip = "Excluded to see if the performance implication of this test is causing the integration tests to break on push to PR")]
public async void Should_Pass_Cake_Runner_Argument_And_Value_To_Build_Script()
{
// Given
Expand Down

0 comments on commit 3b51ae7

Please sign in to comment.