diff --git a/armada/tests/unit/utils/test_validate.py b/armada/tests/unit/utils/test_validate.py
index 4dc03262..3267adc2 100644
--- a/armada/tests/unit/utils/test_validate.py
+++ b/armada/tests/unit/utils/test_validate.py
@@ -218,13 +218,6 @@ data:
                 'airship-armada/master/examples/simple.yaml'
         self.assertTrue(validate.validate_manifest_url(value))
 
-    def test_validate_manifest_filepath(self):
-        value = 'filepath'
-        self.assertFalse(validate.validate_manifest_filepath(value))
-        value = '{}/resources/valid_armada_document.yaml'.format(
-            self.basepath)
-        self.assertTrue(validate.validate_manifest_filepath(value))
-
 
 class ValidateNegativeTestCase(BaseValidateTest):
 
diff --git a/armada/utils/validate.py b/armada/utils/validate.py
index 17bd99e3..0dd602a9 100644
--- a/armada/utils/validate.py
+++ b/armada/utils/validate.py
@@ -234,10 +234,5 @@ def validate_manifest_url(value):
         return False
 
 
-# TODO(MarshM) unused except in unit tests, is this useful?
-def validate_manifest_filepath(value):
-    return os.path.isfile(value)
-
-
 # Fill the cache.
 _load_schemas()