Skip to content

Commit

Permalink
(chocolatey#3526) Remove test for an empty debug log
Browse files Browse the repository at this point in the history
The debug log is used in a lot of places when we don't want the user to
see the output, but we want it available. At the moment it is not
consistent about what we do when they select debug and limit output. It
has been decided at the moment to remove these tests as they are now
breaking by us emitting the user agent to the debug log. A follow up
issue chocolatey#3555 has been created to determine what we want for behaviour
when limit output and debug is selected.
  • Loading branch information
corbob committed Nov 8, 2024
1 parent be0ae44 commit 38cf79b
Showing 1 changed file with 0 additions and 6 deletions.
6 changes: 0 additions & 6 deletions src/chocolatey.tests.integration/scenarios/ListScenarios.cs
Original file line number Diff line number Diff line change
Expand Up @@ -163,12 +163,6 @@ public void Should_not_contain_a_summary()
{
MockLogger.ContainsMessage("packages installed").Should().BeFalse();
}

[Fact]
public void Should_not_contain_debugging_messages()
{
MockLogger.Messages.Should().NotContainKey(LogLevel.Debug.ToStringSafe());
}
}

public class When_listing_local_packages_limiting_output_with_id_only : ScenariosBase
Expand Down

0 comments on commit 38cf79b

Please sign in to comment.