[fix] Fixed required modules check on empty modules list (#94)

* [fix] Fixed required modules check on empty modules list

* Bumping version from 0.1.70 to 0.1.71

Author: filipecosta90

pyproject.toml

@@ -1,6 +1,6 @@
 [tool.poetry]
 name = "redisbench-admin"
-version = "0.1.70"
+version = "0.1.71"
 description = "Redis benchmark run helper. A wrapper around Redis and Redis Modules benchmark tools ( ftsb_redisearch, memtier_benchmark, redis-benchmark, aibench, etc... )."
 authors = ["filipecosta90 <[email protected]>"]
 readme = "README.md"

redisbench_admin/run_local/run_local.py

@@ -139,20 +139,21 @@ def run_local_command_logic(args):
                     module_names,
                     _,
                 ) = extract_module_semver_from_info_modules_cmd(stdout)
-                if len(required_modules) > 0:
-                    logging.info(
-                        "Checking if the following required modules {} are present".format(
-                            required_modules
-                        )
-                    )
-                for required_module in required_modules:
-                    if required_module not in module_names:
-                        raise Exception(
-                            "Unable to detect required module {} in {}. Aborting...".format(
-                                required_module,
-                                module_names,
+                if required_modules is not None:
+                    if len(required_modules) > 0:
+                        logging.info(
+                            "Checking if the following required modules {} are present".format(
+                                required_modules
                             )
                         )
+                        for required_module in required_modules:
+                            if required_module not in module_names:
+                                raise Exception(
+                                    "Unable to detect required module {} in {}. Aborting...".format(
+                                        required_module,
+                                        module_names,
+                                    )
+                                )
 
                 # setup the benchmark
                 start_time, start_time_ms, start_time_str = get_start_time_vars()

redisbench_admin/run_remote/run_remote.py

@@ -398,22 +398,24 @@ def run_remote_command_logic(args):
                     module_names, artifact_versions = extract_artifact_version_remote(
                         server_public_ip, server_plaintext_port, username, private_key
                     )
-                    if len(required_modules) > 0:
-                        logging.info(
-                            "Checking if the following required modules {} are present".format(
-                                required_modules
-                            )
-                        )
-                    for required_module in required_modules:
-                        if required_module not in module_names:
-                            raise Exception(
-                                "Unable to detect required module {} in {}, using remote DB with IP {}, PORT {}. Aborting...".format(
-                                    required_module,
-                                    module_names,
-                                    server_public_ip,
-                                    server_plaintext_port,
+                    if required_modules is not None:
+                        if len(required_modules) > 0:
+                            logging.info(
+                                "Checking if the following required modules {} are present".format(
+                                    required_modules
                                 )
                             )
+                            for required_module in required_modules:
+                                if required_module not in module_names:
+                                    raise Exception(
+                                        "Unable to detect required module {} in {}"
+                                        ", using remote DB with IP {}, PORT {}. Aborting...".format(
+                                            required_module,
+                                            module_names,
+                                            server_public_ip,
+                                            server_plaintext_port,
+                                        )
+                                    )
 
                     artifact_version = artifact_versions[0]
                     (