add option merging to init() - add test for full coverage

Author: pachewise

jwt/api_jwt.py

@@ -42,10 +42,9 @@
 class PyJWT:
     def __init__(self, options: Options | None = None) -> None:
         self.options: FullOptions
-        if options is None:
-            self.options = self._get_default_options()
-        else:
-            self.options = {**self._get_default_options(), **options}
+        self.options = self._get_default_options()
+        if options is not None:
+            self.options = self._merge_options(options)
 
     @staticmethod
     def _get_default_options() -> FullOptions:

tests/test_api_jwt.py

@@ -36,6 +36,14 @@ def payload():
 
 
 class TestJWT:
+    def test_jwt_with_options(self):
+        jwt = PyJWT(options={"verify_signature": False})
+        assert jwt.options["verify_signature"] is False
+        # assert that unrelated option is unchanged from default
+        assert jwt.options["strict_aud"] is False
+        # assert that verify_signature is respected unless verify_exp is overridden
+        assert jwt.options["verify_exp"] is False
+
     def test_decodes_valid_jwt(self, jwt):
         example_payload = {"hello": "world"}
         example_secret = "secret"