-
Notifications
You must be signed in to change notification settings - Fork 29
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add regression tests for Cassandra 2.x and 3.x #45
Comments
Something like this: |
@adamdougal can you take a look at this one please? |
Kinda... Though I think it might be overkill to have a copy of all of the unit tests for each version. The change I'd make, is to move most of the unit tests to the As a minor thing, I'd probably rename the Good start though :) |
Hi Adam,
This is only an example how can be done.
I agree on the change of cql module to cqlmigrate, and the root one to cqlmigrate-parent (or similar).
And the tests are only in test2x, because are cassandra2 tests. In test3x there is only a ref to test2x.
To apply this a new fork should be done (or can I create a branch on the main project)
Regards, Jose.
… On 29 Nov 2016, at 08:25, Adam Dougal ***@***.***> wrote:
Kinda... Though I think it might be overkill to have a copy of all of the unit tests for each version. The change I'd make, is to move most of the unit tests to the cql module. Then just have a copy of the CqlMigratorImplTest in each of the testX modules so that it's tested end to end for each version.
As a minor thing, I'd probably rename the cql module to something different as it doesn't contain any cql, possible even just calling it cqlmigrate?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.
|
CqlMigrate should work in both versions, and should be able to be tested within both versions.
Could be done by creating three modules: one for CqlMigrate and two other for tests. The latter should share the tests meanwhile is possible (no break changes)
The text was updated successfully, but these errors were encountered: