imap-processing 0.9.0__py3-none-any.whl → 0.11.0__py3-none-any.whl

imap_processing/tests/codice/test_codice_l0.py

@@ -1,32 +1,43 @@
-"""Tests the decommutation process for CoDICE CCSDS Packets. This also tests the
-unit conversion process for CoDICE housekeeping data."""
+"""Tests the decommutation process for CoDICE CCSDS Packets
 
-from pathlib import Path
+The tests within ensure that the test L0 data can be decommed and result in
+the expected APIDs, number of packets, and contain valid CCSDS header contents.
+"""
 
-import pandas as pd
 import pytest
 import xarray as xr
 
-from imap_processing import imap_module_directory
 from imap_processing.codice import codice_l0
-from imap_processing.codice.codice_l1a import create_hskp_dataset
-from imap_processing.utils import convert_raw_to_eu
-
-# Define the CCSDS header fields (which will be ignored in these tests)
-CCSDS_HEADER_FIELDS = [
-    "shcoarse",
-    "version",
-    "type",
-    "sec_hdr_flg",
-    "pkt_apid",
-    "seq_flgs",
-    "src_seq_ctr",
-    "pkt_len",
-]
+from imap_processing.codice.utils import CODICEAPID
+
+from .conftest import TEST_L0_FILE
+
+pytestmark = pytest.mark.external_test_data
+
+EXPECTED_RESULTS = {
+    CODICEAPID.COD_NHK: 31778,
+    CODICEAPID.COD_LO_IAL: 18917,
+    CODICEAPID.COD_LO_PHA: 616,
+    CODICEAPID.COD_LO_SW_PRIORITY_COUNTS: 77,
+    CODICEAPID.COD_LO_SW_SPECIES_COUNTS: 77,
+    CODICEAPID.COD_LO_NSW_SPECIES_COUNTS: 77,
+    CODICEAPID.COD_LO_SW_ANGULAR_COUNTS: 77,
+    CODICEAPID.COD_LO_NSW_ANGULAR_COUNTS: 77,
+    CODICEAPID.COD_LO_NSW_PRIORITY_COUNTS: 77,
+    CODICEAPID.COD_LO_INST_COUNTS_AGGREGATED: 77,
+    CODICEAPID.COD_LO_INST_COUNTS_SINGLES: 77,
+    CODICEAPID.COD_HI_IAL: 18883,
+    CODICEAPID.COD_HI_PHA: 633,
+    CODICEAPID.COD_HI_INST_COUNTS_AGGREGATED: 77,
+    CODICEAPID.COD_HI_INST_COUNTS_SINGLES: 77,
+    CODICEAPID.COD_HI_OMNI_SPECIES_COUNTS: 77,
+    CODICEAPID.COD_HI_SECT_SPECIES_COUNTS: 77,
+    CODICEAPID.COD_HI_INST_COUNTS_PRIORITIES: 77,
+}
 
 
 @pytest.fixture(scope="session")
-def decom_test_data() -> xr.Dataset:
+def decom_test_data(_download_test_data) -> xr.Dataset:
     """Read test data from file and return a decommutated housekeeping packet.
 
     Returns
@@ -35,116 +46,39 @@ def decom_test_data() -> xr.Dataset:
         A decommutated housekeeping packet
     """
 
-    packet_file = Path(
-        f"{imap_module_directory}/tests/codice/data/"
-        f"imap_codice_l0_raw_20241110_v001.pkts"
-    )
-
-    packet = codice_l0.decom_packets(packet_file)[1136]
+    packet = codice_l0.decom_packets(TEST_L0_FILE)
 
     return packet
 
 
-@pytest.fixture(scope="session")
-def validation_data() -> pd.core.frame.DataFrame:
-    """Read in validation data from the CSV file
+@pytest.mark.parametrize("apid", EXPECTED_RESULTS.keys())
+def test_ccsds_headers(decom_test_data: xr.Dataset, apid):
+    """Tests that the CCSDS headers are present in the decommed data"""
 
-    Returns
-    -------
-    validation_data : pandas.core.frame.DataFrame
-        The validation data read from the CSV, cleaned up and ready to compare
-        the decommutated packet with
-    """
+    for ccsds_header_field in [
+        "shcoarse",
+        "version",
+        "type",
+        "sec_hdr_flg",
+        "pkt_apid",
+        "seq_flgs",
+        "src_seq_ctr",
+        "pkt_len",
+    ]:
+        assert ccsds_header_field in decom_test_data[apid]
 
-    # Read in the CSV file
-    validation_file = Path(
-        f"{imap_module_directory}/tests/codice/data/"
-        f"idle_export_raw.COD_NHK_20230822_122700.csv"
-    )
-    validation_data = pd.read_csv(validation_file, index_col="SHCOARSE")
 
-    if "timestamp" in validation_data.columns:
-        validation_data.drop(columns=["timestamp"], inplace=True, errors="ignore")
+@pytest.mark.parametrize("apid", EXPECTED_RESULTS.keys())
+def test_expected_apids(decom_test_data: xr.Dataset, apid):
+    """Tests that the expected APIDs are present in the decommed data"""
 
-    return validation_data
+    assert apid in decom_test_data
 
 
-@pytest.mark.xfail(
-    reason="Need to update to validate against new validation CDFs. See issue #1154."
-)
-def test_eu_hskp_data(
-    decom_test_data: xr.Dataset,
-    validation_data: pd.core.frame.DataFrame,
+@pytest.mark.parametrize("apid, expected_num_packets", EXPECTED_RESULTS.items())
+def test_expected_total_packets(
+    decom_test_data: xr.Dataset, apid, expected_num_packets
 ):
-    """Compare the engineering unit (EU) housekeeping data to the validation data.
-
-    Parameters
-    ----------
-    decom_test_data : xr.Dataset
-        The decommutated housekeeping packet
-    validation_data : pandas.core.frame.DataFrame
-        The validation data to compare against
-    """
-
-    l1a_hk_ds = create_hskp_dataset(decom_test_data, "001")
-    eu_hk_data = convert_raw_to_eu(
-        l1a_hk_ds,
-        imap_module_directory / "tests/codice/data/eu_unit_lookup_table.csv",
-        "P_COD_NHK",
-    )
-
-    validation_row = validation_data.loc[decom_test_data.shcoarse]
-
-    # Compare EU values of housekeeping data, skipping CCSDS header fields
-    for field in eu_hk_data:
-        # Skip header values
-        if field in CCSDS_HEADER_FIELDS:
-            continue
-
-        eu_values = getattr(eu_hk_data, field).data
-        validation_values = validation_row[field.upper()]
-
-        # Compare each individual element
-        for eu_val, validation_val in zip(eu_values, validation_values):
-            assert round(eu_val, 5) == round(validation_val, 5)
-
-
-@pytest.mark.xfail(
-    reason="Need to update to validate against new validation CDFs. See issue #1154."
-)
-def test_raw_hskp_data(
-    decom_test_data: xr.Dataset,
-    validation_data: pd.core.frame.DataFrame,
-):
-    """Compare the raw housekeeping data to the validation data.
-
-    Parameters
-    ----------
-    decom_test_data : xr.Dataset
-        The decommutated housekeeping packet
-    validation_data : pandas.core.frame.DataFrame
-        The validation data to compare against
-    """
-
-    validation_row = validation_data.loc[decom_test_data.shcoarse]
-
-    # Compare raw values of housekeeping data
-    for field in decom_test_data:
-        if field not in CCSDS_HEADER_FIELDS:
-            raw_values = getattr(decom_test_data, field).data
-            validation_values = validation_row[field.upper()]
-            for raw_value, validation_value in zip(raw_values, validation_values):
-                assert raw_value == validation_value
-
-
-def test_total_packets_in_data_file(decom_test_data: xr.Dataset):
-    """Test if total packets in data file is correct
-
-    Parameters
-    ----------
-    decom_test_data : xr.Dataset
-        The decommutated housekeeping packet
-    """
+    """Test if total packets in the decommed data is correct"""
 
-    total_packets = 31778
-    assert len(decom_test_data.epoch) == total_packets
+    assert len(decom_test_data[apid].epoch) == expected_num_packets

imap_processing/tests/codice/test_codice_l1a.py

@@ -7,6 +7,7 @@ import pytest
 import xarray as xr
 
 from imap_processing.cdf.utils import load_cdf, write_cdf
+from imap_processing.codice import constants
 from imap_processing.codice.codice_l1a import process_codice_l1a
 
 from .conftest import TEST_L0_FILE, VALIDATION_DATA
@@ -14,18 +15,41 @@ from .conftest import TEST_L0_FILE, VALIDATION_DATA
 logger = logging.getLogger(__name__)
 logger.setLevel(logging.INFO)
 
+pytestmark = pytest.mark.external_test_data
+
+DESCRIPTORS = [
+    "hi-ialirt",
+    "lo-ialirt",
+    "hskp",
+    "lo-counters-aggregated",
+    "lo-counters-singles",
+    "lo-sw-priority",
+    "lo-nsw-priority",
+    "lo-sw-species",
+    "lo-nsw-species",
+    "lo-sw-angular",
+    "lo-nsw-angular",
+    "hi-counters-aggregated",
+    "hi-counters-singles",
+    "hi-omni",
+    "hi-sectored",
+    "hi-priority",
+    "lo-pha",
+    "hi-pha",
+]
+
 EXPECTED_ARRAY_SHAPES = [
     (),  # hi-ialirt  # TODO: Need to implement
     (),  # lo-ialirt  # TODO: Need to implement
     (31778,),  # hskp
-    (77, 128, 6, 6),  # lo-counters-aggregated
-    (77, 128, 24, 6),  # lo-counters-singles
-    (77, 128, 1, 12),  # lo-sw-priority
-    (77, 128, 1, 12),  # lo-nsw-priority
-    (77, 128, 1, 1),  # lo-sw-species
-    (77, 128, 1, 1),  # lo-nsw-species
-    (77, 128, 5, 12),  # lo-sw-angular
-    (77, 128, 19, 12),  # lo-nsw-angular
+    (77, 6, 128),  # lo-counters-aggregated
+    (77, 24, 6, 128),  # lo-counters-singles
+    (77, 12, 128),  # lo-sw-priority
+    (77, 12, 128),  # lo-nsw-priority
+    (77, 1, 128),  # lo-sw-species
+    (77, 1, 128),  # lo-nsw-species
+    (77, 5, 12, 128),  # lo-sw-angular
+    (77, 19, 12, 128),  # lo-nsw-angular
     (77, 1, 6, 1),  # hi-counters-aggregated
     (77, 1, 12, 1),  # hi-counters-singles
     (77, 15, 4, 1),  # hi-omni
@@ -35,43 +59,22 @@ EXPECTED_ARRAY_SHAPES = [
     (),  # hi-pha  # TODO: Need to implement
 ]
 
-EXPECTED_LOGICAL_SOURCES = [
-    "imap_codice_l1a_hi-ialirt",
-    "imap_codice_l1a_lo-ialirt",
-    "imap_codice_l1a_hskp",
-    "imap_codice_l1a_lo-counters-aggregated",
-    "imap_codice_l1a_lo-counters-singles",
-    "imap_codice_l1a_lo-sw-priority",
-    "imap_codice_l1a_lo-nsw-priority",
-    "imap_codice_l1a_lo-sw-species",
-    "imap_codice_l1a_lo-nsw-species",
-    "imap_codice_l1a_lo-sw-angular",
-    "imap_codice_l1a_lo-nsw-angular",
-    "imap_codice_l1a_hi-counters-aggregated",
-    "imap_codice_l1a_hi-counters-singles",
-    "imap_codice_l1a_hi-omni",
-    "imap_codice_l1a_hi-sectored",
-    "imap_codice_l1a_hi-priority",
-    "imap_codice_l1a_lo-pha",
-    "imap_codice_l1a_hi-pha",
-]
-
 EXPECTED_NUM_VARIABLES = [
     0,  # hi-ialirt  # TODO: Need to implement
     0,  # lo-ialirt  # TODO: Need to implement
     148,  # hskp
-    3,  # lo-counters-aggregated
-    3,  # lo-counters-singles
-    7,  # lo-sw-priority
-    4,  # lo-nsw-priority
-    18,  # lo-sw-species
-    10,  # lo-nsw-species
-    6,  # lo-sw-angular
-    3,  # lo-nsw-angular
+    8 + len(constants.LO_COUNTERS_AGGREGATED_VARIABLE_NAMES),  # lo-counters-aggregated
+    9,  # lo-counters-singles
+    13,  # lo-sw-priority
+    10,  # lo-nsw-priority
+    24,  # lo-sw-species
+    16,  # lo-nsw-species
+    12,  # lo-sw-angular
+    9,  # lo-nsw-angular
     1,  # hi-counters-aggregated
     3,  # hi-counters-singles
-    8,  # hi-omni
-    4,  # hi-sectored
+    10,  # hi-omni
+    6,  # hi-sectored
     0,  # hi-priority  # TODO: Need to implement
     0,  # lo-pha  # TODO: Need to implement
     0,  # hi-pha  # TODO: Need to implement
@@ -114,13 +117,27 @@ def test_l1a_data_array_shape(test_l1a_data, index):
         pytest.xfail("Data product is currently unsupported")
 
     for variable in processed_dataset:
+        # For variables with energy dimensions
         if variable in ["energy_table", "acquisition_time_per_step"]:
             assert processed_dataset[variable].data.shape == (128,)
+        # For "support" variables with epoch dimensions
+        elif variable in [
+            "rgfo_half_spin",
+            "nso_half_spin",
+            "sw_bias_gain_mode",
+            "st_bias_gain_mode",
+            "data_quality",
+            "spin_period",
+        ]:
+            assert processed_dataset[variable].data.shape == (
+                len(processed_dataset["epoch"].data),
+            )
+        # For counter variables
         else:
             assert processed_dataset[variable].data.shape == expected_shape
 
 
-@pytest.mark.parametrize("index", range(len(EXPECTED_LOGICAL_SOURCES)))
+@pytest.mark.parametrize("index", range(len(DESCRIPTORS)))
 def test_l1a_logical_sources(test_l1a_data, index):
     """Tests that the Logical source of the dataset is what is expected.
 
@@ -136,11 +153,11 @@ def test_l1a_logical_sources(test_l1a_data, index):
     """
 
     processed_dataset = test_l1a_data[index]
-    expected_logical_source = EXPECTED_LOGICAL_SOURCES[index]
+    expected_logical_source = f"imap_codice_l1a_{DESCRIPTORS[index]}"
 
     # Mark currently broken/unsupported datasets as expected to fail
     # TODO: Remove these once they are supported
-    if index in [0, 1, 2, 15, 16, 17]:
+    if index in [0, 1, 15, 16, 17]:
         pytest.xfail("Data product is currently unsupported")
 
     # Write the dataset to a file to set the logical source attribute
@@ -150,9 +167,11 @@ def test_l1a_logical_sources(test_l1a_data, index):
 
 
 @pytest.mark.parametrize("index", range(len(EXPECTED_NUM_VARIABLES)))
-def test_l1a_num_variables(test_l1a_data, index):
-    """Tests that the data arrays in the generated CDFs have the expected number
-    of variables.
+def test_l1a_num_data_variables(test_l1a_data, index):
+    """Tests that the generated CDFs have the expected number of data variables.
+
+    These data variables include counter data (e.g. hplus, heplus, etc.) as well
+    as any "support" variables (e.g. data_quality, spin_period, etc.).
 
     Parameters
     ----------
@@ -172,14 +191,9 @@ def test_l1a_num_variables(test_l1a_data, index):
     assert len(processed_dataset) == EXPECTED_NUM_VARIABLES[index]
 
 
-@pytest.mark.skip("Awaiting validation data")
 @pytest.mark.parametrize("index", range(len(VALIDATION_DATA)))
-def test_l1a_data_array_values(test_l1a_data: xr.Dataset, index):
-    """Tests that the generated L1a CDF contents are valid.
-
-    Once proper validation files are acquired, this test function should point
-    to those. This function currently just serves as a framework for validating
-    files, but does not actually validate them.
+def test_l1a_validate_data_arrays(test_l1a_data: xr.Dataset, index):
+    """Tests that the generated L1a CDF data array contents are valid.
 
     Parameters
     ----------
@@ -189,15 +203,89 @@ def test_l1a_data_array_values(test_l1a_data: xr.Dataset, index):
         The index of the list to test
     """
 
-    generated_dataset = test_l1a_data
-    validation_dataset = load_cdf(VALIDATION_DATA[index])
+    descriptor = DESCRIPTORS[index]
+
+    if descriptor == "hskp":
+        pytest.skip("Housekeeping data is validated in a separate test")
+
+    # TODO: Currently only the following products can be validated, expand this
+    #       to other data products as I can validate them.
+    able_to_be_validated = [
+        "lo-counters-aggregated",
+        "lo-counters-singles",
+        "lo-sw-angular",
+        "lo-nsw-angular",
+        "lo-sw-priority",
+        "lo-nsw-priority",
+        "lo-sw-species",
+        "lo-nsw-species",
+    ]
+    if descriptor in able_to_be_validated:
+        counters = getattr(
+            constants, f'{descriptor.upper().replace("-","_")}_VARIABLE_NAMES'
+        )
+        processed_dataset = test_l1a_data[index]
+        validation_dataset = load_cdf(VALIDATION_DATA[index])
+
+        for counter in counters:
+            # Ensure the data array shapes are equal
+            assert (
+                processed_dataset[counter].data.shape
+                == validation_dataset[counter].data.shape
+            )
 
-    # Ensure the processed data matches the validation data
-    for variable in validation_dataset:
-        assert variable in generated_dataset
-        if variable != "epoch":
+            # TODO: Once Joey and I figure out some small discrepancies with
+            #       some data products, we should get matching data array shapes
+            #       AND values (i.e. run assert_array_equal on the arrays,
+            #       instead of just checking shape)
+
+    else:
+        pytest.xfail(f"Still need to implement validation for {descriptor}")
+
+
+def test_l1a_validate_hskp_data(test_l1a_data):
+    """Tests that the L1a housekeeping data is valid"""
+
+    # Housekeeping data is the 2nd element in the list of test products
+    hskp_data = test_l1a_data[2]
+    validation_hskp_filepath = VALIDATION_DATA[2]
+
+    # Load the validation housekeeping data
+    validation_hskp_data = load_cdf(validation_hskp_filepath)
+
+    # These variables are present in the decommed test data, but not present in
+    # the validation data
+    # TODO: Ask Joey if these can be removed from the L1a housekeeping CDFs
+    exclude_variables = [
+        "spare_1",
+        "spare_2",
+        "spare_3",
+        "spare_4",
+        "spare_5",
+        "spare_6",
+        "spare_62",
+        "spare_68",
+    ]
+
+    # These variables are not present in the validation data
+    # TODO: Ask joey if this is expected
+    exclude_variables.extend(
+        [
+            "version",
+            "type",
+            "sec_hdr_flg",
+            "pkt_apid",
+            "seq_flgs",
+            "src_seq_ctr",
+            "pkt_len",
+            "chksum",
+        ]
+    )
+
+    for variable in hskp_data:
+        if variable not in exclude_variables:
             np.testing.assert_array_equal(
-                validation_dataset[variable].data, generated_dataset[variable].data[0]
+                hskp_data[variable], validation_hskp_data[variable.upper()]
             )
 
 

imap_processing/tests/conftest.py

@@ -14,10 +14,10 @@ import numpy as np
 import pandas as pd
 import pytest
 import requests
-import spiceypy as spice
+import spiceypy
 
 from imap_processing import imap_module_directory
-from imap_processing.spice.time import met_to_j2000ns
+from imap_processing.spice.time import met_to_ttj2000ns
 
 
 @pytest.fixture(autouse=True)
@@ -42,19 +42,20 @@ def _autoclear_spice():
     prevent the kernel pool from interfering with future tests. Option autouse
     ensures this is run after every test."""
     yield
-    spice.kclear()
+    spiceypy.kclear()
 
 
 @pytest.fixture(scope="session")
 def _download_external_kernels(spice_test_data_path):
-    """This fixture downloads the de440s.bsp and pck00011.tpc kernels into the
-    tests/spice/test_data directory if they do not already exist there. The
-    fixture is not intended to be used directly. It is automatically added to
-    tests marked with "external_kernel" in the hook below."""
+    """This fixture downloads externally-located kernels into the tests/spice/test_data
+    directory if they do not already exist there. The fixture is not intended to be
+    used directly. It is automatically added to tests marked with "external_kernel"
+    in the hook below."""
     logger = logging.getLogger(__name__)
     kernel_urls = [
         "https://naif.jpl.nasa.gov/pub/naif/generic_kernels/spk/planets/de440s.bsp",
         "https://naif.jpl.nasa.gov/pub/naif/generic_kernels/pck/pck00011.tpc",
+        "https://naif.jpl.nasa.gov/pub/naif/generic_kernels/pck/earth_1962_240827_2124_combined.bpc",
     ]
 
     for kernel_url in kernel_urls:
@@ -90,6 +91,56 @@ def _download_external_kernels(spice_test_data_path):
                     raise
 
 
+@pytest.fixture(scope="session")
+def _download_test_data(test_data_paths):
+    """This fixture downloads externally-located test data files into a specific
+    location. The list of files and their storage locations are specified in
+    the `test_data_paths` parameter, which is a list of tuples; the zeroth
+    element being the source of the test file in the AWS S3 bucket, and the
+    first element being the location in which to store the downloaded file."""
+
+    logger = logging.getLogger(__name__)
+
+    for test_data_path in test_data_paths:
+        source = test_data_path[0]
+        destination = test_data_path[1]
+
+        # Download the test data if necessary and write it to the appropriate
+        # directory
+        if not destination.exists():
+            response = requests.get(source, timeout=60)
+            if response.status_code == 200:
+                with open(destination, "wb") as file:
+                    file.write(response.content)
+                logger.info(f"Downloaded file: {source}")
+            else:
+                logger.error(f"Failed to download file: {response.status_code}")
+        else:
+            logger.info(f"File already exists: {destination}")
+
+
+@pytest.fixture(scope="session")
+def test_data_paths():
+    """Defines a list of test data files to download from the AWS S3 bucket
+    and the corresponding location in which to store the downloaded file"""
+    test_data_path_list = [
+        (
+            "https://api.dev.imap-mission.com/download/test_data/imap_codice_l0_raw_20241110_v001.pkts",
+            imap_module_directory
+            / "tests"
+            / "codice"
+            / "data"
+            / "imap_codice_l0_raw_20241110_v001.pkts",
+        ),
+        (
+            "https://api.dev.imap-mission.com/download/test_data/imap_hi_l1a_45sensor-de_20250415_v999.cdf",
+            imap_module_directory
+            / "tests/hi/data/l1/imap_hi_l1a_45sensor-de_20250415_v999.cdf",
+        ),
+    ]
+    return test_data_path_list
+
+
 def pytest_collection_modifyitems(items):
     """
     The use of this hook allows modification of test `Items` after tests have
@@ -100,6 +151,7 @@ def pytest_collection_modifyitems(items):
     | pytest mark         | fixture added              |
     +=====================+============================+
     | external_kernel     | _download_external_kernels |
+    | external_test_data  | _download_test_data        |
     | use_test_metakernel | use_test_metakernel        |
     +---------------------+----------------------------+
 
@@ -109,11 +161,16 @@ def pytest_collection_modifyitems(items):
     pytest hook:
     https://docs.pytest.org/en/stable/reference/reference.html#pytest.hookspec.pytest_collection_modifyitems
     """
+    markers_to_fixtures = {
+        "external_kernel": "_download_external_kernels",
+        "external_test_data": "_download_test_data",
+        "use_test_metakernel": "use_test_metakernel",
+    }
+
     for item in items:
-        if item.get_closest_marker("external_kernel") is not None:
-            item.fixturenames.append("_download_external_kernels")
-        if item.get_closest_marker("use_test_metakernel") is not None:
-            item.fixturenames.append("use_test_metakernel")
+        for marker, fixture in markers_to_fixtures.items():
+            if item.get_closest_marker(marker) is not None:
+                item.fixturenames.append(fixture)
 
 
 @pytest.fixture(scope="session")
@@ -124,22 +181,22 @@ def spice_test_data_path(imap_tests_path):
 @pytest.fixture()
 def furnish_time_kernels(spice_test_data_path):
     """Furnishes (temporarily) the testing LSK and SCLK"""
-    spice.kclear()
+    spiceypy.kclear()
     test_lsk = spice_test_data_path / "naif0012.tls"
     test_sclk = spice_test_data_path / "imap_sclk_0000.tsc"
-    spice.furnsh(str(test_lsk))
-    spice.furnsh(str(test_sclk))
+    spiceypy.furnsh(str(test_lsk))
+    spiceypy.furnsh(str(test_sclk))
     yield test_lsk, test_sclk
-    spice.kclear()
+    spiceypy.kclear()
 
 
 @pytest.fixture()
 def furnish_sclk(spice_test_data_path):
     """Furnishes (temporarily) the SCLK for JPSS stored in the package data directory"""
     test_sclk = spice_test_data_path / "imap_sclk_0000.tsc"
-    spice.furnsh(str(test_sclk))
+    spiceypy.furnsh(str(test_sclk))
     yield test_sclk
-    spice.kclear()
+    spiceypy.kclear()
 
 
 @pytest.fixture()
@@ -148,7 +205,9 @@ def furnish_kernels(spice_test_data_path):
 
     @contextmanager
     def furnish_kernels(kernels: list[Path]):
-        with spice.KernelPool([str(spice_test_data_path / k) for k in kernels]) as pool:
+        with spiceypy.KernelPool(
+            [str(spice_test_data_path / k) for k in kernels]
+        ) as pool:
             yield pool
 
     return furnish_kernels
@@ -227,7 +286,7 @@ def session_test_metakernel(monkeypatch_session, tmpdir_factory, spice_test_data
     -----
     - This fixture needs to `scope=session` so that the SPICE_METAKERNEL
     environment variable is available for other fixtures that require time
-    conversions using spice.
+    conversions using spiceypy.
     - No furnishing of kernels occur as part of this fixture. This allows other
     fixtures with lesser scope or individual tests to override the environment
     variable as needed. Use the `metakernel_path_not_set` fixture in tests that
@@ -239,7 +298,7 @@ def session_test_metakernel(monkeypatch_session, tmpdir_factory, spice_test_data
     make_metakernel_from_kernels(metakernel_path, kernels_to_load)
     monkeypatch_session.setenv("SPICE_METAKERNEL", str(metakernel_path))
     yield str(metakernel_path)
-    spice.kclear()
+    spiceypy.kclear()
 
 
 @pytest.fixture()
@@ -289,7 +348,7 @@ def use_test_metakernel(
         make_metakernel_from_kernels(metakernel_path, kernels_to_load)
         monkeypatch.setenv("SPICE_METAKERNEL", str(metakernel_path))
         yield str(metakernel_path)
-    spice.kclear()
+    spiceypy.kclear()
 
 
 @pytest.fixture()
@@ -404,7 +463,7 @@ def generate_spin_data():
         )
 
         # Convert spin_start_sec to datetime to set repointing times flags
-        spin_start_dates = met_to_j2000ns(spin_start_sec + spin_start_subsec / 1000)
+        spin_start_dates = met_to_ttj2000ns(spin_start_sec + spin_start_subsec / 1000)
         spin_start_dates = cdflib.cdfepoch.to_datetime(spin_start_dates)
 
         # Convert DatetimeIndex to Series for using .dt accessor