Revert "Add global release table subparser (#326)" (#329)

This reverts commit 0ebf5bd.

Author: lacoak21

imap_data_access/__init__.py

@@ -19,15 +19,7 @@
     ScienceFilePath,
     SPICEFilePath,
 )
-from imap_data_access.io import (
-    download,
-    query,
-    query_release_versions,
-    release,
-    reprocess,
-    spice_query,
-    upload,
-)
+from imap_data_access.io import download, query, release, reprocess, spice_query, upload
 from imap_data_access.processing_input import (
     AncillaryInput,
     ProcessingInputCollection,
@@ -56,7 +48,6 @@
     "SpinInput",
     "download",
     "query",
-    "query_release_versions",
     "release",
     "reprocess",
     "spice_query",

imap_data_access/cli.py

@@ -29,7 +29,7 @@
     ScienceFilePath,
     generate_imap_file_path,
 )
-from imap_data_access.io import query, query_release_versions, release, spice_query
+from imap_data_access.io import query, release, spice_query
 from imap_data_access.utils import ReleaseType
 from imap_data_access.webpoda import download_daily_data
 
@@ -348,12 +348,7 @@ def _release_parser(args: argparse.Namespace):
     args : argparse.Namespace
         An object containing the parsed arguments and their values
     """
-    if not getattr(args, "release_type", None):
-        raise ValueError(
-            "--release-type is required when submitting a release. "
-            "Run 'release -h' for more information."
-        )
-    # All other validation is handled in io.py
+    # All validation is now handled in io.py
     release(
         instrument=args.instrument,
         release_type=args.release_type,
@@ -366,11 +361,6 @@ def _release_parser(args: argparse.Namespace):
     print("Successfully submitted release request to the IMAP SDC.")
 
 
-def _release_versions_parser(args: argparse.Namespace):
-    """Query the latest global release number from the IMAP SDC."""
-    print(query_release_versions())
-
-
 def add_query_args(subparser: ArgumentParser) -> None:
     """Add query arguments to subparser.
 
@@ -706,31 +696,7 @@ def main():
         "release",
         help=release_help,
         formatter_class=argparse.RawTextHelpFormatter,
-        description=(
-            "Submit a release to the IMAP SDC, or use the 'query' subcommand\n"
-            "to look up the latest global release number.\n\n"
-            "Examples:\n"
-            "  imap-data-access release --release-type release ...\n"
-            "  imap-data-access release query\n"
-        ),
     )
-    # --- nested subparsers (e.g. `release query`) ---
-    release_subparsers = parser_release.add_subparsers(dest="release_subcommand")
-
-    release_query_parser = release_subparsers.add_parser(
-        "query",
-        help="Query the latest global release number.",
-        description=(
-            "Query the IMAP SDC latest global release.\n"
-            "Returns one record with release_number and updated_date.\n\n"
-            "Example:\n"
-            "  imap-data-access release query\n"
-        ),
-        formatter_class=argparse.RawTextHelpFormatter,
-    )
-    release_query_parser.set_defaults(func=_release_versions_parser)
-
-    # --- submit/unrelease arguments
     parser_release.add_argument(
         "--instrument",
         type=str,
@@ -756,7 +722,7 @@ def main():
     parser_release.add_argument(
         "--release-type",
         type=str,
-        required=False,  # enforced in _release_parser when no subcommand is given
+        required=True,
         metavar="ReleaseType",
         help=(
             "Type of release:\n"

imap_data_access/io.py

@@ -536,30 +536,6 @@ def upload(file_path: Union[Path, str]) -> None:
     logger.info("File %s uploaded successfully", file_path)
 
 
-def query_release_versions() -> dict:
-    """Query the latest global release in the IMAP SDC.
-
-    Returns
-    -------
-    dict
-        Latest global release record containing ``release_number`` and
-        ``updated_date``.
-    """
-    url = f"{_get_base_url()}/global-release/latest"
-    request = requests.Request(method="GET", url=url).prepare()
-
-    logger.info("Querying release versions at url %s", request.url)
-    with _make_request(request) as response:
-        payload = response.json()
-        logger.debug("Received JSON: %s", payload)
-
-    if not isinstance(payload, dict):
-        raise IMAPDataAccessError(
-            "Unexpected response format for global release latest query."
-        )
-    return payload
-
-
 def release(
     *,
     instrument: Optional[str] = None,

tests/test_cli.py

@@ -49,27 +49,3 @@ def test_cli_error_message(capsys):
             cli.main()
     captured = capsys.readouterr()
     assert "FileNotFoundError" in captured.err
-
-
-# ---------------------------------------------------------------------------
-# release query CLI tests
-# ---------------------------------------------------------------------------
-RELEASE_VERSION_RECORD = {
-    "release_number": 1,
-    "updated_date": "2026-04-01T00:00:00",
-}
-
-
-def test_cli_release_query(capsys):
-    """Test that `release query` returns the latest global release dict."""
-    with mock.patch.object(sys, "argv", ["imap-data-access", "release", "query"]):
-        with mock.patch(
-            "imap_data_access.cli.query_release_versions",
-            return_value=RELEASE_VERSION_RECORD,
-        ) as mock_qrv:
-            cli.main()
-
-    captured = capsys.readouterr()
-    assert "release_number" in captured.out
-    assert "updated_date" in captured.out
-    mock_qrv.assert_called_once_with()

tests/test_io.py

@@ -959,53 +959,3 @@ def test_release_early_release_with_manifest(mock_send_request, api_key, tmp_pat
     last_request = mock_send_request.call_args[0][0]
     assert "/release" in last_request.url
     assert "manifest_file=" in last_request.url
-
-
-# ---------------------------------------------------------------------------
-# query_release_versions tests
-# ---------------------------------------------------------------------------
-
-RELEASE_VERSION_RECORD = {
-    "release_number": 1,
-    "updated_date": "2026-04-01T00:00:00",
-}
-
-
-def test_query_release_versions(mock_send_request):
-    """Test a basic call to query_release_versions hits /global-release/latest.
-
-    Parameters
-    ----------
-    mock_send_request : unittest.mock.MagicMock
-        Mock object for requests.Session
-    """
-    mock_response = MagicMock()
-    mock_response.json.return_value = RELEASE_VERSION_RECORD
-    mock_send_request.return_value = mock_response
-
-    results = imap_data_access.query_release_versions()
-    assert results == RELEASE_VERSION_RECORD
-
-    mock_send_request.assert_called_once()
-    sent_request = mock_send_request.call_args[0][0]
-    assert sent_request.url == "https://api.test.com/global-release/latest"
-    assert sent_request.method == "GET"
-
-
-def test_query_release_versions_bad_payload(mock_send_request):
-    """Test that query_release_versions rejects non-dict responses.
-
-    Parameters
-    ----------
-    mock_send_request : unittest.mock.MagicMock
-        Mock object for requests.Session
-    """
-    mock_response = MagicMock()
-    mock_response.json.return_value = []
-    mock_send_request.return_value = mock_response
-
-    with pytest.raises(
-        imap_data_access.io.IMAPDataAccessError, match="Unexpected response"
-    ):
-        imap_data_access.query_release_versions()
-    mock_send_request.assert_called_once()