[7.x] Changed Upgrade Assistant test to reflect hiding the UI (#96714)

x-pack/test/functional/apps/upgrade_assistant/upgrade_assistant.ts

@@ -16,21 +16,35 @@ export default function upgradeAssistantFunctionalTests({
   const PageObjects = getPageObjects(['upgradeAssistant', 'common']);
   const log = getService('log');
   const retry = getService('retry');
+  const security = getService('security');
+  const testSubjects = getService('testSubjects');
+
+  // Updated for the hiding of the UA UI.
+  describe('Upgrade Checkup', function () {
+    this.tags('skipFirefox');
+
+    before(async () => {
+      await esArchiver.load('empty_kibana');
+      await security.testUser.setRoles(['global_upgrade_assistant_role']);
+    });
 
-  // Failing: See https://github.com/elastic/kibana/issues/86546
-  describe.skip('Upgrade Checkup', function () {
-    this.tags('includeFirefox');
-    before(async () => await esArchiver.load('empty_kibana'));
     after(async () => {
       await PageObjects.upgradeAssistant.waitForTelemetryHidden();
       await esArchiver.unload('empty_kibana');
     });
 
-    it('allows user to navigate to upgrade checkup', async () => {
+    it('Overview page', async () => {
+      await PageObjects.upgradeAssistant.navigateToPage();
+      await retry.waitFor('Upgrade Assistant overview page to be visible', async () => {
+        return testSubjects.exists('comingSoonPrompt');
+      });
+    });
+
+    it.skip('allows user to navigate to upgrade checkup', async () => {
       await PageObjects.upgradeAssistant.navigateToPage();
     });
 
-    it('allows user to toggle deprecation logging', async () => {
+    it.skip('allows user to toggle deprecation logging', async () => {
       log.debug('expect initial state to be ON');
       expect(await PageObjects.upgradeAssistant.deprecationLoggingEnabledLabel()).to.be('On');
       expect(await PageObjects.upgradeAssistant.isDeprecationLoggingEnabled()).to.be(true);
@@ -53,15 +67,15 @@ export default function upgradeAssistantFunctionalTests({
       });
     });
 
-    it('allows user to open cluster tab', async () => {
+    it.skip('allows user to open cluster tab', async () => {
       await PageObjects.upgradeAssistant.navigateToPage();
       await PageObjects.upgradeAssistant.clickTab('cluster');
       expect(await PageObjects.upgradeAssistant.issueSummaryText()).to.be(
         'You have no cluster issues.'
       );
     });
 
-    it('allows user to open indices tab', async () => {
+    it.skip('allows user to open indices tab', async () => {
       await PageObjects.upgradeAssistant.navigateToPage();
       await PageObjects.upgradeAssistant.clickTab('indices');
       expect(await PageObjects.upgradeAssistant.issueSummaryText()).to.be(

x-pack/test/functional/config.js

@@ -42,7 +42,6 @@ export default async function ({ readConfigFile }) {
       resolve(__dirname, './apps/maps'),
       resolve(__dirname, './apps/status_page'),
       resolve(__dirname, './apps/timelion'),
-      resolve(__dirname, './apps/upgrade_assistant'),
       resolve(__dirname, './apps/visualize'),
       resolve(__dirname, './apps/uptime'),
       resolve(__dirname, './apps/saved_objects_management'),
@@ -61,6 +60,10 @@ export default async function ({ readConfigFile }) {
       resolve(__dirname, './apps/management'),
       resolve(__dirname, './apps/reporting'),
 
+      //This upgrade assistant file needs to be run after the rest of the jobs because of
+      //it being destructive.
+      resolve(__dirname, './apps/upgrade_assistant'),
+
       // This license_management file must be last because it is destructive.
       resolve(__dirname, './apps/license_management'),
     ],