Merge pull request #120 from RedisLabsModules/fixes_following_rdb_preamble_flag

Fix typo in compare env field

Author: alonre24

RLTest/env.py

@@ -151,7 +151,7 @@ def getKwargs(self):
 class Env:
     RTestInstance = None
     EnvCompareParams = ['module', 'moduleArgs', 'env', 'useSlaves', 'shardsCount', 'useAof',
-                        'udsRdbPreamble', 'forceTcp']
+                        'useRdbPreamble', 'forceTcp']
 
     def compareEnvs(self, env):
         if env is None:

tests/unit/test_env.py

@@ -40,6 +40,19 @@ def test_get_env_str(self):
         self.env.stop()
         assert self.env.isUp() == False
 
+    def test_compare_env(self):
+        self.env = Env(env='oss', logDir=self.test_dir, redisBinaryPath=REDIS_BINARY,
+                       redisEnterpriseBinaryPath=REDIS_ENTERPRISE_BINARY, dmcBinaryPath=DMC_PROXY_BINARY)
+        env = Env(env='oss', logDir=self.test_dir, redisBinaryPath=REDIS_BINARY,
+                  redisEnterpriseBinaryPath=REDIS_ENTERPRISE_BINARY, dmcBinaryPath=DMC_PROXY_BINARY)
+        assert self.env.compareEnvs(env) is True
+        env.stop()
+        env = Env(env='oss', logDir=self.test_dir, redisBinaryPath=REDIS_BINARY,
+                  redisEnterpriseBinaryPath=REDIS_ENTERPRISE_BINARY, dmcBinaryPath=DMC_PROXY_BINARY, useAof=True)
+        assert self.env.compareEnvs(env) is False
+        env.stop()
+        self.env.stop()
+
     def test_get_connection(self):
         pass