dannycranmer commented on PR #203: URL: https://github.com/apache/flink-connector-aws/pull/203#issuecomment-2955057842
@darenwkt can you clarify your understanding of the expectation of each module? As far as I understand the e2e tests are meant to hit real services (and require creds), while the integration tests that hit mocked services (TestContainers) reside in the connector repos (example https://github.com/apache/flink-connector-aws/blob/main/flink-connector-aws/flink-connector-kinesis/src/test/java/org/apache/flink/streaming/connectors/kinesis/FlinkKinesisITCase.java). It seems like the DDB tests are actually IT tests and based on my understanding should stay where they are. Also I see there are other IT tests in the e2e module. @hlteoh37 what are your expectations on this? -- 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]
