From 38cf79bdfa05416ec1e18470a15beb916a66155a Mon Sep 17 00:00:00 2001 From: Cory Knox Date: Fri, 8 Nov 2024 07:49:22 -0800 Subject: [PATCH] (#3526) Remove test for an empty debug log 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 #3555 has been created to determine what we want for behaviour when limit output and debug is selected. --- src/chocolatey.tests.integration/scenarios/ListScenarios.cs | 6 ------ 1 file changed, 6 deletions(-) diff --git a/src/chocolatey.tests.integration/scenarios/ListScenarios.cs b/src/chocolatey.tests.integration/scenarios/ListScenarios.cs index 01818fdcb..0e01deb95 100644 --- a/src/chocolatey.tests.integration/scenarios/ListScenarios.cs +++ b/src/chocolatey.tests.integration/scenarios/ListScenarios.cs @@ -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