From bac0f507ee8b210af06eca797a33eac05bf47c9b Mon Sep 17 00:00:00 2001 From: dparmar18 Date: Wed, 4 May 2022 16:56:48 +0530 Subject: [PATCH] qa: removed test_without_target_dir Rationale: get and put now demand both the paths mandatorily. Also testing of get and put without target paths have been take of in other tests in class TestGetAndPut(). Signed-off-by: Dhairya Parmar (cherry picked from commit 31ce2eff60242f962ba648f1c71f616ec5c90284) --- qa/tasks/cephfs/test_cephfs_shell.py | 32 ---------------------------- 1 file changed, 32 deletions(-) diff --git a/qa/tasks/cephfs/test_cephfs_shell.py b/qa/tasks/cephfs/test_cephfs_shell.py index a7b9bbd28da..32a8c089b1c 100644 --- a/qa/tasks/cephfs/test_cephfs_shell.py +++ b/qa/tasks/cephfs/test_cephfs_shell.py @@ -309,38 +309,6 @@ class TestRmdir(TestCephFSShell): self.mount_a.stat(self.dir_name) class TestGetAndPut(TestCephFSShell): - def test_without_target_dir(self): - """ - Test put and get commands without target path. - """ - tempdir = self.mount_a.client_remote.mkdtemp() - tempdirname = path.basename(tempdir) - files = ('dump1', 'dump2', 'dump3', tempdirname) - - for i, file_ in enumerate(files[ : -1]): - size = i + 1 - ofarg = 'of=' + path.join(tempdir, file_) - bsarg = 'bs=' + str(size) + 'M' - self.mount_a.run_shell_payload(f"dd if=/dev/urandom {ofarg} {bsarg} count=1") - - self.run_cephfs_shell_cmd('put ' + tempdir) - for file_ in files: - if file_ == tempdirname: - self.mount_a.stat(path.join(self.mount_a.mountpoint, file_)) - else: - self.mount_a.stat(path.join(self.mount_a.mountpoint, - tempdirname, file_)) - - self.mount_a.run_shell_payload(f"rm -rf {tempdir}") - - self.run_cephfs_shell_cmd('get ' + tempdirname) - pwd = self.get_cephfs_shell_cmd_output('!pwd') - for file_ in files: - if file_ == tempdirname: - self.mount_a.run_shell_payload(f"stat {path.join(pwd, file_)}") - else: - self.mount_a.run_shell_payload(f"stat {path.join(pwd, tempdirname, file_)}") - def test_get_with_target_name(self): """ Test that get passes with target name -- 2.47.3