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, 11 deletions

diff --git a/src/mongo/db/repl/applier_test.cpp b/src/mongo/db/repl/applier_test.cpp
index 0774b1bed29..f876a884605 100644
--- a/src/mongo/db/repl/applier_test.cpp
+++ b/src/mongo/db/repl/applier_test.cpp
@@ -49,17 +49,14 @@ namespace {
 
     class ApplierTest : public ReplicationExecutorTest {
     public:
-        /**
-         * Creates an initial error status suitable for checking if
-         * applier has modified the 'status' field in test fixture.
-         */
-        static Status getDetectableErrorStatus();
 
-        void setUp() override;
-        void tearDown() override;
         Applier* getApplier() const;
 
     protected:
+
+        void setUp() override;
+        void tearDown() override;
+
         /**
          * Test function to check behavior when we fail to apply one of the operations.
          */
@@ -69,10 +66,6 @@ namespace {
         std::unique_ptr<unittest::Barrier> _barrier;
     };
 
-    Status ApplierTest::getDetectableErrorStatus() {
-        return Status(ErrorCodes::InternalError, "Not mutated");
-    }
-
     void ApplierTest::setUp() {
         ReplicationExecutorTest::setUp();
         launchExecutorThread();