Sanity checks now have multiple modes (file, file-list, dir, all volumes)

Author: tomaszmrugalski

CHANGELOG.md

@@ -12,6 +12,9 @@
 - Unit tests for config, CLI, login parsing, scope resolution, and URL helpers
 - `check` command renamed to `doctor`
 - `repo` command renamed to `volumes`
+- the 'volumes` command can now scan files in multiple modes: file (-f, or --file), file-list (-l, or --list),
+  directory (-d, or --dir) or all volumes (--sanity-files)
+
 
 0.0.1
 

src/cmd_volumes.py

@@ -476,11 +476,11 @@ def sanity_db(cm: ConfigManager, args) -> int:
 def cmd_volumes(cm: ConfigManager, args) -> int:
     """Manages the on disk images repository."""
 
-    if args.sanity_files:
+    if args.file or args.list or args.dir or args.all_files:
         return sanity_files(cm, args)
 
     if args.sanity_db:
         return sanity_db(cm, args)
 
-    print("ERROR: No sanity check selected. Use --sanity-files or --sanity-db to check the repository.")
+    print("ERROR: No sanity check selected. Use -d (--dir) or -l (--list) or -f (--file) or -a (--all-files) or --sanity-db to check the repository.")
     return 1

src/hevelius-runner.py

@@ -359,8 +359,8 @@ def build_parser() -> argparse.ArgumentParser:
     repo_parser.add_argument("-d", "--dir",   help="Check all FITS files recursively in this specific directory", type=str)
     repo_parser.add_argument("-s", "--show-header", help="Displays all entries in FITS header for each file", action='store_true')
     repo_parser.add_argument("-t", "--dry-run", help="Don't do the actual DB upsert", action='store_true')
+    repo_parser.add_argument("-a", "--all-files", help="Check files against tasks; defaults to scanning all configured paths.volumes when no --file/--list/--dir is given", action='store_true')
     repo_parser.add_argument("--sanity-db", help="Goes through the list of tasks in a database and checks if all files are present", action='store_true')
-    repo_parser.add_argument("--sanity-files", help="Check files against tasks; defaults to scanning all configured paths.volumes when no --file/--list/--dir is given", action='store_true')
     repo_parser.add_argument("--min-task-id", help="Minimum task ID to check (for sanity-db)", type=int)
     repo_parser.add_argument("--max-task-id", help="Maximum task ID to check (for sanity-db)", type=int)
     repo_parser.add_argument("--delete-invalid", help="Delete invalid tasks (no filename or missing file) when using sanity-db", action='store_true')