commit 8b4445383b4c0467bee1c20e63167ca438ff9b68
author Alex Klein <saklein@chromium.org> Tue Apr 11 16:35:24 2023 -0600
committer Chromeos LUCI <chromeos-scoped@luci-project-accounts.iam.gserviceaccount.com> Thu Apr 27 18:29:04 2023 +0000
tree 0ff2ca0dbd3bdf8190a0e22e434bc4051b787499
parent e37b8768ee121f196a7e2ef83378622cf8a4a56f

scripts: Lint fixes.

line-too-long and docstring-section-indent fixes. Misc spelling
and grammar fixes.

BUG=None
TEST=CQ

Change-Id: I1969b189b8b007db65d10327729b8777c83597c6
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/chromite/+/4416213
Auto-Submit: Alex Klein <saklein@chromium.org>
Reviewed-by: Anuj Jamwal <anujjamwal@google.com>
Tested-by: Alex Klein <saklein@chromium.org>
Commit-Queue: Anuj Jamwal <anujjamwal@google.com>

scripts/cros_fuzz_unittest.py

diff --git a/scripts/cros_fuzz_unittest.py b/scripts/cros_fuzz_unittest.py
index c0ef936..2fdf37d 100644
--- a/scripts/cros_fuzz_unittest.py
+++ b/scripts/cros_fuzz_unittest.py
@@ -123,10 +123,10 @@
         self.corpus = None
 
     def _Helper(self, expected_command=None):
-        """Calls LimitFuzzing and asserts fuzz_command equals |expected_command|.
+        """Call LimitFuzzing and assert fuzz_command equals |expected_command|.
 
-        If |expected| is None, then it is set to self.fuzz_command before calling
-        LimitFuzzing.
+        If |expected| is None, then it is set to self.fuzz_command before
+        calling LimitFuzzing.
         """
         if expected_command is None:
             expected_command = self.fuzz_command[:]
@@ -150,7 +150,7 @@
         self._Helper(expected)
 
     def testNoLimitOrCorpus(self):
-        """Tests that a limit is added when user specifies no corpus or limit."""
+        """Test a limit is added when user specifies no corpus or limit."""
         expected = self.fuzz_command + [DEFAULT_MAX_TOTAL_TIME_OPTION]
         self._Helper(expected)
 
@@ -210,7 +210,7 @@
         self._Helper()
 
     def testFuzzArgs(self):
-        """Tests that the correct command is used when fuzz_args is specified."""
+        """Test the correct command is used when fuzz_args is specified."""
         fuzz_args = [DEFAULT_MAX_TOTAL_TIME_OPTION, "-fake_arg=fake_value"]
         self.expected_command.extend(fuzz_args)
         self.fuzz_args = " ".join(fuzz_args)
@@ -294,7 +294,7 @@
     """Tests RunSysrootCommand."""
 
     def testRunSysrootCommand(self):
-        """Tests RunSysrootCommand creates a proper command to run in sysroot."""
+        """Test RunSysrootCommand creates a proper command to run in sysroot."""
         command = ["./fuzz", "-rss_limit_mb=4096"]
         sysroot_path = _SetPathToSysroot()
         cros_fuzz.RunSysrootCommand(command)
@@ -309,7 +309,7 @@
     TEST_ENV_VAR = "TEST_VAR"
 
     def testUseAndFeaturesNotClobbered(self):
-        """Tests that values of certain environment variables are appended to."""
+        """Test values of certain environment variables are appended to."""
         vars_and_values = {"FEATURES": "foo", "USE": "bar"}
         for var, value in vars_and_values.items():
             os.environ[var] = value