Add clarifying comment on metric update testing

1 files changed, 3 insertions, 0 deletions

diff --git a/storage/src/tests/distributor/visitoroperationtest.cpp b/storage/src/tests/distributor/visitoroperationtest.cpp
index 055fe9b9ab9..7bd9eea29b0 100644
--- a/storage/src/tests/distributor/visitoroperationtest.cpp
+++ b/storage/src/tests/distributor/visitoroperationtest.cpp
@@ -1674,6 +1674,9 @@ VisitorOperationTest::statistical_metrics_not_updated_on_wrong_distribution()
                         "ReturnCode(WRONG_DISTRIBUTION, distributor:100 storage:2)"),
             runEmptyVisitor(createVisitorCommand("wrongdist", id, nullId)));
 
+    // Note that we're testing the number of _times_ the metric has been
+    // updated, not the value with which it's been updated (which would be zero
+    // even in the case we actually did update the statistical metrics).
     CPPUNIT_ASSERT_EQUAL(int64_t(0), defaultVisitorMetrics().buckets_per_visitor.getCount());
     CPPUNIT_ASSERT_EQUAL(int64_t(0), defaultVisitorMetrics().docs_per_visitor.getCount());
     CPPUNIT_ASSERT_EQUAL(int64_t(0), defaultVisitorMetrics().bytes_per_visitor.getCount());