tenthe commented on PR #3504:
URL: https://github.com/apache/streampipes/pull/3504#issuecomment-2682598846

   Hi @Suvrat1629,  
   
   the checkstyle looks good—thanks for submitting all the PRs! 
   
   Just a small note for future unit tests on processing elements:  
   You can leverage our existing test framework to simplify testing processing 
element logic (e.g., 
`org.apache.streampipes.processors.enricher.jvm.processor.expression.MathExpressionProcessorTest`).
 We also provide 
`org.apache.streampipes.test.executors.ProcessingElementTestExecutor` to 
automate recurring tasks.  
   
   Feel free to extend the testing framework if you have ideas for 
improvements.  
   
   If you have any questions, feel free to ask here or start a GitHub 
discussion.  
   
   Cheers,  
   Philipp 


-- 
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.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to