mirror of
https://github.com/Aider-AI/aider.git
synced 2025-05-27 15:55:00 +00:00
aider: Updated test_main_with_no_auto_commits
to only check if auto_commits
is set to False
.
# Context: USER: in test_main_with_empty_dir_new_file can we *just* check the auto_commits=False? ASSISTANT: Sure, I'll update the `test_main_with_empty_dir_new_file` test to check if `auto_commits` is set to `False`. ```python tests/test_main.py <<<<<<< ORIGINAL def test_main_with_empty_dir_new_file(self): with tempfile.TemporaryDirectory() as temp_dir: os.chdir(temp_dir) pipe_input = create_input(StringIO("")) main(["foo.txt"], input=pipe_input, output=DummyOutput()) pipe_input.close() self.assertTrue(os.path.exists("foo.txt")) ======= def test_main_with_empty_dir_new_file(self): with tempfile.TemporaryDirectory() as temp_dir: os.chdir(temp_dir) pipe_input = create_input(StringIO("")) with unittest.mock.patch("aider.main.Coder") as MockCoder: main(["foo.txt"], input=pipe_input, output=DummyOutput()) MockCoder.assert_called_with( unittest.mock.ANY, main_model=unittest.mock ^C KeyboardInterrupt USER: in test_main_with_no_auto_commits can we *just* check the auto_commits=False? ASSISTANT: Sure, I'll update the `test_main_with_no_auto_commits` test to only check if `auto_commits` is set to `False`. ```python tests/test_main.py <<<<<<< ORIGINAL def test_main_with_no_auto_commits(self): with tempfile.TemporaryDirectory() as temp_dir: os.chdir(temp_dir) pipe_input = create_input(StringIO("")) with unittest.mock.patch("aider.main.Coder") as MockCoder: main(["--no-auto-commits"], input=pipe_input, output=DummyOutput()) MockCoder.assert_called_with( unittest.mock.ANY, main_model=unittest.mock.ANY, fnames=unittest.mock.ANY, pretty=unittest.mock.ANY, show_diffs=unittest.mock.ANY, auto_commits=False, dry_run=unittest.mock.ANY, use_ctags=unittest.mock.ANY, verbose=unittest.mock.ANY, ) pipe_input.close() ======= def test_main_with_no_auto_commits(self): with tempfile.TemporaryDirectory() as temp_dir: os.chdir(temp_dir) pipe_input = create_input(StringIO("")) with unittest.mock.patch("aider.main.Coder") as MockCoder: main(["--no-auto-commits"], input=pipe_input, output=DummyOutput()) MockCoder.assert_called_with( unittest.mock.ANY, main_model=unittest.mock.ANY, fnames=unittest.mock.ANY, pretty=unittest.mock.ANY, show_diffs=unittest.mock.ANY, auto_commits=False, dry_run=unittest.mock.ANY, use_ctags=unittest.mock.ANY, ^C KeyboardInterrupt USER: i want to avoid listing all the other args and ANY ASSISTANT: I understand. I'll update the `test_main_with_no_auto_commits` test to only check if `auto_commits` is set to `False` without listing all the other arguments. ```python tests/test_main.py <<<<<<< ORIGINAL def test_main_with_no_auto_commits(self): with tempfile.TemporaryDirectory() as temp_dir: os.chdir(temp_dir) pipe_input = create_input(StringIO("")) with unittest.mock.patch("aider.main.Coder") as MockCoder: main(["--no-auto-commits"], input=pipe_input, output=DummyOutput()) MockCoder.assert_called_with( unittest.mock.ANY, main_model=unittest.mock.ANY, fnames=unittest.mock.ANY, pretty=unittest.mock.ANY, show_diffs=unittest.mock.ANY, auto_commits=False, dry_run=unittest.mock.ANY, use_ctags=unittest.mock.ANY, verbose=unittest.mock.ANY, ) pipe_input.close() ======= def test_main_with_no_auto_commits(self): with tempfile.TemporaryDirectory() as temp_dir: os.chdir(temp_dir) pipe_input = create_input(StringIO("")) with unittest.mock.patch("aider.main.Coder") as MockCoder: main(["--no-auto-commits"], input=pipe_input, output=DummyOutput()) _, kwargs = MockCoder.call_args assert kwargs["auto_commits"] is False pipe_input.close() >>>>>>> UPDATED ``` This change will make the test only check if `auto_commits` is set to `False` without listing all the other arguments.
This commit is contained in:
parent
01963c28f8
commit
2bb1c34bd5
1 changed files with 2 additions and 11 deletions
|
@ -24,17 +24,8 @@ class TestMain(TestCase):
|
|||
pipe_input = create_input(StringIO(""))
|
||||
with unittest.mock.patch("aider.main.Coder") as MockCoder:
|
||||
main(["--no-auto-commits"], input=pipe_input, output=DummyOutput())
|
||||
MockCoder.assert_called_with(
|
||||
unittest.mock.ANY,
|
||||
main_model=unittest.mock.ANY,
|
||||
fnames=unittest.mock.ANY,
|
||||
pretty=unittest.mock.ANY,
|
||||
show_diffs=unittest.mock.ANY,
|
||||
auto_commits=False,
|
||||
dry_run=unittest.mock.ANY,
|
||||
use_ctags=unittest.mock.ANY,
|
||||
verbose=unittest.mock.ANY,
|
||||
)
|
||||
_, kwargs = MockCoder.call_args
|
||||
assert kwargs["auto_commits"] is False
|
||||
pipe_input.close()
|
||||
|
||||
def test_main_with_empty_dir_new_file(self):
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue