]> git.apps.os.sepia.ceph.com Git - ceph.git/commit
qa/cephfs/cap_tester: simplify CapTester and its instantiation
authorRishabh Dave <ridave@redhat.com>
Tue, 18 Apr 2023 14:55:01 +0000 (20:25 +0530)
committerRishabh Dave <ridave@redhat.com>
Sat, 6 Apr 2024 10:14:16 +0000 (15:44 +0530)
commit5f183580d06db7a8050f5e310ce49d3b99c0dc27
treec960177ff353d13469a4a59a1415d8bce864641d
parent331040b2516c1fefc206d216a265072c86e54bb0
qa/cephfs/cap_tester: simplify CapTester and its instantiation

Class CapTester contains two distinct immiscible group of methods: one
that tests MON caps and other that tests MDS caps. When using CapTester
for the former reason the instantiation neither needs mount object and
the path where files for testing will be created nor it needs to run the
method that creates files for testing rw permissions. When using
this class for latter the case is the exact opposite.

Create 2 separate classes for each of these purpose and class that
inherits both of these classes so that instantiating the class becomes
as simple as it can be.

Signed-off-by: Rishabh Dave <ridave@redhat.com>
(cherry picked from commit f0ffade0525674b1f851797b1107060d8aa93d5c)

Conflicts:
 - qa/tasks/cephfs/caps_helper.py:
   This file went through changes due to merging of commit 59c9104c54d6.
qa/tasks/cephfs/caps_helper.py
qa/tasks/cephfs/test_admin.py
qa/tasks/cephfs/test_multifs_auth.py