wrote tests on no additional arguments, 1 and 2 arguments. need to test for invalid arguments.

Author: yucongalicechen

src/diffpy/labpdfproc/labpdfprocapp.py

@@ -62,8 +62,8 @@ def get_args():
         "--additional-info",
         metavar=("KEY=VALUE"),
         action="append",
-        help="specify key-value pairs to be loaded into metadata. You can specify multiple "
-        "paris by calling -add multiple times",
+        help="specify key-value pairs to be loaded into metadata by using key=value. "
+        "You can specify multiple paris by calling -add multiple times.",
     )
     args = p.parse_args()
     return args

src/diffpy/labpdfproc/tests/test_tools.py

@@ -0,0 +1,30 @@
+from argparse import ArgumentParser
+
+import pytest
+
+from diffpy.labpdfproc.tools import load_additional_info
+
+params5 = [
+    ([], []),
+    ([["weather=rainy"]], [["weather", "rainy"]]),
+    ([["weather=rainy", "day=tuesday"]], [["weather", "rainy"], ["day", "tuesday"]]),
+]
+
+
+@pytest.mark.parametrize("inputs, expected", params5)
+def test_load_additional_info(inputs, expected):
+    actual_parser = ArgumentParser()
+    actual_parser.add_argument("-add", "--additional-info", action="append", metavar=("KEY=VALUE"))
+    actual_args = actual_parser.parse_args([])
+    expected_parser = ArgumentParser()
+    expected_args = expected_parser.parse_args([])
+
+    if not inputs:
+        actual_args = load_additional_info(actual_args)
+        assert actual_args == expected_args
+    else:
+        setattr(actual_args, "additional_info", inputs[0])
+        actual_args = load_additional_info(actual_args)
+        for expected_pair in expected:
+            setattr(expected_args, expected_pair[0], expected_pair[1])
+        assert actual_args == expected_args