search

scylladb / kafka-connect-scylladb

Kafka Connect Scylladb Sink
Apache License 2.0
42 stars 22 forks source link

Add configuration for integration testing #65

Closed Bouncheck closed 2 years ago

Bouncheck commented 2 years ago

Adds docker-maven-plugin with configuration that uses preexisting compose file. Bumps verisions of maven-surefire-plugin and maven-failsafe-plugin. Removes 1 dependency due to its issues with integration-test goal. Minor fixes due to small bugs like Integer to String conversion errors.

avelanarius commented 2 years ago

When running those tests, I have the following errors:

[ERROR] Failures: 
[ERROR]   ScyllaDbSinkConnectorIT.stop:108->assertRow:1030 Field does not match. Query = 'SELECT * FROM decimalTesting WHERE id=1234;' ==> expected: <123456789123> but was: <123.456789123>
[ERROR]   ScyllaDbSinkConnectorIT.tableMissingManageTableDisabled:716 Unexpected exception type thrown ==> expected: <org.apache.kafka.connect.errors.DataException> but was: <org.apache.kafka.connect.errors.ConnectException>
[ERROR] Errors: 
[ERROR]   ScyllaDbSinkConnectorIT.deleteMissingTable:557 » Connect Exception occurred wh...
[ERROR]   ScyllaDbSinkConnectorIT.deleteNullValueRow:579 » InvalidQuery Unknown identifi...
[ERROR]   ScyllaDbSinkConnectorIT.insert:166 » InvalidQuery Unknown identifier firstname
[ERROR]   ScyllaDbSinkConnectorIT.stop:108->assertRow:1013 » InvalidQuery Undefined name...
[ERROR]   ScyllaDbSinkConnectorIT.offsetsExistInKafka:1001 » InvalidQuery Unknown identi...
[ERROR]   ScyllaDbSinkConnectorIT.offsetsExistInScylla:821 » InvalidQuery Unknown identi...
[ERROR]   ScyllaDbSinkConnectorIT.tableExistsAlter:668 » InvalidQuery Unknown identifier...
[ERROR]   ScyllaDbSinkConnectorIT.tableExistsAlterManageTableDisabled:752 » InvalidQuery
[ERROR]   ScyllaDbSinkConnectorIT.tableExistsNoChange:624 » InvalidQuery Unknown identif...
[ERROR]   ScyllaDbSinkConnectorIT.testRecordWithTtl:862 » ClassCast java.lang.String can...
[ERROR]   ScyllaDbSinkConnectorIT.testRecordWithoutTtl:950 » InvalidQuery Unknown identi...
[INFO] 
[ERROR] Tests run: 18, Failures: 2, Errors: 11, Skipped: 0
avelanarius commented 2 years ago

Closing, as #66 was merged which includes this PR.