mjsax commented on a change in pull request #8759:
URL: https://github.com/apache/kafka/pull/8759#discussion_r433055994


##########
File path: 
streams/test-utils/src/test/java/org/apache/kafka/streams/TopologyTestDriverTest.java
##########
@@ -711,6 +715,56 @@ public void shouldUseSourceSpecificDeserializers() {
         assertThat(result2.getValue(), equalTo(source2Value));
     }
 
+    @Test
+    public void shouldPassRecordHeadersIntoDeSerializers() {

Review comment:
       We don't verify that header are deserialized -- we just verify that the 
right function is called, that hands the header into the deserializer (ie, to 
allow used to read them as metadata).
   
   I guess we should use mocks for the deserializers, but not for the 
serializers. Thus, not using mocks make the test use the same patter for both 
what seems "nicer".
   
   > Moreover, we could build a customized boolean deserializer which outputs 
true when the header is provided, or false otherwise.
   
   Would this make the test simpler? I personally doubt it.




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


Reply via email to