feat: optionally disable config bootstrap #75
Conversation
|
Nevermind, it still stored the config 🤔 |
Huh, weird. This totally looks like it'd work. Maybe you're not using the latest archive when looking for the stored config? |
|
Oh, maybe this changeset only prevents storage of the manifest but doesn't change whether the actual config files get stored? |
Yup
On it! |
| @@ -646,6 +646,53 @@ def test_create_archive_with_sources_and_used_config_paths_calls_borg_with_sourc | |||
| global_arguments=flexmock(log_json=False, used_config_paths=['/etc/borgmatic/config.yaml']), | |||
| ) | |||
|
|
|||
| def test_create_archive_with_sources_and_used_config_paths_with_store_config_files_false_calls_borg_with_sources_and_no_config_paths(): | |||
There was a problem hiding this comment.
Disclaimer: this is Copilot generated. I just took a glance (lines 691 and 677) and it made sense + it passed.
|
The test looks fine to me (even though that function name makes even me blush), but what about a test for the manifest code path? |
|
Done! |
|
Awesome, thank you! |
#725