From cc36f18fe8ca192ad97680d9c07c3d3b64a4b20e Mon Sep 17 00:00:00 2001
From: futa-ikeda <51409893+futa-ikeda@users.noreply.github.com>
Date: Thu, 19 Sep 2024 09:38:26 -0400
Subject: [PATCH] Comment out button to add new configured addon for a given
 provider (#2329)

-   Ticket: []
-   Feature flag: n/a

## Purpose
- Disallow configuring multiple accounts for a given external-storage-services

## Summary of Changes
- Comment out button that allows users to connect another account for a given external-storage-service
---
 .github/workflows/CI.yml                  | 10 +--
 app/guid-node/addons/index/template.hbs   |  5 +-
 tests/acceptance/guid-node/addons-test.ts | 75 ++++++++++++-----------
 3 files changed, 46 insertions(+), 44 deletions(-)

diff --git a/.github/workflows/CI.yml b/.github/workflows/CI.yml
index d8744ece95b..28ede2217e8 100644
--- a/.github/workflows/CI.yml
+++ b/.github/workflows/CI.yml
@@ -42,7 +42,7 @@ jobs:
         key: cached_node_modules_${{ secrets.CACHE_VERSION }}_${{ hashFiles('**/yarn.lock') }}
         restore-keys: cached_node_modules_${{ secrets.CACHE_VERSION }}_
     - run: yarn build:test
-    - uses: actions/upload-artifact@v2
+    - uses: actions/upload-artifact@v4
       with:
         name: dist
         path: ./dist
@@ -67,7 +67,7 @@ jobs:
     steps:
     - uses: actions/checkout@v2
     - uses: volta-cli/action@v1
-    - uses: actions/download-artifact@v2
+    - uses: actions/download-artifact@v4
       with:
         name: dist
         path: ./dist
@@ -91,7 +91,7 @@ jobs:
     steps:
     - uses: actions/checkout@v2
     - uses: volta-cli/action@v1
-    - uses: actions/download-artifact@v2
+    - uses: actions/download-artifact@v4
       with:
         name: dist
         path: ./dist
@@ -115,7 +115,7 @@ jobs:
     steps:
     - uses: actions/checkout@v2
     - uses: volta-cli/action@v1
-    - uses: actions/download-artifact@v2
+    - uses: actions/download-artifact@v4
       with:
         name: dist
         path: ./dist
@@ -139,7 +139,7 @@ jobs:
     steps:
     - uses: actions/checkout@v2
     - uses: volta-cli/action@v1
-    - uses: actions/download-artifact@v2
+    - uses: actions/download-artifact@v4
       with:
         name: dist
         path: ./dist
diff --git a/app/guid-node/addons/index/template.hbs b/app/guid-node/addons/index/template.hbs
index 6e65c146538..b4067be715e 100644
--- a/app/guid-node/addons/index/template.hbs
+++ b/app/guid-node/addons/index/template.hbs
@@ -207,7 +207,8 @@
                                 </div>
                             {{/each}}
                         </div>
-                        <div local-class='float-right'>
+                        {{!-- Remove ability to add a new configured addon for now --}}
+                        {{!-- <div local-class='float-right'>
                             <Button
                                 data-test-add-another-location-button
                                 data-analytics-name='Add another location'
@@ -219,7 +220,7 @@
                             >
                                 {{t 'addons.list.add-another-location'}}
                             </Button>
-                        </div>
+                        </div> --}}
                         <OsfDialog
                             @isOpen={{manager.confirmRemoveConnectedLocation}}
                             @onClose={{action (mut manager.confirmRemoveConnectedLocation) false}}
diff --git a/tests/acceptance/guid-node/addons-test.ts b/tests/acceptance/guid-node/addons-test.ts
index afa6c445619..4a7955a908b 100644
--- a/tests/acceptance/guid-node/addons-test.ts
+++ b/tests/acceptance/guid-node/addons-test.ts
@@ -1,4 +1,4 @@
-import { click as untrackedClick, fillIn } from '@ember/test-helpers';
+import { fillIn } from '@ember/test-helpers';
 import { setupMirage } from 'ember-cli-mirage/test-support';
 import { percySnapshot } from 'ember-percy';
 import { module, test } from 'qunit';
@@ -301,41 +301,42 @@ module('Acceptance | guid-node/addons', hooks => {
         await click('[data-test-addons-tab-connected-accounts]');
         assert.dom('[data-test-addon-card]').exists({ count: 2 }, '2 providers with configured accounts are present');
 
-        // Add new Box account from configured accounts list page
-        await click('[data-test-addon-card="Box"] [data-test-addon-card-configure]');
-        await click('[data-test-add-another-location-button]');
-        // Terms page
-        assert.dom('[data-test-addon-accept-terms-button]').exists('Terms shown first for new account');
-        await click('[data-test-addon-accept-terms-button]');
-        // New or existing account page
-        assert.dom('[data-test-addon-existing-account-button]')
-            .exists('Existing account button is present for a provider with authorized account');
-        assert.dom('[data-test-addon-new-account-button]').exists('New account button is present');
-        await click('[data-test-addon-existing-account-button]');
-        // Existing account page
-        assert.dom('[data-test-existing-authorized-accounts-input]')
-            .exists({ count: 1 }, 'Options for choosing existing authorized account present');
-        assert.dom('[data-test-addon-authorize-button]')
-            .doesNotExist('Authorize button is not present before choosing an account');
-        await untrackedClick('[data-test-existing-authorized-accounts-input]');
-        assert.dom('[data-test-addon-authorize-button]')
-            .exists('Authorize button is present after choosing an account');
-        await percySnapshot('Acceptance | guid-node/addons | Adding new configured addons | existing account page');
-        await click('[data-test-addon-authorize-button]');
-        // Confirm setup page
-        assert.dom('[data-test-addon-confirm-setup-button]').exists('Confirm setup button is present');
-        await percySnapshot('Acceptance | guid-node/addons | Adding new configured addons | confirm setup page');
-        await click('[data-test-addon-confirm-setup-button]');
-        // Configure page
-        assert.dom('[data-test-display-name-input]').exists('Name input is present');
-        assert.dom('[data-test-root-folder-save]').isDisabled('Save button is disabled');
-        await fillIn('[data-test-display-name-input]', 'New Box Account Display Name');
-        await click('[data-test-root-folder-option]:first-child');
-        await click('[data-test-root-folder-save]');
-
-        // check to see if new account is added
-        await click('[data-test-addon-card="Box"] [data-test-addon-card-configure]');
-        assert.dom('[data-test-edit-connected-location]').exists({ count: 2 }, 'Two editable accounts are present');
-        assert.dom('[data-test-remove-connected-location]').exists({ count: 2 }, 'Two remove buttons are present');
+        // Skip adding new account, as this is no longer part of the requirements
+        // // Add new Box account from configured accounts list page
+        // await click('[data-test-addon-card="Box"] [data-test-addon-card-configure]');
+        // await click('[data-test-add-another-location-button]');
+        // // Terms page
+        // assert.dom('[data-test-addon-accept-terms-button]').exists('Terms shown first for new account');
+        // await click('[data-test-addon-accept-terms-button]');
+        // // New or existing account page
+        // assert.dom('[data-test-addon-existing-account-button]')
+        //     .exists('Existing account button is present for a provider with authorized account');
+        // assert.dom('[data-test-addon-new-account-button]').exists('New account button is present');
+        // await click('[data-test-addon-existing-account-button]');
+        // // Existing account page
+        // assert.dom('[data-test-existing-authorized-accounts-input]')
+        //     .exists({ count: 1 }, 'Options for choosing existing authorized account present');
+        // assert.dom('[data-test-addon-authorize-button]')
+        //     .doesNotExist('Authorize button is not present before choosing an account');
+        // await untrackedClick('[data-test-existing-authorized-accounts-input]');
+        // assert.dom('[data-test-addon-authorize-button]')
+        //     .exists('Authorize button is present after choosing an account');
+        // await percySnapshot('Acceptance | guid-node/addons | Adding new configured addons | existing account page');
+        // await click('[data-test-addon-authorize-button]');
+        // // Confirm setup page
+        // assert.dom('[data-test-addon-confirm-setup-button]').exists('Confirm setup button is present');
+        // await percySnapshot('Acceptance | guid-node/addons | Adding new configured addons | confirm setup page');
+        // await click('[data-test-addon-confirm-setup-button]');
+        // // Configure page
+        // assert.dom('[data-test-display-name-input]').exists('Name input is present');
+        // assert.dom('[data-test-root-folder-save]').isDisabled('Save button is disabled');
+        // await fillIn('[data-test-display-name-input]', 'New Box Account Display Name');
+        // await click('[data-test-root-folder-option]:first-child');
+        // await click('[data-test-root-folder-save]');
+
+        // // check to see if new account is added
+        // await click('[data-test-addon-card="Box"] [data-test-addon-card-configure]');
+        // assert.dom('[data-test-edit-connected-location]').exists({ count: 2 }, 'Two editable accounts are present');
+        // assert.dom('[data-test-remove-connected-location]').exists({ count: 2 }, 'Two remove buttons are present');
     });
 });