fix tests

metno · Feb 13, 2023 · c3c27a3 · c3c27a3
1 parent 8bb3724
commit c3c27a3
Show file tree

Hide file tree

Showing 7 changed files with 26 additions and 21 deletions.
diff --git a/dmci/api/worker.py b/dmci/api/worker.py
@@ -155,7 +155,7 @@ def _check_information_content(self, data):
         xml_doc = etree.fromstring(data)
         valid = self._extract_metadata_id(xml_doc)
         if not valid:
-            return False, "Input MMD XML file has no valid UUID metadata_identifier"
+            return False, "Input MMD XML file has no valid uri:UUID metadata_identifier"
 
         # Check XML file
         logger.info("Performing in depth checking.")
@@ -193,6 +193,13 @@ def _extract_metadata_id(self, xml_doc):
                 logger.info("XML file metadata_identifier UUID: %s" % fileUUID)
                 break
 
+        if fileUUID is None:
+            logger.warning("No UUID found in XML file")
+            return False
+        if namespace is None:
+            logger.warning("No namespace found in XML file")
+            return False
+
         try:
             self._file_metadata_id = uuid.UUID(fileUUID)
             logger.debug("File UUID: %s" % str(fileUUID))

diff --git a/tests/files/api/aqua-modis-parent.xml b/tests/files/api/aqua-modis-parent.xml
@@ -1,5 +1,5 @@
 <mmd:mmd xmlns:mmd="http://www.met.no/schema/mmd" xmlns:gml="http://www.opengis.net/gml">
-  <mmd:metadata_identifier>64db6102-14ce-41e9-b93b-61dbb2cb8b4e</mmd:metadata_identifier>
+  <mmd:metadata_identifier>test.no:64db6102-14ce-41e9-b93b-61dbb2cb8b4e</mmd:metadata_identifier>
   <mmd:title xml:lang="en">Direct Broadcast data processed in satellite swath to L1C.</mmd:title>
   <mmd:title xml:lang="no">Direktesendte satellittdata prosessert i satellittsveip til L1C.</mmd:title>
   <mmd:abstract xml:lang="en">Direct Broadcast data received at MET NORWAY Oslo. Processed by standard processing software to geolocated and calibrated values in satellite swath in received instrument resolution.</mmd:abstract>

diff --git a/tests/files/api/passing.xml b/tests/files/api/passing.xml
@@ -125,7 +125,7 @@
     <mmd:description>Direct download of file</mmd:description>
     <mmd:resource>https://thredds.met.no/thredds/fileServer/remotesensingsatellite/polar-swath/2021/04/29/aqua-modis-1km-20210429002844-20210429003955.nc</mmd:resource>
   </mmd:data_access>
-  <mmd:related_dataset relation_type="parent">64db6102-14ce-41e9-b93b-61dbb2cb8b4e</mmd:related_dataset>
+  <mmd:related_dataset relation_type="parent">test.no:64db6102-14ce-41e9-b93b-61dbb2cb8b4e</mmd:related_dataset>
   <mmd:storage_information>
     <mmd:file_name>aqua-modis-1km-20210429002844-20210429003955.nc</mmd:file_name>
     <mmd:file_location>/lustre/storeB/immutable/archive/projects/remotesensing/satellite-thredds/polar-swath/2021/04/29/aqua-modis-1km-20210429002844-20210429003955.nc</mmd:file_location>

diff --git a/tests/files/mmd/parent-uuid-list.xml b/tests/files/mmd/parent-uuid-list.xml
@@ -1,12 +1,12 @@
 <?xml version="1.0" encoding="utf-8"?>
 <parent>
-    <id>64db6102-14ce-41e9-b93b-61dbb2cb8b4e</id>
-    <id>bc82c179-144e-415a-8dd2-64d3569a8d50</id>
-    <id>da280021-13d8-425e-9783-64911d772397</id>
-    <id>d42548cc-337f-4005-91f4-a5dc306244b0</id>
-    <id>e1d0863d-71d3-4b9f-bb17-2af42d4956e7</id>
-    <id>6827f045-36c1-4678-a0bf-d91b41f8eefb</id>
-    <id>c7f8731b-5cfe-4cb5-ac57-168a19a2957b</id>
-    <id>f6cbb81c-1ce1-4080-b242-819e59cee78d</id>
-    <id>8e5ec6e8-f0ac-47cc-a869-44373c204848</id>
+    <id>test.no:64db6102-14ce-41e9-b93b-61dbb2cb8b4e</id>
+    <id>test.no:bc82c179-144e-415a-8dd2-64d3569a8d50</id>
+    <id>test.no:da280021-13d8-425e-9783-64911d772397</id>
+    <id>test.no:d42548cc-337f-4005-91f4-a5dc306244b0</id>
+    <id>test.no:e1d0863d-71d3-4b9f-bb17-2af42d4956e7</id>
+    <id>test.no:6827f045-36c1-4678-a0bf-d91b41f8eefb</id>
+    <id>test.no:c7f8731b-5cfe-4cb5-ac57-168a19a2957b</id>
+    <id>test.no:f6cbb81c-1ce1-4080-b242-819e59cee78d</id>
+    <id>test.no:8e5ec6e8-f0ac-47cc-a869-44373c204848</id>
 </parent>
diff --git a/tests/files/reference/parent_translated.xml b/tests/files/reference/parent_translated.xml
@@ -1,7 +1,7 @@
 <?xml version="1.0"?>
 <gmd:MD_Metadata xmlns:gmd="http://www.isotc211.org/2005/gmd" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xmlns:xlink="http://www.w3.org/1999/xlink" xmlns:gml="http://www.opengis.net/gml" xmlns:gmx="http://www.isotc211.org/2005/gmx" xmlns:gco="http://www.isotc211.org/2005/gco" xsi:schemaLocation="http://www.isotc211.org/2005/gmd http://schemas.opengis.net/iso/19139/20060504/gmd/gmd.xsd http://www.isotc211.org/2005/gmx http://schemas.opengis.net/iso/19139/20060504/gmx/gmx.xsd">
   <gmd:fileIdentifier>
-    <gco:CharacterString>64db6102-14ce-41e9-b93b-61dbb2cb8b4e</gco:CharacterString>
+    <gco:CharacterString>test.no:64db6102-14ce-41e9-b93b-61dbb2cb8b4e</gco:CharacterString>
   </gmd:fileIdentifier>
   <gmd:language>
     <gmd:LanguageCode codeList="http://www.loc.gov/standards/iso639-2" codeListValue="eng">English</gmd:LanguageCode>
@@ -96,7 +96,7 @@
           <gmd:identifier>
             <gmd:MD_Identifier>
               <gmd:code>
-                <gco:CharacterString>64db6102-14ce-41e9-b93b-61dbb2cb8b4e</gco:CharacterString>
+                <gco:CharacterString>test.no:64db6102-14ce-41e9-b93b-61dbb2cb8b4e</gco:CharacterString>
               </gmd:code>
             </gmd:MD_Identifier>
           </gmd:identifier>

diff --git a/tests/files/reference/pycsw_dist_translated.xml b/tests/files/reference/pycsw_dist_translated.xml
@@ -1,7 +1,7 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <gmd:MD_Metadata xmlns:gmd="http://www.isotc211.org/2005/gmd" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" xmlns:xlink="http://www.w3.org/1999/xlink" xmlns:gml="http://www.opengis.net/gml" xmlns:gmx="http://www.isotc211.org/2005/gmx" xmlns:gco="http://www.isotc211.org/2005/gco" xsi:schemaLocation="http://www.isotc211.org/2005/gmd http://schemas.opengis.net/iso/19139/20060504/gmd/gmd.xsd http://www.isotc211.org/2005/gmx http://schemas.opengis.net/iso/19139/20060504/gmx/gmx.xsd">
   <gmd:fileIdentifier>
-    <gco:CharacterString>a1ddaf0f-cae0-4a15-9b37-3468e9cb1a2b</gco:CharacterString>
+    <gco:CharacterString>test.no:a1ddaf0f-cae0-4a15-9b37-3468e9cb1a2b</gco:CharacterString>
   </gmd:fileIdentifier>
   <gmd:language>
     <gmd:LanguageCode codeList="http://www.loc.gov/standards/iso639-2" codeListValue="eng">English</gmd:LanguageCode>
@@ -10,7 +10,7 @@
     <gmd:MD_CharacterSetCode codeListValue="utf8" codeList="http://standards.iso.org/ittf/PubliclyAvailableStandards/ISO_19139_Schemas/resources/codelist/ML_gmxCodelists.xml#MD_CharacterSetCode"/>
   </gmd:characterSet>
   <gmd:parentIdentifier>
-    <gco:CharacterString>64db6102-14ce-41e9-b93b-61dbb2cb8b4e</gco:CharacterString>
+    <gco:CharacterString>test.no:64db6102-14ce-41e9-b93b-61dbb2cb8b4e</gco:CharacterString>
   </gmd:parentIdentifier>
   <gmd:hierarchyLevel>
     <gmd:MD_ScopeCode codeList="http://standards.iso.org/ittf/PubliclyAvailableStandards/ISO_19139_Schemas/resources/Codelist/ML_gmxCodelists.xml#MD_ScopeCode" codeListValue="dataset"/>
@@ -91,7 +91,7 @@
           <gmd:identifier>
             <gmd:MD_Identifier>
               <gmd:code>
-                <gco:CharacterString>a1ddaf0f-cae0-4a15-9b37-3468e9cb1a2b</gco:CharacterString>
+                <gco:CharacterString>test.no:a1ddaf0f-cae0-4a15-9b37-3468e9cb1a2b</gco:CharacterString>
               </gmd:code>
             </gmd:MD_Identifier>
           </gmd:identifier>

diff --git a/tests/test_api/test_worker.py b/tests/test_api/test_worker.py
@@ -166,16 +166,14 @@ def testApiWorker_CheckInfoContent(monkeypatch, filesDir):
     failFile = os.path.join(filesDir, "api", "failing.xml")
     failData = bytes(readFile(failFile), "utf-8")
     assert tstWorker._check_information_content(failData) == (
-        False, "Input MMD XML file has no valid UUID metadata_identifier"
+        False, "Input MMD XML file has no valid uri:UUID metadata_identifier"
     )
 
     # Check Error report
     failFile = os.path.join(filesDir, "api", "failing.xml")
     failData = (
         b"<root>"
-        b"  <metadata_identifier> \
-                  test.no:00000000-0000-0000-0000-000000000000 \
-            </metadata_identifier>"
+        b"  <metadata_identifier>met.no:00000000-0000-0000-0000-000000000000</metadata_identifier>"
         b"  <resource>imap://met.no</resource>"
         b"  <geographic_extent>"
         b"    <rectangle>"