stricter command line checks, including no file dups

Author: JohnMount

coverage.txt

@@ -1,5 +1,5 @@
 ============================= test session starts ==============================
-platform darwin -- Python 3.9.12, pytest-7.1.1, pluggy-1.0.0
+platform darwin -- Python 3.9.7, pytest-6.2.5, py-1.11.0, pluggy-1.0.0
 rootdir: /Users/johnmount/Documents/work/wvpy/pkg
 plugins: anyio-3.5.0, cov-3.0.0
 collected 19 items
@@ -19,26 +19,16 @@ tests/test_stats1.py .                                                   [ 89%]
 tests/test_threshold_stats.py .                                          [ 94%]
 tests/test_typs_in_frame.py .                                            [100%]
 
-=============================== warnings summary ===============================
-../../../../opt/anaconda3/envs/research_env/lib/python3.9/site-packages/seaborn/rcmod.py:82
-  /Users/johnmount/opt/anaconda3/envs/research_env/lib/python3.9/site-packages/seaborn/rcmod.py:82: DeprecationWarning: distutils Version classes are deprecated. Use packaging.version instead.
-    if LooseVersion(mpl.__version__) >= "3.0":
-
-../../../../opt/anaconda3/envs/research_env/lib/python3.9/site-packages/setuptools/_distutils/version.py:351
-  /Users/johnmount/opt/anaconda3/envs/research_env/lib/python3.9/site-packages/setuptools/_distutils/version.py:351: DeprecationWarning: distutils Version classes are deprecated. Use packaging.version instead.
-    other = LooseVersion(other)
-
--- Docs: https://docs.pytest.org/en/stable/how-to/capture-warnings.html
-
----------- coverage: platform darwin, python 3.9.12-final-0 ----------
+---------- coverage: platform darwin, python 3.9.7-final-0 -----------
 Name                      Stmts   Miss  Cover
 ---------------------------------------------
 wvpy/__init__.py              3      0   100%
 wvpy/jtools.py              136     34    75%
-wvpy/pysheet.py              58     58     0%
-wvpy/render_workbook.py      29     29     0%
+wvpy/pysheet.py              62     62     0%
+wvpy/render_workbook.py      35     35     0%
 wvpy/util.py                321      7    98%
 ---------------------------------------------
-TOTAL                       547    128    77%
+TOTAL                       557    138    75%
+
 
-======================= 19 passed, 2 warnings in 14.35s ========================
+============================= 19 passed in 23.71s ==============================

pkg/build/lib/wvpy/pysheet.py

@@ -14,14 +14,22 @@ def main() -> int:
     parser = argparse.ArgumentParser(description="Convert between .py and .ipynb")
     parser.add_argument('--quiet', action='store_true', help='delete input file')
     parser.add_argument('--delete', action='store_true', help='delete input file')
-    parser.add_argument('infile', metavar='infile', type=str, nargs='+',
+    parser.add_argument(
+        'infile', 
+        metavar='infile', 
+        type=str, 
+        nargs='+',
         help='name of input file(s)')
     args = parser.parse_args()
-    assert len(args.infile) > 0
     # some pre-checks
+    assert len(args.infile) > 0
+    assert len(set(args.infile)) == len(args.infile)
+    assert isinstance(args.quiet, bool)
+    assert isinstance(args.delete, bool)
     input_suffices_seen = set()
     for input_file_name in args.infile:
         assert isinstance(input_file_name, str)
+        assert len(input_file_name) > 0
         assert os.path.exists(input_file_name)
         if input_file_name.endswith('.py'):
             input_suffices_seen.add('.py')

pkg/build/lib/wvpy/render_workbook.py

@@ -12,15 +12,27 @@
 
 def main() -> int:
     parser = argparse.ArgumentParser(description="Render .py or .ipynb to .html by executing in Jupyter")
-    parser.add_argument('infile', metavar='infile', type=str, nargs='+',
+    parser.add_argument(
+        'infile', 
+        metavar='infile', 
+        type=str, 
+        nargs='+',
         help='name of input file(s)')
     parser.add_argument('--strip_input', action='store_true')
     parser.add_argument('--quiet', action='store_true')
     args = parser.parse_args()
+    # checks
+    assert isinstance(args.quiet, bool)
+    assert isinstance(args.strip_input, bool)
     assert len(args.infile) > 0
+    assert len(set(args.infile)) == len(args.infile)
     for input_file_name in args.infile:
         assert isinstance(input_file_name, str)
+        assert len(input_file_name) > 0
+        assert input_file_name.endswith('.py') or input_file_name.endswith('.ipynb')
         assert os.path.exists(input_file_name)
+    # do the work
+    for input_file_name in args.infile:
         render_as_html(input_file_name, exclude_input=args.strip_input, verbose=args.quiet == False)
     return 0