Browse Source

Removing TODO that basically entails testing ruamel.yaml round-tripping, which in theory already has its own tests.

Dan Helfman 8 years ago
parent
commit
ac2a63763f
2 changed files with 1 additions and 4 deletions
  1. 0 3
      borgmatic/commands/convert_config.py
  2. 1 1
      borgmatic/config/generate.py

+ 0 - 3
borgmatic/commands/convert_config.py

@@ -93,9 +93,6 @@ def main():  # pragma: no cover
             mode=source_config_file_mode,
             mode=source_config_file_mode,
         )
         )
 
 
-        # TODO: As a backstop, check that the written config can actually be read and parsed, and
-        # that it matches the destination config data structure that was written.
-
         display_result(args)
         display_result(args)
     except (ValueError, OSError) as error:
     except (ValueError, OSError) as error:
         print(error, file=sys.stderr)
         print(error, file=sys.stderr)

+ 1 - 1
borgmatic/config/generate.py

@@ -50,7 +50,7 @@ def write_configuration(config_filename, config, mode=0o600):
 
 
     try:
     try:
         os.makedirs(os.path.dirname(config_filename), mode=0o700)
         os.makedirs(os.path.dirname(config_filename), mode=0o700)
-    except FileExistsError:
+    except (FileExistsError, FileNotFoundError):
         pass
         pass
 
 
     with open(config_filename, 'w') as config_file:
     with open(config_filename, 'w') as config_file: