change from isoformat extended to basic, and remove url parsing in test

podaac/merger/harmony/service.py

@@ -88,7 +88,7 @@ def process_catalog(self, catalog: Catalog):
         first_granule_url = []
         get_granule_url(items[0], first_granule_url)
         first_url_name = Path(first_granule_url[0]).stem
-        filename = f'{first_url_name}_{datetimes[1].isoformat()}_{collection}_merged.nc4'
+        filename = f'{first_url_name}_{datetimes[1].strftime("%Y%m%dT%H%M%SZ")}_{collection}_merged.nc4'
 
         with TemporaryDirectory() as temp_dir:
             self.logger.info('Starting granule downloads')

tests/test_harmony.py

@@ -81,8 +81,8 @@ def test_service_invoke(self):
             collection_name = in_message['sources'][0]['collection']
 
             # Sanity checks on metadata
-            self.assertTrue(unquote(data['href'], encoding='utf-8', errors='replace').endswith(f"{properties['end_datetime']}_{collection_name}_merged.nc4"))
-            self.assertTrue(unquote(data['title'], encoding='utf-8', errors='replace').endswith(f"{properties['end_datetime']}_{collection_name}_merged.nc4"))
+            self.assertTrue(data['href'].endswith(f"{properties['end_datetime']}_{collection_name}_merged.nc4"))
+            self.assertTrue(data['title'].endswith(f"{properties['end_datetime']}_{collection_name}_merged.nc4"))
             self.assertEqual(data['type'], 'application/x-netcdf4')
             self.assertEqual(data['roles'], ['data'])
 
@@ -96,7 +96,7 @@ def test_service_invoke(self):
             ]
 
             path = urlsplit(data['href']).path
-            dataset = Dataset(unquote(path, encoding='utf-8', errors='replace'))
+            dataset = Dataset(path)
             subset_files = dataset['subset_files'][:].tolist()
             subset_files.sort()