clean up the read module after adding cloud-read capabilities (#502)

- update tests to match new parse_source function
Co-authored-by: Rachel Wegener <[email protected]>

doc/source/user_guide/documentation/components.rst

@@ -27,14 +27,6 @@ granules
    :undoc-members:
    :show-inheritance:
 
-is2cat
-------
-
-.. automodule:: icepyx.core.is2cat
-   :members:
-   :undoc-members:
-   :show-inheritance:
-
 is2ref
 ------
 

doc/source/user_guide/documentation/query.rst

@@ -23,7 +23,6 @@ Attributes
    Query.cycles
    Query.dates
    Query.end_time
-   Query.file_vars
    Query.granules
    Query.order_vars
    Query.product

icepyx/core/is2ref.py

@@ -378,8 +378,10 @@ def extract_product(filepath, auth=None):
             # ATL14 saves the short_name as an array ['ATL14']
             product = product[0]
         product = _validate_product(product)
-    except KeyError:
-        raise "Unable to parse the product name from file metadata"
+    except KeyError as e:
+        raise Exception(
+            "Unable to parse the product name from file metadata"
+        ).with_traceback(e.__traceback__)
 
     # Close the file reader
     f.close()
@@ -421,8 +423,10 @@ def extract_version(filepath, auth=None):
         if isinstance(version, bytes):
             version = version.decode()
 
-    except KeyError:
-        raise "Unable to parse the version from file metadata"
+    except KeyError as e:
+        raise Exception(
+            "Unable to parse the version from file metadata"
+        ).with_traceback(e.__traceback__)
 
     # Close the file reader
     f.close()