From aa9938aacb02ab010754241d468d34702be9e473 Mon Sep 17 00:00:00 2001 From: dparmar18 Date: Tue, 19 Apr 2022 16:58:11 +0530 Subject: [PATCH] qa: added testcase to check get cmd doesn't create subdirs on its own Signed-off-by: Dhairya Parmar --- qa/tasks/cephfs/test_cephfs_shell.py | 13 +++++++++++++ 1 file changed, 13 insertions(+) diff --git a/qa/tasks/cephfs/test_cephfs_shell.py b/qa/tasks/cephfs/test_cephfs_shell.py index e2b86bef989..71084816630 100644 --- a/qa/tasks/cephfs/test_cephfs_shell.py +++ b/qa/tasks/cephfs/test_cephfs_shell.py @@ -378,6 +378,19 @@ class TestGetAndPut(TestCephFSShell): # NOTE: following command must run successfully. self.mount_a.run_shell('stat dump5', cwd=None) + def test_get_doesnt_create_dir(self): + # if get cmd is creating subdirs on its own then dump7 will be + # stored as ./dump7/tmp/dump7 and not ./dump7, therefore + # if doing `cat ./dump7` returns non-zero exit code(i.e. 1) then + # it implies that no such file exists at that location + dir_abspath = path.join(self.mount_a.mountpoint, 'tmp') + self.mount_a.run_shell_payload(f"mkdir {dir_abspath}") + self.mount_a.client_remote.write_file(path.join(dir_abspath, 'dump7'), + 'somedata') + self.get_cephfs_shell_cmd_output("get /tmp/dump7 ./dump7") + # test that dump7 exists + self.mount_a.run_shell("cat ./dump7", cwd=None) + def test_get_to_console(self): """ Test that get passes with target name -- 2.39.5