Merge pull request #739 from kids-first/fix-faketemplates-cmd

🐛 Delete templates in faketemplates cmd if flag is given
kids-first · Sep 1, 2021 · 0131710 · 0131710
2 parents 4b6c158 + ee09f89
commit 0131710
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/creator/data_templates/management/commands/faketemplates.py b/creator/data_templates/management/commands/faketemplates.py
@@ -81,7 +81,7 @@ def add_arguments(self, parser):
         )
         parser.add_argument(
             "--delete",
-            action='store_false',
+            action='store_true',
             help="Delete all templates in the organization before creating "
             " new ones",
         )

diff --git a/tests/data_templates/test_commands.py b/tests/data_templates/test_commands.py
@@ -38,7 +38,7 @@ def test_faketemplates(db, mocker, tmpdir):
     org = OrganizationFactory()
     management.call_command("faketemplates", org.name)
     msgs = [call.args[0] for call in mock_logger.info.call_args_list]
-    assert "Deleted" in "\n".join(msgs)
+    assert "Deleted" not in "\n".join(msgs)
     mock_create_templates.assert_called_with(TEMPLATES_PATH, org)
 
     for mock in [mock_create_templates, mock_logger]:
@@ -49,10 +49,10 @@ def test_faketemplates(db, mocker, tmpdir):
     with open(fp, "w") as package_file:
         package_file.write("the package")
     management.call_command(
-        "faketemplates", org.name, delete=False, template_package=fp
+        "faketemplates", org.name, delete=True, template_package=fp
     )
     msgs = [call.args[0] for call in mock_logger.info.call_args_list]
-    assert "Deleted" not in "\n".join(msgs)
+    assert "Deleted" in "\n".join(msgs)
     mock_create_templates.assert_called_with(fp, org)