Skip to content

Commit 0254cbc

Browse files
author
Maximilian Hippler
committed
Resolve PR conversation
1 parent e6f22dc commit 0254cbc

File tree

1 file changed

+23
-5
lines changed

1 file changed

+23
-5
lines changed

src/test/java/org/influxdb/InfluxDBTest.java

Lines changed: 23 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -24,8 +24,19 @@
2424
import java.time.Instant;
2525
import java.time.ZoneId;
2626
import java.time.format.DateTimeFormatter;
27-
import java.util.*;
28-
import java.util.concurrent.*;
27+
import java.util.ArrayList;
28+
import java.util.Arrays;
29+
import java.util.Collections;
30+
import java.util.List;
31+
import java.util.Set;
32+
import java.util.concurrent.BlockingQueue;
33+
import java.util.concurrent.Callable;
34+
import java.util.concurrent.CountDownLatch;
35+
import java.util.concurrent.ExecutorService;
36+
import java.util.concurrent.Executors;
37+
import java.util.concurrent.LinkedBlockingQueue;
38+
import java.util.concurrent.ThreadFactory;
39+
import java.util.concurrent.TimeUnit;
2940
import java.util.concurrent.atomic.LongAdder;
3041
import java.util.function.Consumer;
3142

@@ -187,13 +198,20 @@ public void accept(QueryResult queryResult) {
187198
* see Issue #602
188199
*/
189200
@Test
190-
public void testCallbackQueryFailureHandling() {
201+
public void testCallbackQueryFailureHandling() throws Throwable {
191202
final AsyncResult<QueryResult> res = new AsyncResult<>();
192203

193204
this.influxDB.query(new Query("SHOW SERRIES"), res.resultConsumer, res.errorConsumer);
194205

195-
Assertions.assertThrows(InfluxDBException.class, res::result,
196-
"Malformed query should throw InfluxDBException");
206+
try{
207+
res.result();
208+
Assertions.fail("Malformed query should throw InfluxDBException");
209+
}
210+
catch (InfluxDBException e){
211+
Assertions.assertTrue(e.getMessage().matches("Bad Request[\\s-a-zA-Z{}:\",0-9]*error parsing query: " +
212+
"found SERRIES, expected[\\s-a-zA-Z{}:\",0-9]*"),
213+
"Error string not expected");
214+
}
197215
}
198216

199217
/**

0 commit comments

Comments
 (0)