Address review comments

Author: caspervonb

nats/src/nats/js/api.py

@@ -339,7 +339,8 @@ def from_response(cls, resp: Dict[str, Any]):
 
     def as_dict(self) -> Dict[str, object]:
         result = super().as_dict()
-        result["inactive_threshold"] = self._to_nanoseconds(self.inactive_threshold)
+        if self.inactive_threshold is not None:
+            result["inactive_threshold"] = self._to_nanoseconds(self.inactive_threshold)
         return result
 
 

nats/tests/test_js.py

@@ -5476,8 +5476,8 @@ async def test_stream_consumer_limits(self):
         )
 
         cinfo = await js.consumer_info("CONSUMERLIMITS", "consumer1")
-        # The consumer should have inherited or be constrained by stream limits
-        assert cinfo.config.max_ack_pending is not None
+        # The consumer should have inherited the stream's consumer limits
+        assert cinfo.config.max_ack_pending == 1000
 
         # Test with only max_ack_pending set
         await js.add_stream(