From: Patrick Seidensal <pseidensal@suse.com>
Date: Fri, 23 Aug 2019 23:25:45 +0000 (+0200)
Subject: mgr/dashboard: refactoring Helper class to remove PageHelper instances
X-Git-Tag: v15.1.0~1745^2~8
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=ac901bf4cf6f06d440e1e845255afb8357b8eede;p=ceph.git

mgr/dashboard: refactoring Helper class to remove PageHelper instances

Signed-off-by: Patrick Seidensal <pseidensal@suse.com>
---

diff --git a/src/pybind/mgr/dashboard/frontend/e2e/block/images.e2e-spec.ts b/src/pybind/mgr/dashboard/frontend/e2e/block/images.e2e-spec.ts
index a4f3c6d6d4c6..f046966451aa 100644
--- a/src/pybind/mgr/dashboard/frontend/e2e/block/images.e2e-spec.ts
+++ b/src/pybind/mgr/dashboard/frontend/e2e/block/images.e2e-spec.ts
@@ -7,8 +7,8 @@ describe('Images page', () => {
   let images: ImagesPageHelper;
 
   beforeAll(() => {
-    images = new Helper().images;
-    pools = new Helper().pools;
+    images = new ImagesPageHelper();
+    pools = new PoolPageHelper();
   });
 
   afterEach(async () => {
diff --git a/src/pybind/mgr/dashboard/frontend/e2e/block/iscsi.e2e-spec.ts b/src/pybind/mgr/dashboard/frontend/e2e/block/iscsi.e2e-spec.ts
index 0e78102895ef..e4aee230624e 100644
--- a/src/pybind/mgr/dashboard/frontend/e2e/block/iscsi.e2e-spec.ts
+++ b/src/pybind/mgr/dashboard/frontend/e2e/block/iscsi.e2e-spec.ts
@@ -5,7 +5,7 @@ describe('Iscsi Page', () => {
   let iscsi: IscsiPageHelper;
 
   beforeAll(() => {
-    iscsi = new Helper().iscsi;
+    iscsi = new IscsiPageHelper();
   });
 
   afterEach(async () => {
diff --git a/src/pybind/mgr/dashboard/frontend/e2e/block/mirroring.e2e-spec.ts b/src/pybind/mgr/dashboard/frontend/e2e/block/mirroring.e2e-spec.ts
index 3e1fb764c0c8..205779a6d2e8 100644
--- a/src/pybind/mgr/dashboard/frontend/e2e/block/mirroring.e2e-spec.ts
+++ b/src/pybind/mgr/dashboard/frontend/e2e/block/mirroring.e2e-spec.ts
@@ -7,8 +7,8 @@ describe('Mirroring page', () => {
   let mirroring: MirroringPageHelper;
 
   beforeAll(() => {
-    mirroring = new Helper().mirroring;
-    pools = new Helper().pools;
+    mirroring = new MirroringPageHelper();
+    pools = new PoolPageHelper();
   });
 
   afterEach(async () => {
diff --git a/src/pybind/mgr/dashboard/frontend/e2e/cluster/alerts.e2e-spec.ts b/src/pybind/mgr/dashboard/frontend/e2e/cluster/alerts.e2e-spec.ts
index d706a3355b58..d20a3f4f3e6b 100644
--- a/src/pybind/mgr/dashboard/frontend/e2e/cluster/alerts.e2e-spec.ts
+++ b/src/pybind/mgr/dashboard/frontend/e2e/cluster/alerts.e2e-spec.ts
@@ -5,7 +5,7 @@ describe('Alerts page', () => {
   let alerts: AlertsPageHelper;
 
   beforeAll(() => {
-    alerts = new Helper().alerts;
+    alerts = new AlertsPageHelper();
   });
 
   afterEach(async () => {
diff --git a/src/pybind/mgr/dashboard/frontend/e2e/cluster/configuration.e2e-spec.ts b/src/pybind/mgr/dashboard/frontend/e2e/cluster/configuration.e2e-spec.ts
index 9c1c9ad51dca..e25ea4ac460c 100644
--- a/src/pybind/mgr/dashboard/frontend/e2e/cluster/configuration.e2e-spec.ts
+++ b/src/pybind/mgr/dashboard/frontend/e2e/cluster/configuration.e2e-spec.ts
@@ -5,7 +5,7 @@ describe('Configuration page', () => {
   let configuration: ConfigurationPageHelper;
 
   beforeAll(() => {
-    configuration = new Helper().configuration;
+    configuration = new ConfigurationPageHelper();
   });
 
   afterEach(async () => {
diff --git a/src/pybind/mgr/dashboard/frontend/e2e/cluster/crush-map.e2e-spec.ts b/src/pybind/mgr/dashboard/frontend/e2e/cluster/crush-map.e2e-spec.ts
index c995b6b96620..4df82e1d3428 100644
--- a/src/pybind/mgr/dashboard/frontend/e2e/cluster/crush-map.e2e-spec.ts
+++ b/src/pybind/mgr/dashboard/frontend/e2e/cluster/crush-map.e2e-spec.ts
@@ -6,7 +6,7 @@ describe('CRUSH map page', () => {
   let crushmap: CrushMapPageHelper;
 
   beforeAll(() => {
-    crushmap = new Helper().crushMap;
+    crushmap = new CrushMapPageHelper();
   });
 
   afterEach(async () => {
diff --git a/src/pybind/mgr/dashboard/frontend/e2e/cluster/hosts.e2e-spec.ts b/src/pybind/mgr/dashboard/frontend/e2e/cluster/hosts.e2e-spec.ts
index 2b80588419df..9619c7878b5d 100644
--- a/src/pybind/mgr/dashboard/frontend/e2e/cluster/hosts.e2e-spec.ts
+++ b/src/pybind/mgr/dashboard/frontend/e2e/cluster/hosts.e2e-spec.ts
@@ -5,7 +5,7 @@ describe('Hosts page', () => {
   let hosts: HostsPageHelper;
 
   beforeAll(() => {
-    hosts = new Helper().hosts;
+    hosts = new HostsPageHelper();
   });
 
   afterEach(async () => {
diff --git a/src/pybind/mgr/dashboard/frontend/e2e/cluster/logs.e2e-spec.ts b/src/pybind/mgr/dashboard/frontend/e2e/cluster/logs.e2e-spec.ts
index 54ef1bc98ac7..72aa8c895f09 100644
--- a/src/pybind/mgr/dashboard/frontend/e2e/cluster/logs.e2e-spec.ts
+++ b/src/pybind/mgr/dashboard/frontend/e2e/cluster/logs.e2e-spec.ts
@@ -18,9 +18,9 @@ describe('Logs page', () => {
   const minute = today.getMinutes();
 
   beforeAll(() => {
-    logs = new Helper().logs;
-    pools = new Helper().pools;
-    configuration = new Helper().configuration;
+    logs = new LogsPageHelper();
+    pools = new PoolPageHelper();
+    configuration = new ConfigurationPageHelper();
   });
 
   afterEach(async () => {
diff --git a/src/pybind/mgr/dashboard/frontend/e2e/cluster/mgr-modules.e2e-spec.ts b/src/pybind/mgr/dashboard/frontend/e2e/cluster/mgr-modules.e2e-spec.ts
index a6248c6e27e7..0a238d4e076b 100644
--- a/src/pybind/mgr/dashboard/frontend/e2e/cluster/mgr-modules.e2e-spec.ts
+++ b/src/pybind/mgr/dashboard/frontend/e2e/cluster/mgr-modules.e2e-spec.ts
@@ -5,7 +5,7 @@ describe('Manager modules page', () => {
   let mgrmodules: ManagerModulesPageHelper;
 
   beforeAll(() => {
-    mgrmodules = new Helper().mgrModules;
+    mgrmodules = new ManagerModulesPageHelper();
   });
 
   afterEach(async () => {
diff --git a/src/pybind/mgr/dashboard/frontend/e2e/cluster/monitors.e2e-spec.ts b/src/pybind/mgr/dashboard/frontend/e2e/cluster/monitors.e2e-spec.ts
index 8b91ab9c4245..122f923fd3e9 100644
--- a/src/pybind/mgr/dashboard/frontend/e2e/cluster/monitors.e2e-spec.ts
+++ b/src/pybind/mgr/dashboard/frontend/e2e/cluster/monitors.e2e-spec.ts
@@ -5,7 +5,7 @@ describe('Monitors page', () => {
   let monitors: MonitorsPageHelper;
 
   beforeAll(() => {
-    monitors = new Helper().monitors;
+    monitors = new MonitorsPageHelper();
   });
 
   afterEach(async () => {
diff --git a/src/pybind/mgr/dashboard/frontend/e2e/cluster/osds.e2e-spec.ts b/src/pybind/mgr/dashboard/frontend/e2e/cluster/osds.e2e-spec.ts
index 235412f54841..55c482a35efd 100644
--- a/src/pybind/mgr/dashboard/frontend/e2e/cluster/osds.e2e-spec.ts
+++ b/src/pybind/mgr/dashboard/frontend/e2e/cluster/osds.e2e-spec.ts
@@ -5,7 +5,7 @@ describe('OSDs page', () => {
   let osds: OSDsPageHelper;
 
   beforeAll(() => {
-    osds = new Helper().osds;
+    osds = new OSDsPageHelper();
   });
 
   afterEach(async () => {
diff --git a/src/pybind/mgr/dashboard/frontend/e2e/dashboard.e2e-spec.ts b/src/pybind/mgr/dashboard/frontend/e2e/dashboard.e2e-spec.ts
index df1d03abc74d..711be73a0ddb 100644
--- a/src/pybind/mgr/dashboard/frontend/e2e/dashboard.e2e-spec.ts
+++ b/src/pybind/mgr/dashboard/frontend/e2e/dashboard.e2e-spec.ts
@@ -19,14 +19,13 @@ describe('Dashboard Main Page', () => {
   let iscsi: IscsiPageHelper;
 
   beforeAll(() => {
-    const h = new Helper();
-    dashboard = h.dashboard;
-    daemons = h.daemons;
-    hosts = h.hosts;
-    osds = h.osds;
-    pools = h.pools;
-    monitors = h.monitors;
-    iscsi = h.iscsi;
+    dashboard = new DashboardPageHelper();
+    daemons = new DaemonsPageHelper();
+    hosts = new HostsPageHelper();
+    osds = new OSDsPageHelper();
+    pools = new PoolPageHelper();
+    monitors = new MonitorsPageHelper();
+    iscsi = new IscsiPageHelper();
   });
 
   afterEach(async () => {
diff --git a/src/pybind/mgr/dashboard/frontend/e2e/filesystems/filesystems.e2e-spec.ts b/src/pybind/mgr/dashboard/frontend/e2e/filesystems/filesystems.e2e-spec.ts
index dc339f6df78e..efbf9222ddb8 100644
--- a/src/pybind/mgr/dashboard/frontend/e2e/filesystems/filesystems.e2e-spec.ts
+++ b/src/pybind/mgr/dashboard/frontend/e2e/filesystems/filesystems.e2e-spec.ts
@@ -5,7 +5,7 @@ describe('Filesystems page', () => {
   let filesystems: FilesystemsPageHelper;
 
   beforeAll(() => {
-    filesystems = new Helper().filesystems;
+    filesystems = new FilesystemsPageHelper();
   });
 
   afterEach(async () => {
diff --git a/src/pybind/mgr/dashboard/frontend/e2e/helper.po.ts b/src/pybind/mgr/dashboard/frontend/e2e/helper.po.ts
index a27e57e394a4..64e0cd1c3c8f 100644
--- a/src/pybind/mgr/dashboard/frontend/e2e/helper.po.ts
+++ b/src/pybind/mgr/dashboard/frontend/e2e/helper.po.ts
@@ -26,7 +26,6 @@ export class Helper {
   pools: PoolPageHelper;
   buckets: BucketsPageHelper;
   images: ImagesPageHelper;
-  iscsi: IscsiPageHelper;
   mirroring: MirroringPageHelper;
   dashboard: DashboardPageHelper;
   usermgmt: UserMgmtPageHelper;
@@ -42,9 +41,7 @@ export class Helper {
   crushMap: CrushMapPageHelper;
   configuration: ConfigurationPageHelper;
   alerts: AlertsPageHelper;
-  mirroring: MirroringPageHelper;
   iscsi: IscsiPageHelper;
-  dashboard: DashboardPageHelper;
 
   constructor() {
     this.pools = new PoolPageHelper();
diff --git a/src/pybind/mgr/dashboard/frontend/e2e/nfs/nfs.e2e-spec.ts b/src/pybind/mgr/dashboard/frontend/e2e/nfs/nfs.e2e-spec.ts
index 00aaffa3bd8f..f53e30ff86d8 100644
--- a/src/pybind/mgr/dashboard/frontend/e2e/nfs/nfs.e2e-spec.ts
+++ b/src/pybind/mgr/dashboard/frontend/e2e/nfs/nfs.e2e-spec.ts
@@ -5,7 +5,7 @@ describe('Nfs page', () => {
   let nfs: NfsPageHelper;
 
   beforeAll(() => {
-    nfs = new Helper().nfs;
+    nfs = new NfsPageHelper();
   });
 
   afterEach(async () => {
diff --git a/src/pybind/mgr/dashboard/frontend/e2e/pools/pools.e2e-spec.ts b/src/pybind/mgr/dashboard/frontend/e2e/pools/pools.e2e-spec.ts
index 514b77c240d2..1ec08b2865a2 100644
--- a/src/pybind/mgr/dashboard/frontend/e2e/pools/pools.e2e-spec.ts
+++ b/src/pybind/mgr/dashboard/frontend/e2e/pools/pools.e2e-spec.ts
@@ -6,7 +6,7 @@ describe('Pools page', () => {
   const poolName = 'pool_e2e_pool_test';
 
   beforeAll(async () => {
-    pools = new Helper().pools;
+    pools = new PoolPageHelper();
     await pools.navigateTo();
   });
 
diff --git a/src/pybind/mgr/dashboard/frontend/e2e/rgw/buckets.e2e-spec.ts b/src/pybind/mgr/dashboard/frontend/e2e/rgw/buckets.e2e-spec.ts
index b8e6a6b47870..1631b645f905 100644
--- a/src/pybind/mgr/dashboard/frontend/e2e/rgw/buckets.e2e-spec.ts
+++ b/src/pybind/mgr/dashboard/frontend/e2e/rgw/buckets.e2e-spec.ts
@@ -6,8 +6,7 @@ describe('RGW buckets page', () => {
   let buckets: BucketsPageHelper;
 
   beforeAll(async () => {
-    const helper = new Helper();
-    buckets = helper.buckets;
+    buckets = new BucketsPageHelper();
   });
 
   afterEach(async () => {
diff --git a/src/pybind/mgr/dashboard/frontend/e2e/rgw/daemons.e2e-spec.ts b/src/pybind/mgr/dashboard/frontend/e2e/rgw/daemons.e2e-spec.ts
index d8ed69fe777f..ba8a2b46466a 100644
--- a/src/pybind/mgr/dashboard/frontend/e2e/rgw/daemons.e2e-spec.ts
+++ b/src/pybind/mgr/dashboard/frontend/e2e/rgw/daemons.e2e-spec.ts
@@ -5,7 +5,7 @@ describe('RGW daemons page', () => {
   let daemons: DaemonsPageHelper;
 
   beforeAll(() => {
-    daemons = new Helper().daemons;
+    daemons = new DaemonsPageHelper();
   });
 
   afterEach(async () => {
diff --git a/src/pybind/mgr/dashboard/frontend/e2e/rgw/users.e2e-spec.ts b/src/pybind/mgr/dashboard/frontend/e2e/rgw/users.e2e-spec.ts
index 72ce100e5a20..bf6cf71726e9 100644
--- a/src/pybind/mgr/dashboard/frontend/e2e/rgw/users.e2e-spec.ts
+++ b/src/pybind/mgr/dashboard/frontend/e2e/rgw/users.e2e-spec.ts
@@ -6,7 +6,7 @@ describe('RGW users page', () => {
   const user_name = '000user_create_edit_delete';
 
   beforeAll(() => {
-    users = new Helper().users;
+    users = new UsersPageHelper();
   });
 
   afterEach(async () => {