fix: save_config stringifies JSON before writing

[andrewwhitecdw]

This PR addresses the following issue in bobber/bobber.py: save_config stringifies JSON before writing.

Changes

• bobber/bobber.py: save_config stringifies JSON before writing.

Details

--- a/bobber/bobber.py
+++ b/bobber/bobber.py
@@ -1,1 +1,1 @@
-        fp.write(json.dumps(settings))
+        json.dump(settings, fp)

Tests

• tests/test_bobber_save_config.py

--- /dev/null
+++ b/tests/test_bobber_save_config.py
@@ -0,0 +1,15 @@
+import unittest
+from argparse import Namespace
+from unittest.mock import mock_open, patch
+import bobber.bobber as bobber_module
+
+
+class TestSaveConfig(unittest.TestCase):
+    def test_save_config_uses_json_dump(self):
+        args = Namespace(log_path='/tmp', command='run')
+        with patch('builtins.open', mock_open()), \
+             patch.object(bobber_module.json, 'dump') as dump_mock:
+            bobber_module.save_config(args)
+        dump_mock.assert_called_once()

Contributor guidelines

Per this repo's CONTRIBUTING.md:

• All commits are signed off (Signed-off-by trailer, DCO).

[andrewwhitecdw]

Closing per adversarial audit: branch contains off-scope or not-a-bug changes.

[andrewwhitecdw]

script closed on accident