From 51a2a10cfc2ab339320ba2c1137723b933d9f51c Mon Sep 17 00:00:00 2001
From: Jeff Daley <jeffdaley@users.noreply.github.com>
Date: Thu, 11 Apr 2024 11:44:05 -0400
Subject: [PATCH] Show group approvers in "Request review" modal (#672)

* Update sidebar.hbs

* Update document-test.ts

* Cleanup

* Update document-test.ts
---
 web/app/components/document/sidebar.hbs             | 2 +-
 web/tests/acceptance/authenticated/document-test.ts | 9 ++++++++-
 2 files changed, 9 insertions(+), 2 deletions(-)

diff --git a/web/app/components/document/sidebar.hbs b/web/app/components/document/sidebar.hbs
index 6103f6657..21569e3a2 100644
--- a/web/app/components/document/sidebar.hbs
+++ b/web/app/components/document/sidebar.hbs
@@ -769,7 +769,7 @@
               <Inputs::PeopleSelect
                 @includeGroups={{true}}
                 @renderInPlace={{true}}
-                @selected={{this.approvers}}
+                @selected={{this.allApprovers}}
                 @onChange={{this.updateApprovers}}
                 @disabled={{M.taskIsRunning}}
                 class="mb-0"
diff --git a/web/tests/acceptance/authenticated/document-test.ts b/web/tests/acceptance/authenticated/document-test.ts
index 6579cdae7..af5f34353 100644
--- a/web/tests/acceptance/authenticated/document-test.ts
+++ b/web/tests/acceptance/authenticated/document-test.ts
@@ -74,6 +74,7 @@ const TRANSFER_OWNERSHIP_MODAL = "[data-test-transfer-ownership-modal]";
 const OWNERSHIP_TRANSFERRED_MODAL = "[data-test-ownership-transferred-modal]";
 const TRANSFERRING_DOC = "[data-test-transferring-doc]";
 const SELECT_NEW_OWNER_LABEL = "[data-test-select-new-owner-label]";
+const MODAL_PEOPLE_SELECT = "dialog [data-test-people-select]";
 const PEOPLE_SELECT_INPUT = ".ember-power-select-trigger-multiple-input";
 const PEOPLE_SELECT_OPTION =
   ".ember-power-select-option:not(.ember-power-select-option--no-matches-message)";
@@ -776,7 +777,7 @@ module("Acceptance | authenticated/document", function (hooks) {
     assert.dom(PUBLISH_FOR_REVIEW_MODAL_SELECTOR).exists();
 
     // Add an approver
-    await click("dialog [data-test-people-select]");
+    await click(MODAL_PEOPLE_SELECT);
 
     await fillIn(
       ".ember-power-select-trigger-multiple-input",
@@ -1806,5 +1807,11 @@ module("Acceptance | authenticated/document", function (hooks) {
     const doc = this.server.schema.document.find(1).attrs;
 
     assert.true(doc.approverGroups.includes(email));
+
+    await click(SIDEBAR_PUBLISH_FOR_REVIEW_BUTTON_SELECTOR);
+
+    assert
+      .dom(MODAL_PEOPLE_SELECT)
+      .containsText(name, "the modal PeopleSelect includes groups");
   });
 });