2/N remove :memcached target: switch to MockMemcached() in mcrouter tests

Summary:
<public visible description>
The goal is to switch all tests consistently to use MockMemcached

Reviewed By: alikhtarov

Differential Revision: D78342902

fbshipit-source-id: e4dbb946343985bc2b6ba1aa064642d28e24ec1b

Author: Ivan Gramberg

third-party/mcrouter/src/mcrouter/test/test_additional_fields.py

@@ -6,15 +6,15 @@
 
 # pyre-unsafe
 
-from mcrouter.test.MCProcess import Memcached
+from mcrouter.test.MCProcess import MockMemcached
 from mcrouter.test.McrouterTestCase import McrouterTestCase
 
 
 class TestAdditionalFields(McrouterTestCase):
     config = "./mcrouter/test/test_basic_caret.json"
 
     def setUp(self):
-        self.mc = self.add_server(Memcached())
+        self.mc = self.add_server(MockMemcached())
         self.mcrouter = self.add_mcrouter(self.config)
 
     def test_basic(self):

third-party/mcrouter/src/mcrouter/test/test_allow_only_gets.py

@@ -8,15 +8,15 @@
 
 import time
 
-from mcrouter.test.MCProcess import Memcached
+from mcrouter.test.MCProcess import MockMemcached
 from mcrouter.test.McrouterTestCase import McrouterTestCase
 
 
 class TestAllowGetsOnly(McrouterTestCase):
     config = "./mcrouter/test/mcrouter_test_basic_1_1_1.json"
 
     def setUp(self):
-        self.mc = self.add_server(Memcached())
+        self.mc = self.add_server(MockMemcached())
         extra_args = ["--allow-only-gets"]
         self.mcr = self.add_mcrouter(self.config, extra_args=extra_args)
 

third-party/mcrouter/src/mcrouter/test/test_bigvalue.py

@@ -6,7 +6,7 @@
 
 # pyre-unsafe
 
-from mcrouter.test.MCProcess import Memcached
+from mcrouter.test.MCProcess import MockMemcached
 from mcrouter.test.McrouterTestCase import McrouterTestCase
 
 MC_MSG_FLAG_BIG_VALUE = 0x8000
@@ -25,7 +25,7 @@ class TestBigvalue(McrouterTestCase):
     ]
 
     def setUp(self):
-        self.add_server(Memcached())
+        self.add_server(MockMemcached())
 
     def get_mcrouter(self):
         return self.add_mcrouter(self.config, extra_args=self.extra_args)

third-party/mcrouter/src/mcrouter/test/test_bucketized_poolroute.py

@@ -6,7 +6,7 @@
 
 # pyre-unsafe
 
-from mcrouter.test.MCProcess import Memcached
+from mcrouter.test.MCProcess import MockMemcached
 from mcrouter.test.McrouterTestCase import McrouterTestCase
 
 
@@ -16,11 +16,11 @@ class TestBucketizedPoolRoute(McrouterTestCase):
 
     def setUp(self):
         # The order here corresponds to the order of hosts in the .json
-        self.mc1 = self.add_server(Memcached())
-        self.mc2 = self.add_server(Memcached())
-        self.mc3 = self.add_server(Memcached())
-        self.mc4 = self.add_server(Memcached())
-        self.mc5 = self.add_server(Memcached())
+        self.mc1 = self.add_server(MockMemcached())
+        self.mc2 = self.add_server(MockMemcached())
+        self.mc3 = self.add_server(MockMemcached())
+        self.mc4 = self.add_server(MockMemcached())
+        self.mc5 = self.add_server(MockMemcached())
 
     def get_mcrouter(self, additional_args=()):
         extra_args = self.extra_args[:]
@@ -47,11 +47,11 @@ class TestNonBucketizedPoolRoute(McrouterTestCase):
 
     def setUp(self):
         # The order here corresponds to the order of hosts in the .json
-        self.mc1 = self.add_server(Memcached())
-        self.mc2 = self.add_server(Memcached())
-        self.mc3 = self.add_server(Memcached())
-        self.mc4 = self.add_server(Memcached())
-        self.mc5 = self.add_server(Memcached())
+        self.mc1 = self.add_server(MockMemcached())
+        self.mc2 = self.add_server(MockMemcached())
+        self.mc3 = self.add_server(MockMemcached())
+        self.mc4 = self.add_server(MockMemcached())
+        self.mc5 = self.add_server(MockMemcached())
 
     def get_mcrouter(self, additional_args=()):
         extra_args = self.extra_args[:]

third-party/mcrouter/src/mcrouter/test/test_carbonlookaside_route.py

@@ -13,6 +13,8 @@
 import time
 from string import Template
 
+from mcrouter.test.MCProcess import MockMemcached
+
 from mcrouter.test.McrouterTestCase import McrouterTestCase
 
 
@@ -153,7 +155,7 @@ class TestCarbonLookasideRouteBasic(McrouterTestCase):
     extra_args = []
 
     def setUp(self):
-        self.mc = self.add_server(self.make_memcached())
+        self.mc = self.add_server(MockMemcached())
         self.tmpConfig = CarbonLookasideTmpConfig(
             self.prefix, self.ttl, self.mc.getport()
         )
@@ -214,7 +216,7 @@ class TestCarbonLookasideRouteExpiry(McrouterTestCase):
     extra_args = []
 
     def setUp(self):
-        self.mc = self.add_server(self.make_memcached())
+        self.mc = self.add_server(MockMemcached())
         self.tmpConfig = CarbonLookasideTmpConfig(
             self.prefix, self.ttl, self.mc.getport()
         )
@@ -253,7 +255,7 @@ class TestCarbonLookasideRouteNoExpiry(McrouterTestCase):
     extra_args = []
 
     def setUp(self):
-        self.mc = self.add_server(self.make_memcached())
+        self.mc = self.add_server(MockMemcached())
         self.tmpConfig = CarbonLookasideTmpConfig(
             self.prefix, self.ttl, self.mc.getport()
         )
@@ -291,7 +293,7 @@ class TestCarbonLookasideRouteLeases(McrouterTestCase):
     extra_args = []
 
     def setUp(self):
-        self.mc = self.add_server(self.make_memcached())
+        self.mc = self.add_server(MockMemcached())
         self.tmpConfig = CarbonLookasideTmpConfig(
             self.prefix, self.ttl, self.mc.getport(), "true", 10, 10
         )
@@ -352,7 +354,7 @@ class TestCarbonLookasideRouteLeasesHotMiss(McrouterTestCase):
     extra_args = []
 
     def setUp(self):
-        self.mc = self.add_server(self.make_memcached())
+        self.mc = self.add_server(MockMemcached())
         self.tmpConfig = CarbonLookasideTmpConfig(
             self.prefix, self.ttl, self.mc.getport(), "true", 10, 3, 0, 5000
         )
@@ -411,7 +413,7 @@ class TestCarbonLookasideRouteExpiryMsTTLBase(McrouterTestCase):
     extra_args = []
 
     def setUp(self):
-        self.mc = self.add_server(self.make_memcached())
+        self.mc = self.add_server(MockMemcached())
         self.tmpConfig = CarbonLookasideTmpConfig(
             self.prefix,
             self.ttl,
@@ -451,7 +453,7 @@ class TestCarbonLookasideRouteExpiryMsTTL(McrouterTestCase):
     extra_args = []
 
     def setUp(self):
-        self.mc = self.add_server(self.make_memcached())
+        self.mc = self.add_server(MockMemcached())
         self.tmpConfig = CarbonLookasideTmpConfig(
             self.prefix,
             self.ttl,

third-party/mcrouter/src/mcrouter/test/test_config_params.py

@@ -9,7 +9,7 @@
 import tempfile
 import time
 
-from mcrouter.test.MCProcess import Memcached
+from mcrouter.test.MCProcess import MockMemcached
 from mcrouter.test.McrouterTestCase import McrouterTestCase
 
 
@@ -61,7 +61,7 @@ class TestConstShardHash(McrouterTestCase):
     config = "./mcrouter/test/test_config_params.json"
 
     def test_config_params(self):
-        mc = self.add_server(Memcached())
+        mc = self.add_server(MockMemcached())
         self.port_map = {}
         extra_args = ["--config-params", f"PORT:{mc.getport()},POOL:A"]
         mcrouter = self.add_mcrouter(self.config, extra_args=extra_args)

third-party/mcrouter/src/mcrouter/test/test_const_shard_hash.py

@@ -6,7 +6,7 @@
 
 # pyre-unsafe
 
-from mcrouter.test.MCProcess import Memcached
+from mcrouter.test.MCProcess import MockMemcached
 from mcrouter.test.McrouterTestCase import McrouterTestCase
 
 
@@ -15,8 +15,8 @@ class TestConstShardHash(McrouterTestCase):
     extra_args = []
 
     def test_const_shard_hash(self):
-        mc1 = self.add_server(Memcached())
-        mc2 = self.add_server(Memcached())
+        mc1 = self.add_server(MockMemcached())
+        mc2 = self.add_server(MockMemcached())
         mcrouter = self.add_mcrouter(self.config, extra_args=self.extra_args)
 
         key = "foo:0:test"

third-party/mcrouter/src/mcrouter/test/test_custom_failover.py

@@ -7,7 +7,7 @@
 # pyre-unsafe
 
 
-from mcrouter.test.MCProcess import Memcached
+from mcrouter.test.MCProcess import MockMemcached
 from mcrouter.test.McrouterTestCase import McrouterTestCase
 from mcrouter.test.mock_servers import CustomErrorServer
 
@@ -18,7 +18,7 @@ class TestCustomFailover(McrouterTestCase):
 
     def setUp(self):
         self.mc1 = self.add_server(CustomErrorServer())
-        self.mc2 = self.add_server(Memcached())
+        self.mc2 = self.add_server(MockMemcached())
         self.mcr = self.add_mcrouter(self.config, extra_args=self.extra_args)
 
     def test_failover(self):
@@ -39,7 +39,7 @@ class TestCustomFailoverOverride(McrouterTestCase):
 
     def setUp(self):
         self.mc1 = self.add_server(CustomErrorServer())
-        self.mc2 = self.add_server(Memcached())
+        self.mc2 = self.add_server(MockMemcached())
 
         self.mcr = self.add_mcrouter(self.config, extra_args=self.extra_args)
 

third-party/mcrouter/src/mcrouter/test/test_debug_fifos.py

@@ -9,7 +9,7 @@
 import os
 import time
 
-from mcrouter.test.MCProcess import Memcached
+from mcrouter.test.MCProcess import MockMemcached
 from mcrouter.test.McrouterTestCase import McrouterTestCase
 
 
@@ -18,7 +18,7 @@ class TestDebugFifos(McrouterTestCase):
     extra_args = ["--proxy-threads=1"]
 
     def setUp(self):
-        self.add_server(Memcached())
+        self.add_server(MockMemcached())
         self.mcrouter = self.add_mcrouter(self.config, extra_args=self.extra_args)
 
     def get_fifo(self, substr):

third-party/mcrouter/src/mcrouter/test/test_latency_injection_route.py

@@ -8,7 +8,7 @@
 
 from datetime import datetime
 
-from mcrouter.test.MCProcess import Memcached
+from mcrouter.test.MCProcess import MockMemcached
 from mcrouter.test.McrouterTestCase import McrouterTestCase
 
 
@@ -18,7 +18,7 @@ class TestLatencyInjectionRoute(McrouterTestCase):
     config_latency_total = "./mcrouter/test/test_latency_injection_total.json"
 
     def setUp(self) -> None:
-        self.mc = self.add_server(Memcached())
+        self.mc = self.add_server(MockMemcached())
 
         self.mcrouter_latency_before = self.add_mcrouter(self.config_latency_before)
         self.mcrouter_latency_after = self.add_mcrouter(self.config_latency_after)