Skip to content

Commit

Permalink
Test: Update AggregatesTest to use List of Doubles
Browse files Browse the repository at this point in the history
The kotlinx codec was picking up the double array as
it was serializable. However, it should be limited
to top level data, value or sealed kotlin classes.

JAVA-5035
  • Loading branch information
rozza committed Jun 23, 2023
1 parent 206a119 commit a7b9ce0
Showing 1 changed file with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -97,11 +97,11 @@ public void shouldGroupWithQuantile(final BsonField quantileAccumulator,
private static Stream<Arguments> setWindowFieldWithQuantileSource() {
return Stream.of(
Arguments.of(null,
WindowOutputFields.percentile("result", "$num1", new double[]{0.1, 0.9}, QuantileMethod.approximate(),
WindowOutputFields.percentile("result", "$num1", asList(0.1, 0.9), QuantileMethod.approximate(),
documents(UNBOUNDED, UNBOUNDED)),
asList(asList(1.0, 3.0), asList(1.0, 3.0), asList(1.0, 3.0))),
Arguments.of("$partitionId",
WindowOutputFields.percentile("result", "$num1", new double[]{0.1, 0.9}, QuantileMethod.approximate(), null),
WindowOutputFields.percentile("result", "$num1", asList(0.1, 0.9), QuantileMethod.approximate(), null),
asList(asList(1.0, 2.0), asList(1.0, 2.0), asList(3.0, 3.0))),
Arguments.of(null,
WindowOutputFields.median("result", "$num1", QuantileMethod.approximate(), documents(UNBOUNDED, UNBOUNDED)),
Expand Down

0 comments on commit a7b9ce0

Please sign in to comment.