add test for ids overriding all other parameters, and for incorrect behavior around the accept and content-type for service-desc endpoint

Author: philvarner

CHANGELOG.md

@@ -5,6 +5,21 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+## [Unreleased] - TBD
+
+### Added
+
+- Validation that using the `ids` parameter does not override all other parameters, as was the behavior
+  prior to v1.0.0-beta.1.
+
+### Changed
+
+### Deprecated
+
+### Removed
+
+### Fixed
+
 ## [0.0.2] - 2022-09-26
 
 ### Added

poetry.lock

@@ -21,6 +21,7 @@ click = ">=8.0.1"
 pystac-client = "^0.5.0"
 requests = "^2.28.1"
 pystac = {extras = ["orjson"], version = "^1.6.1"}
+jsonschema = "^4.16.0"
 
 [tool.poetry.dev-dependencies]
 Pygments = ">=2.10.0"

pyproject.toml

@@ -26,6 +26,11 @@
     default=True,
     help="Test all validations with POST method for requests in addition to GET",
 )
+@click.option(
+    "--collection",
+    required=True,
+    help="The name of the collection to use for some tests.",
+)
 @click.option(
     "--conformance",
     "conformance_classes",
@@ -45,7 +50,11 @@
     help="Conformance class URIs to validate",
 )
 def main(
-    log_level: str, root_url: str, post: bool, conformance_classes: List[str]
+    log_level: str,
+    root_url: str,
+    post: bool,
+    conformance_classes: List[str],
+    collection: str,
 ) -> int:
     """STAC API Validator."""
 
@@ -54,7 +63,9 @@ def main(
     print(f"Validating {root_url}", flush=True)
 
     try:
-        (warnings, errors) = validate_api(root_url, post, conformance_classes)
+        (warnings, errors) = validate_api(
+            root_url, post, conformance_classes, collection
+        )
     except Exception as e:
         print(f"Failed.\nError {root_url}: {type(e)} {str(e)}")
         traceback.print_exc()