Skip to content

Commit f3d31ed

Browse files
committed
Fix to rename indexes when the table has them
1 parent afcec38 commit f3d31ed

22 files changed

+145
-45
lines changed

core/src/integration-test/java/com/scalar/db/storage/cassandra/CassandraAdminCaseSensitivityIntegrationTest.java

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -88,4 +88,8 @@ public void renameTable_ForNonExistingTable_ShouldThrowIllegalArgumentException(
8888
@Override
8989
@Disabled("Cassandra does not support renaming tables")
9090
public void renameTable_IfNewTableNameAlreadyExists_ShouldThrowIllegalArgumentException() {}
91+
92+
@Override
93+
@Disabled("Cassandra does not support renaming tables")
94+
public void renameTable_ForExistingTableWithIndexes_ShouldRenameTableAndIndexesCorrectly() {}
9195
}

core/src/integration-test/java/com/scalar/db/storage/cassandra/CassandraAdminIntegrationTest.java

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -87,4 +87,8 @@ public void renameTable_ForNonExistingTable_ShouldThrowIllegalArgumentException(
8787
@Override
8888
@Disabled("Cassandra does not support renaming tables")
8989
public void renameTable_IfNewTableNameAlreadyExists_ShouldThrowIllegalArgumentException() {}
90+
91+
@Override
92+
@Disabled("Cassandra does not support renaming tables")
93+
public void renameTable_ForExistingTableWithIndexes_ShouldRenameTableAndIndexesCorrectly() {}
9094
}

core/src/integration-test/java/com/scalar/db/storage/cassandra/ConsensusCommitAdminIntegrationTestWithCassandra.java

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -87,4 +87,8 @@ public void renameTable_ForNonExistingTable_ShouldThrowIllegalArgumentException(
8787
@Override
8888
@Disabled("Cassandra does not support renaming tables")
8989
public void renameTable_IfNewTableNameAlreadyExists_ShouldThrowIllegalArgumentException() {}
90+
91+
@Override
92+
@Disabled("Cassandra does not support renaming tables")
93+
public void renameTable_ForExistingTableWithIndexes_ShouldRenameTableAndIndexesCorrectly() {}
9094
}

core/src/integration-test/java/com/scalar/db/storage/cassandra/SingleCrudOperationTransactionAdminIntegrationTestWithCassandra.java

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -82,4 +82,8 @@ public void renameTable_ForNonExistingTable_ShouldThrowIllegalArgumentException(
8282
@Override
8383
@Disabled("Cassandra does not support renaming tables")
8484
public void renameTable_IfNewTableNameAlreadyExists_ShouldThrowIllegalArgumentException() {}
85+
86+
@Override
87+
@Disabled("Cassandra does not support renaming tables")
88+
public void renameTable_ForExistingTableWithIndexes_ShouldRenameTableAndIndexesCorrectly() {}
8589
}

core/src/integration-test/java/com/scalar/db/storage/cosmos/ConsensusCommitAdminIntegrationTestWithCosmos.java

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -79,4 +79,8 @@ public void renameTable_ForNonExistingTable_ShouldThrowIllegalArgumentException(
7979
@Override
8080
@Disabled("Cosmos DB does not support renaming tables")
8181
public void renameTable_IfNewTableNameAlreadyExists_ShouldThrowIllegalArgumentException() {}
82+
83+
@Override
84+
@Disabled("Cosmos DB does not support renaming tables")
85+
public void renameTable_ForExistingTableWithIndexes_ShouldRenameTableAndIndexesCorrectly() {}
8286
}

core/src/integration-test/java/com/scalar/db/storage/cosmos/CosmosAdminCaseSensitivityIntegrationTest.java

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -79,4 +79,8 @@ public void renameTable_ForNonExistingTable_ShouldThrowIllegalArgumentException(
7979
@Override
8080
@Disabled("Cosmos DB does not support renaming tables")
8181
public void renameTable_IfNewTableNameAlreadyExists_ShouldThrowIllegalArgumentException() {}
82+
83+
@Override
84+
@Disabled("Cosmos DB does not support renaming tables")
85+
public void renameTable_ForExistingTableWithIndexes_ShouldRenameTableAndIndexesCorrectly() {}
8286
}

core/src/integration-test/java/com/scalar/db/storage/cosmos/CosmosAdminIntegrationTest.java

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -78,4 +78,8 @@ public void renameTable_ForNonExistingTable_ShouldThrowIllegalArgumentException(
7878
@Override
7979
@Disabled("Cosmos DB does not support renaming tables")
8080
public void renameTable_IfNewTableNameAlreadyExists_ShouldThrowIllegalArgumentException() {}
81+
82+
@Override
83+
@Disabled("Cosmos DB does not support renaming tables")
84+
public void renameTable_ForExistingTableWithIndexes_ShouldRenameTableAndIndexesCorrectly() {}
8185
}

core/src/integration-test/java/com/scalar/db/storage/cosmos/SingleCrudOperationTransactionAdminIntegrationTestWithCosmos.java

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -73,4 +73,8 @@ public void renameTable_ForNonExistingTable_ShouldThrowIllegalArgumentException(
7373
@Override
7474
@Disabled("Cosmos DB does not support renaming tables")
7575
public void renameTable_IfNewTableNameAlreadyExists_ShouldThrowIllegalArgumentException() {}
76+
77+
@Override
78+
@Disabled("Cosmos DB does not support renaming tables")
79+
public void renameTable_ForExistingTableWithIndexes_ShouldRenameTableAndIndexesCorrectly() {}
7680
}

core/src/integration-test/java/com/scalar/db/storage/dynamo/ConsensusCommitAdminIntegrationTestWithDynamo.java

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -84,4 +84,8 @@ public void renameTable_ForNonExistingTable_ShouldThrowIllegalArgumentException(
8484
@Override
8585
@Disabled("DynamoDB does not support renaming tables")
8686
public void renameTable_IfNewTableNameAlreadyExists_ShouldThrowIllegalArgumentException() {}
87+
88+
@Override
89+
@Disabled("DynamoDB does not support renaming tables")
90+
public void renameTable_ForExistingTableWithIndexes_ShouldRenameTableAndIndexesCorrectly() {}
8791
}

core/src/integration-test/java/com/scalar/db/storage/dynamo/DynamoAdminCaseSensitivityIntegrationTest.java

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -84,4 +84,8 @@ public void renameTable_ForNonExistingTable_ShouldThrowIllegalArgumentException(
8484
@Override
8585
@Disabled("DynamoDB does not support renaming tables")
8686
public void renameTable_IfNewTableNameAlreadyExists_ShouldThrowIllegalArgumentException() {}
87+
88+
@Override
89+
@Disabled("DynamoDB does not support renaming tables")
90+
public void renameTable_ForExistingTableWithIndexes_ShouldRenameTableAndIndexesCorrectly() {}
8791
}

0 commit comments

Comments
 (0)