Skip to content

Commit fea0be7

Browse files
jacobaustin123copybara-github
authored andcommitted
CL fixes for pytype and pylint.
PiperOrigin-RevId: 339893046 Change-Id: I9df74fefee03eb801262bccdf1f92999453bfb85
1 parent 5aa754a commit fea0be7

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

fire/main_test.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -78,7 +78,7 @@ def testFileNameModuleDuplication(self):
7878
def testFileNameModuleFileFailure(self):
7979
# Confirm that an invalid file that masks a non-existent module fails.
8080
with self.assertRaisesRegex(ValueError,
81-
r'Fire can only be called on \.py files\.'):
81+
r'Fire can only be called on \.py files\.'): # pylint: disable=line-too-long # pytype: disable=attribute-error
8282
dirname = os.path.dirname(self.file.name)
8383
with testutils.ChangeDirectory(dirname):
8484
with open('foobar', 'w'):

fire/testutils.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -73,11 +73,11 @@ def assertOutputMatches(self, stdout='.*', stderr='.*', capture=True):
7373
raise AssertionError('%s: Expected %r to match %r' %
7474
(name, value, regexp))
7575

76-
def assertRaisesRegex(self, *args, **kwargs):
76+
def assertRaisesRegex(self, *args, **kwargs): # pylint: disable=arguments-differ
7777
if sys.version_info.major == 2:
7878
return super(BaseTestCase, self).assertRaisesRegexp(*args, **kwargs) # pylint: disable=deprecated-method
7979
else:
80-
return super(BaseTestCase, self).assertRaisesRegex(*args, **kwargs) # pylint: disable=no-member,arguments-differ
80+
return super(BaseTestCase, self).assertRaisesRegex(*args, **kwargs) # pylint: disable=no-member
8181

8282
@contextlib.contextmanager
8383
def assertRaisesFireExit(self, code, regexp='.*'):

0 commit comments

Comments
 (0)