Merge pull request #2022 from rkoumis/test-agent-exist-ok-true

Ask the agent to create a directory that already exists

agent/test_agent.py

@@ -369,14 +369,18 @@ def test_mkdir_valid(self):
         assert os.path.exists(new_dir)
         assert os.path.isdir(new_dir)
 
-    def test_mkdir_invalid(self):
+    def test_mkdir_missing(self):
         """Ensure we get an error returned when the mkdir request fails."""
         form = {}
         js = self.post_form("mkdir", form, 400)
         assert js["message"] == "No dirpath has been provided"
 
-        root = pathlib.Path(tempfile.gettempdir()).root
-        form = {"dirpath": root, "mode": 0o777}
+    @pytest.mark.skip("Not many paths are actually invalid")
+    def test_mkdir_invalid(self):
+        """Ensure we get an error returned when the mkdir request fails."""
+        # TODO come up with an invalid directory path for windows / linux
+        invalid = ""
+        form = {"dirpath": invalid, "mode": 0o777}
         js = self.post_form("mkdir", form, 500)
         assert js["message"] == "Error creating directory"