Skip to content

Commit 8eab315

Browse files
committed
Polishing.
Add $median to documentation. Add test to ensure adherence to TypedAggregationContext. Remove duplicate imports. See #4472 Original Pull Request: #4515
1 parent 8c12384 commit 8eab315

File tree

3 files changed

+9
-3
lines changed

3 files changed

+9
-3
lines changed

spring-data-mongodb/src/main/java/org/springframework/data/mongodb/core/aggregation/ArithmeticOperators.java

-2
Original file line numberDiff line numberDiff line change
@@ -32,8 +32,6 @@
3232
import org.springframework.data.mongodb.core.aggregation.AccumulatorOperators.Sum;
3333
import org.springframework.data.mongodb.core.aggregation.SetWindowFieldsOperation.WindowUnit;
3434
import org.springframework.data.mongodb.core.aggregation.SetWindowFieldsOperation.WindowUnits;
35-
import org.springframework.data.mongodb.core.aggregation.SetWindowFieldsOperation.WindowUnit;
36-
import org.springframework.data.mongodb.core.aggregation.SetWindowFieldsOperation.WindowUnits;
3735
import org.springframework.lang.Nullable;
3836
import org.springframework.util.Assert;
3937
import org.springframework.util.ObjectUtils;

spring-data-mongodb/src/test/java/org/springframework/data/mongodb/core/aggregation/AccumulatorOperatorsUnitTests.java

+8
Original file line numberDiff line numberDiff line change
@@ -152,6 +152,14 @@ void rendersMedianWithExpression() {
152152
.isEqualTo(Document.parse("{ $median: { input: [\"$scoreOne\", {\"$sum\": \"$scoreTwo\"}], method: \"approximate\" } }"));
153153
}
154154

155+
@Test // GH-4472
156+
void rendersMedianCorrectlyWithTypedAggregationContext() {
157+
158+
assertThat(valueOf("midichlorianCount").median()
159+
.toDocument(TestAggregationContext.contextFor(Jedi.class)))
160+
.isEqualTo(Document.parse("{ $median: { input: \"$force\", method: \"approximate\" } }"));
161+
}
162+
155163
static class Jedi {
156164

157165
String name;

src/main/antora/modules/ROOT/pages/mongodb/aggregation-framework.adoc

+1-1
Original file line numberDiff line numberDiff line change
@@ -112,7 +112,7 @@ At the time of this writing, we provide support for the following Aggregation Op
112112
| `setEquals`, `setIntersection`, `setUnion`, `setDifference`, `setIsSubset`, `anyElementTrue`, `allElementsTrue`
113113

114114
| Group/Accumulator Aggregation Operators
115-
| `addToSet`, `bottom`, `bottomN`, `covariancePop`, `covarianceSamp`, `expMovingAvg`, `first`, `firstN`, `last`, `lastN` `max`, `maxN`, `min`, `minN`, `avg`, `push`, `sum`, `top`, `topN`, `count` (+++*+++), `percentile`, `stdDevPop`, `stdDevSamp`
115+
| `addToSet`, `bottom`, `bottomN`, `covariancePop`, `covarianceSamp`, `expMovingAvg`, `first`, `firstN`, `last`, `lastN` `max`, `maxN`, `min`, `minN`, `avg`, `push`, `sum`, `top`, `topN`, `count` (+++*+++), `median`, `percentile`, `stdDevPop`, `stdDevSamp`
116116

117117
| Arithmetic Aggregation Operators
118118
| `abs`, `acos`, `acosh`, `add` (+++*+++ via `plus`), `asin`, `asin`, `atan`, `atan2`, `atanh`, `ceil`, `cos`, `cosh`, `derivative`, `divide`, `exp`, `floor`, `integral`, `ln`, `log`, `log10`, `mod`, `multiply`, `pow`, `round`, `sqrt`, `subtract` (+++*+++ via `minus`), `sin`, `sinh`, `tan`, `tanh`, `trunc`

0 commit comments

Comments
 (0)