Skip to content

Commit

Permalink
ArgSuite: do not test options.config_file
Browse files Browse the repository at this point in the history
  • Loading branch information
elazarg committed Feb 2, 2018
1 parent 83b6c85 commit a2ebf80
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 1 deletion.
2 changes: 1 addition & 1 deletion mypy/options.py
Original file line number Diff line number Diff line change
Expand Up @@ -131,7 +131,7 @@ def __init__(self) -> None:
self.scripts_are_modules = False

# Config file name
self.config_file = 'setup.cfg' # type: Optional[str]
self.config_file = None # type: Optional[str]

# Write junit.xml to given file
self.junit_xml = None # type: Optional[str]
Expand Down
2 changes: 2 additions & 0 deletions mypy/test/testargs.py
Original file line number Diff line number Diff line change
Expand Up @@ -14,4 +14,6 @@ class ArgSuite(Suite):
def test_coherence(self) -> None:
options = Options()
_, parsed_options = process_options([], require_targets=False)
# FIX: test this too. Requires changing working dir to avoid finding 'setup.cfg'
options.config_file = parsed_options.config_file
assert_equal(options, parsed_options)

0 comments on commit a2ebf80

Please sign in to comment.