SERVER-18036 moved getDetectableErrorStatus() to replication executor test fixture

make all setUp/tearDown test fixture functions for data replicator protected

1 files changed, 4 insertions, 7 deletions

diff --git a/src/mongo/client/fetcher_test.cpp b/src/mongo/client/fetcher_test.cpp
index 6414dbae1f9..204e89892d2 100644
--- a/src/mongo/client/fetcher_test.cpp
+++ b/src/mongo/client/fetcher_test.cpp
@@ -49,10 +49,7 @@ namespace {
 
     class FetcherTest : public ReplicationExecutorTest {
     public:
-        static Status getDetectableErrorStatus();
         FetcherTest();
-        void setUp() override;
-        void tearDown() override;
         void clear();
         void scheduleNetworkResponse(const BSONObj& obj);
         void scheduleNetworkResponse(ErrorCodes::Error code, const std::string& reason);
@@ -61,6 +58,10 @@ namespace {
         void finishProcessingNetworkResponse();
 
     protected:
+
+        void setUp() override;
+        void tearDown() override;
+
         Status status;
         CursorId cursorId;
         Fetcher::Documents documents;
@@ -76,10 +77,6 @@ namespace {
                        BSONObjBuilder* getMoreBob);
     };
 
-    Status FetcherTest::getDetectableErrorStatus() {
-        return Status(ErrorCodes::InternalError, "Not mutated");
-    }
-
     FetcherTest::FetcherTest()
         : status(getDetectableErrorStatus()),
           cursorId(-1),