From 993e5147a9f98c14b678ee2a116c29a61d7de8c3 Mon Sep 17 00:00:00 2001 From: Michael Savigny Date: Tue, 13 Feb 2024 20:33:21 +0000 Subject: [PATCH] Remove extra prints I forgot to in previous cl. MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Had some debugging logging in autoninjas test. Removed it. Lines were erroneously added in https://crrev.com/c/5272474 to address b/277197166 Lines initially added to debug a problem in during development. In the initial review I received a +1 CR with a comment requesting they be removed. I made the change, started the patchset upload, went to a meeting, forgot to enter the patchset comment (thinking I had done it before the meeting) and submitted the change without the patchset. Basically context switching got me 😅 Bug: b/277197166 Change-Id: I504569c713dbca7302988c39230caf9e9e6fa2c0 Reviewed-on: https://chromium-review.googlesource.com/c/chromium/tools/depot_tools/+/5292094 Reviewed-by: Bruce Dawson Reviewed-by: Ben Segall Commit-Queue: Bruce Dawson Auto-Submit: Michael Savigny --- tests/autoninja_test.py | 4 ---- 1 file changed, 4 deletions(-) diff --git a/tests/autoninja_test.py b/tests/autoninja_test.py index 689f7c980..a3c065d54 100755 --- a/tests/autoninja_test.py +++ b/tests/autoninja_test.py @@ -91,8 +91,6 @@ class AutoninjaTest(trial_dir.TestCase): with self.assertRaises(SystemExit): self.assertEqual( autoninja.main(['autoninja.py', '-C', out_dir]), 1) - self.maxDiff = None - print(f.getvalue()) self.assertIn( "The gn arg `use_goma=true` is no longer supported.", f.getvalue()) @@ -115,8 +113,6 @@ class AutoninjaTest(trial_dir.TestCase): with self.assertRaises(SystemExit): self.assertEqual( autoninja.main(['autoninja.py', '-C', out_dir]), 1) - self.maxDiff = None - print(f.getvalue()) self.assertIn( "The gn arg `use_goma=true` is no longer supported.", f.getvalue())