Skip to content

Commit beddf17

Browse files
committed
Fixing test: long are now returned as String, tenant must not have underscores
1 parent 68c61fa commit beddf17

File tree

3 files changed

+11
-8
lines changed

3 files changed

+11
-8
lines changed

astra-sdk/src/test/java/com/datastax/astra/sdk/devops/ApiDevopsStreamingAstraTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -90,7 +90,7 @@ public void should_create_tenant() throws InterruptedException {
9090
StreamingClient sc = new StreamingClient(client.getToken().get());
9191
System.out.println("- Create a tenant");
9292
// Giving
93-
tmpTenant = "sdk_java_junit_" + UUID.randomUUID().toString().substring(0,7);
93+
tmpTenant = "sdk-java-junit-" + UUID.randomUUID().toString().substring(0,7);
9494
// When
9595
Assert.assertFalse(sc.tenant(tmpTenant).exist());
9696
LOGGER.info("Tenant " + tmpTenant + " does not exist");

stargate-sdk/src/main/java/com/datastax/stargate/sdk/doc/test/ApiDocumentCollectionsTest.java

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -123,19 +123,21 @@ public void d_should_delete_collection() throws InterruptedException {
123123
@Order(5)
124124
@DisplayName("05-Assign a Json Schema")
125125
public void e_should_set_schema() {
126+
LOGGER.info("should_set_schema");
126127
// Given
127128
String randomCollection = UUID.randomUUID().toString().replaceAll("-", "");
128129
CollectionClient cc = nsClient.collection(randomCollection);
129130
cc.create();
130131
Assertions.assertTrue(cc.exist());
131132
// Then I can add a person with negative age
132-
cc.document("doc1").upsert(new Person("first", "last", -20, null));
133+
cc.document("doc1").upsert(new Person("first", "last", 20, null));
133134
// When Assign Schema
134135
Assertions.assertFalse(cc.getSchema().isPresent());
135136
cc.setSchema(TEST_JSON_SCHEMA);
136137
// Then a schema is present
137138
Assertions.assertTrue(cc.getSchema().isPresent());
138139
// And validation should be enabled
140+
LOGGER.info("negative_should_rise_error");
139141
Assertions.assertThrows(IllegalArgumentException.class, () -> {
140142
cc.document("doc1").upsert(new Person("first", "last", -20, null));
141143
});

stargate-sdk/src/main/java/com/datastax/stargate/sdk/rest/test/ApiDataRecordTest.java

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -86,7 +86,6 @@ public void b_should_delete_row()
8686
@Test
8787
@Order(3)
8888
@DisplayName("03-should-update-a-row")
89-
@SuppressWarnings("unchecked")
9089
public void c_should_update_row()
9190
throws InterruptedException {
9291
// Given
@@ -108,15 +107,16 @@ public void c_should_update_row()
108107
.key("Sci-Fi",1990, "line_update")
109108
.findPage(QueryWithKey.builder().build());
110109
Assertions.assertTrue(rrp.getResults().size() == 1);
111-
Map<String, Object > map = (Map<String, Object>) rrp.getResults().get(0).get("upload");
112-
Assertions.assertEquals(130000000, map.get("nano"));
110+
// When you read a long you get back a String
111+
Map<String, Object > map = (Map<String, Object>) rrp.getResults().get(0);
112+
Assertions.assertEquals("line_update", map.get("title"));
113+
Assertions.assertEquals(1990, map.get("year"));
113114
}
114115

115116

116117
@Test
117118
@Order(4)
118119
@DisplayName("04-should-replace-a-row")
119-
@SuppressWarnings("unchecked")
120120
public void d_should_replace_row()
121121
throws InterruptedException {
122122
// Given
@@ -136,8 +136,9 @@ public void d_should_replace_row()
136136
.key("Sci-Fi",1990, "line_update")
137137
.findPage(QueryWithKey.builder().build());
138138
Assertions.assertTrue(rrp.getResults().size() == 1);
139-
Map<String, Object > map = (Map<String, Object>) rrp.getResults().get(0).get("upload");
140-
Assertions.assertEquals(130000000, map.get("nano"));
139+
Map<String, Object > map = (Map<String, Object>) rrp.getResults().get(0);
140+
Assertions.assertEquals("line_update", map.get("title"));
141+
Assertions.assertEquals(1990, map.get("year"));
141142
}
142143

143144
@Test

0 commit comments

Comments
 (0)