cleanup

Signed-off-by: nstarman <[email protected]>

src/cosmology/api/_core.py

@@ -23,19 +23,19 @@ class Cosmology(Protocol[ArrayT_co, InputT_contra]):  # type: ignore[misc]
     """Cosmology API Protocol."""
 
     def __cosmology_namespace__(
-        self, /, *, api_version: str | None = None
+        self, *, api_version: str | None = None
     ) -> CosmologyNamespace:
         """Returns an object that has all the cosmology API functions on it.
 
         Parameters
         ----------
-        api_version: Optional[str]
+        api_version: str | None, keyword-only
             string representing the version of the cosmology API specification
             to be returned, in ``'YYYY.MM'`` form, for example, ``'2020.10'``.
             If it is ``None``, it should return the namespace corresponding to
             latest version of the cosmology API specification.  If the given
             version is invalid or not implemented for the given module, an error
-            should be raised. Default: ``None``.
+            should be raised. Default: `None`.
 
         Returns
         -------
@@ -57,19 +57,19 @@ def name(self) -> str | None:
     # Convenience methods
 
     def constants(
-        self, /, *, api_version: str | None = None
+        self, *, api_version: str | None = None
     ) -> CosmologyConstantsNamespace:
         """Returns an object that has all the cosmology API functions on it.
 
         Parameters
         ----------
-        api_version: Optional[str]
+        api_version: str | None, keyword-only
             string representing the version of the cosmology API specification
             to be returned, in ``'YYYY.MM'`` form, for example, ``'2020.10'``.
             If it is ``None``, it should return the namespace corresponding to
             latest version of the cosmology API specification.  If the given
             version is invalid or not implemented for the given module, an error
-            should be raised. Default: ``None``.
+            should be raised. Default: `None`.
 
         Returns
         -------

tests/api/compat/test_standard.py

@@ -9,6 +9,7 @@
 import pytest
 from cosmology.api import (
     Cosmology,
+    CosmologyConstantsNamespace,
     CosmologyNamespace,
     CosmologyWrapper,
     StandardCosmology,
@@ -40,7 +41,7 @@ class ExampleStandardCosmologyWrapper:
     # TODO: more examples?
 
 
-def test_compliant_bkg_wrapper(cosmology_ns, standard_attrs, standard_meths):
+def test_compliant_standard_wrapper(cosmology_ns, standard_attrs, standard_meths):
     """
     Test that a compliant instance is a
     `cosmology.api.CosmologyWrapper`. In particular, this tests
@@ -55,7 +56,6 @@ def _return_1arg(self, z: Array, /) -> Array:
 
     def _cosmology_namespace_(
         self,
-        /,
         *,
         api_version: str | None = None,
     ) -> CosmologyNamespace:
@@ -67,6 +67,11 @@ def name(self) -> str | None:
     def _getattr_(self, name: str) -> object:
         return getattr(self.cosmo, name)
 
+    def constants(
+        self, *, api_version: str | None = None
+    ) -> CosmologyConstantsNamespace:
+        return cosmology_ns.constants
+
     ExampleStandardCosmologyWrapper = make_dataclass(
         "ExampleStandardCosmologyWrapper",
         [("cosmo", object)],

tests/api/test_core.py

@@ -40,13 +40,12 @@ class Cosmology:
 
         def __cosmology_namespace__(
             self,
-            /,
             *,
             api_version: str | None = None,
         ) -> CosmologyNamespace:
             return cosmology_ns
 
-        def constants(self, /, *, api_version: str | None = None):
+        def constants(self, *, api_version: str | None = None):
             return self.__cosmology_namespace__(api_version=api_version).constants
 
     cosmo = Cosmology()