diff options
author | Jian Cai <jiancai@google.com> | 2020-01-16 11:12:54 -0800 |
---|---|---|
committer | Jian Cai <jiancai@google.com> | 2020-01-17 01:49:12 +0000 |
commit | b47813efda60622787afcb0a9bc3070031ac4e9f (patch) | |
tree | ae18e7ccf2c130da1dddb5b2588d5088aa0866b3 /llvm_tools/patch_manager_unittest.py | |
parent | 0410b6515691a2fd3410917849d32495490d6999 (diff) | |
download | toolchain-utils-b47813efda60622787afcb0a9bc3070031ac4e9f.tar.gz |
patch_manager: appease pylint
Fix unused-argument complaints in unit tests.
BUG=chromium:1042953
TEST=verified locally.
Change-Id: I7037fc10f8160a5f1f8929a3256e91255dd125ac
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/third_party/toolchain-utils/+/2006330
Reviewed-by: George Burgess <gbiv@chromium.org>
Reviewed-by: Manoj Gupta <manojgupta@chromium.org>
Tested-by: Jian Cai <jiancai@google.com>
Diffstat (limited to 'llvm_tools/patch_manager_unittest.py')
-rwxr-xr-x | llvm_tools/patch_manager_unittest.py | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/llvm_tools/patch_manager_unittest.py b/llvm_tools/patch_manager_unittest.py index 90200ab8..688c2fe6 100755 --- a/llvm_tools/patch_manager_unittest.py +++ b/llvm_tools/patch_manager_unittest.py @@ -14,11 +14,11 @@ import subprocess import unittest import unittest.mock as mock +import patch_manager from failure_modes import FailureModes from test_helpers import CallCountsToMockFunctions from test_helpers import CreateTemporaryJsonFile from test_helpers import WritePrettyJsonFile -import patch_manager class PatchManagerTest(unittest.TestCase): @@ -326,8 +326,8 @@ class PatchManagerTest(unittest.TestCase): # Simulate behavior of 'GetPathToPatch()' when the absolute path to the # patch does not exist. def PathToPatchDoesNotExist(filesdir_path, rel_patch_path): - raise ValueError( - 'The absolute path to %s does not exist' % abs_patch_path) + raise ValueError('The absolute path to %s does not exist' % os.path.join( + filesdir_path, rel_patch_path)) # Use the test function to simulate the behavior of 'GetPathToPatch()'. mock_get_path_to_patch.side_effect = PathToPatchDoesNotExist @@ -463,7 +463,7 @@ class PatchManagerTest(unittest.TestCase): # Simulate behavior for 'ApplyPatch()' when applying multiple applicable # patches. @CallCountsToMockFunctions - def MultipleCallsToApplyPatches(call_count, src_path, path_to_patch): + def MultipleCallsToApplyPatches(call_count, _, path_to_patch): if call_count < 3: self.assertEqual( path_to_patch, @@ -569,7 +569,7 @@ class PatchManagerTest(unittest.TestCase): # Simulate behavior for 'ApplyPatch()' when applying multiple applicable # patches. @CallCountsToMockFunctions - def MultipleCallsToApplyPatches(call_count, src_path, path_to_patch): + def MultipleCallsToApplyPatches(call_count, _, path_to_patch): if call_count < 3: self.assertEqual( path_to_patch, |