Merge pull request #6 from weaverba137/directory-search-fixes

Directory search fixes

LICENSE.rst

@@ -9,7 +9,7 @@ are permitted provided that the following conditions are met:
 * Redistributions in binary form must reproduce the above copyright notice, this
   list of conditions and the following disclaimer in the documentation and/or
   other materials provided with the distribution.
-* Neither the name of the {organization} nor the names of its
+* Neither the name of the HPSSPy Project nor the names of its
   contributors may be used to endorse or promote products derived from
   this software without specific prior written permission.
 

doc/changes.rst

@@ -7,6 +7,13 @@ Release Notes
 
 *Python 2 support will be dropped starting with this release.*
 
+0.4.2 (unreleased)
+------------------
+
+* Further fixes for mapping HTAR file names back to directories (PR `#6`_).
+
+.. _`#6`: https://github.com/weaverba137/hpsspy/pull/6
+
 0.4.1 (2019-01-16)
 ------------------
 

hpsspy/__init__.py

@@ -23,4 +23,4 @@ class HpssOSError(HpssError):
     pass
 
 
-__version__ = '0.5.0.dev242'
+__version__ = '0.4.2.dev242'

hpsspy/scan.py

@@ -387,12 +387,19 @@ def extract_directory_name(filename):
     :class:`str`
         Name of a directory.
     """
-    prefix = os.path.dirname(filename).replace('/', '_') + '_'
+    d = os.path.dirname(filename)
     basefile = os.path.basename(filename).rsplit('.', 1)[0]  # remove .tar
+    if not d:
+        return basefile
+    prefix = d.replace('/', '_') + '_'
     try:
         i = basefile.index(prefix)
     except ValueError:
-        return basefile
+        try:
+            prefix = '_'.join(prefix.split('_')[1:])
+            i = basefile.index(prefix)
+        except ValueError:
+            return basefile
     return basefile[(i + len(prefix)):]
 
 

hpsspy/test/test_scan.py

@@ -238,8 +238,13 @@ def test_extract_directory_name(self):
                                     'protodesi_images_fpc_analysis_' +
                                     'stability_dither-33022.tar'))
         self.assertEqual(d, 'stability_dither-33022')
+        d = extract_directory_name(('buzzard/buzzard_v1.6_desicut/8/' +
+                                    'buzzard_v1.6_desicut_8_7.tar'))
+        self.assertEqual(d, '7')
         d = extract_directory_name('foo/bar/batch.tar')
         self.assertEqual(d, 'batch')
+        d = extract_directory_name('batch.tar')
+        self.assertEqual(d, 'batch')
 
 
 def test_suite():