]> www.infradead.org Git - users/hch/nvme-cli.git/commitdiff
nvme-cli: tests: remove nsid parameter in a testcase of error-log
authorMinwoo Im <minwoo.im.dev@gmail.com>
Sat, 13 Jan 2018 17:03:06 +0000 (02:03 +0900)
committerMinwoo Im <minwoo.im.dev@gmail.com>
Sat, 13 Jan 2018 17:03:06 +0000 (02:03 +0900)
Remove nsid parameter in a testcase of error-log subcommand.

This parameter has been removed by a commit 98ffd4b on the master branch
of linux-nvme/nvme-cli.

Signed-off-by: Minwoo Im <minwoo.im.dev@gmail.com>
tests/nvme_error_log_test.py
tests/nvme_test.py

index cedb24bbc679c5989f35ac20ae95cee638f61dfc..a6a451ddb2b90d6ad8cb753f0f4fb6527250c642 100644 (file)
@@ -21,7 +21,6 @@
 NVMe Smart Log Verification Testcase:-
 
     1. Execute error-log on controller.
-    2. Execute error-log on each available namespace.
 
 """
 
@@ -57,30 +56,8 @@ class TestNVMeErrorLogCmd(TestNVMe):
             - Returns:
                 - 0 on success, error code on failure.
         """
-        return self.get_error_log("0xFFFFFFFF")
-
-    def get_error_log_ns(self, nsid):
-        """ Wrapper for executing error-log on a namespace.
-            - Args:
-                - nsid: namespace id to be used in error-log command.
-            - Returns:
-                - 0 on success, error code on failure.
-        """
-        return self.get_error_log(nsid)
-
-    def get_error_log_all_ns(self):
-        """ Wrapper for executing error-log on all the namespaces.
-            - Args:
-                - None:
-            - Returns:
-                - 0 on success, error code on failure.
-        """
-        ns_list = self.get_ns_list()
-        for nsid in range(0, len(ns_list)):
-            self.get_error_log_ns(ns_list[nsid])
-        return 0
+        return self.get_error_log()
 
     def test_get_error_log(self):
         """ Testcase main """
         assert_equal(self.get_error_log_ctrl(), 0)
-        assert_equal(self.get_error_log_all_ns(), 0)
index c3ee16ec4c571183df926432260f1b0718076bce..c4541fdcf69542b114e240f486aba8cd2fbcdca4 100644 (file)
@@ -349,15 +349,15 @@ class TestNVMe(object):
         print "host_write_commands " + host_write_commands
         return err
 
-    def get_error_log(self, nsid):
+    def get_error_log(self):
         """ Wrapper for nvme error-log command.
             - Args:
-                - nsid : namespace id to get error log from.
+                - None
             - Returns:
                 - 0 on success, error code on failure.
         """
         pattern = re.compile("^ Entry\[[ ]*[0-9]+\]")
-        error_log_cmd = "nvme error-log " + self.ctrl + " -n " + str(nsid)
+        error_log_cmd = "nvme error-log " + self.ctrl
         proc = subprocess.Popen(error_log_cmd,
                                 shell=True,
                                 stdout=subprocess.PIPE)