[GitHub] [trafficcontrol] dpham692 commented on a change in pull request #6071: Migrate Profiles test from E2E to integrate test suite TP

[GitBox <gi...@apache.org>]

dpham692 commented on a change in pull request #6071:
URL: https://github.com/apache/trafficcontrol/pull/6071#discussion_r688631833



##########
File path: traffic_portal/test/integration/specs/Profiles.spec.ts
##########
@@ -46,6 +46,24 @@ profiles.tests.forEach(async profilesData => {
                 await profilesPage.OpenConfigureMenu();
                 await profilesPage.OpenProfilesPage();
             });
+            profilesData.check.forEach(check => {
+                it(check.description, async () => {
+                    expect(await profilesPage.CheckCSV(check.Name)).toBe(true);
+                    await profilesPage.OpenProfilesPage();
+                });
+            });
+            profilesData.toggle.forEach(toggle => {
+                it(toggle.description, async () => {
+                    if(toggle.description.includes('hide')){
+                        expect(await profilesPage.ToggleTableColumn(toggle.Name)).toBe(false);
+                        await profilesPage.OpenProfilesPage();
+                    }else{
+                        expect(await profilesPage.ToggleTableColumn(toggle.Name)).toBe(true);
+                        await profilesPage.OpenProfilesPage();

Review comment:
       I do the reopening Profiles page after each test because if the tests fail it will go back to the Profiles Page and continue with other tests.




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@trafficcontrol.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org