|
While the defect report isn't quite right (the call to future.default_timed_get() will wait for the future or time out), we could improve on the current code by capturing the return and confirming that the chunk manager is available. We will use this ticket to do so. Here is a proposed diff:
diff --git a/src/mongo/s/query/sharded_agg_test_fixture.h b/src/mongo/s/query/sharded_agg_test_fixture.h
|
index 48f1f71f99..f72d6c2f6e 100644
|
--- a/src/mongo/s/query/sharded_agg_test_fixture.h
|
+++ b/src/mongo/s/query/sharded_agg_test_fixture.h
|
@@ -109,7 +109,8 @@ public:
|
return response;
|
}());
|
|
- future.default_timed_get().get();
|
+ auto routingInfo = future.default_timed_get();
|
+ ASSERT(routingInfo->cm());
|
}
|
|
protected:
|
|