Reduce code conflicts (#589) (#590)

* Item count in Content panel Signed-off-by: bowenlan-amzn <bowenlan23@gmail.com> * Constants Signed-off-by: bowenlan-amzn <bowenlan23@gmail.com> * Datastream security message Signed-off-by: bowenlan-amzn <bowenlan23@gmail.com> * Unify the refresh Signed-off-by: bowenlan-amzn <bowenlan23@gmail.com> * Update test snapshot Signed-off-by: bowenlan-amzn <bowenlan23@gmail.com> * Update cypress test Signed-off-by: bowenlan-amzn <bowenlan23@gmail.com> Signed-off-by: bowenlan-amzn <bowenlan23@gmail.com> (cherry picked from commit 74c0da1) Co-authored-by: bowenlan-amzn <bowenlan23@gmail.com>
opensearch-project · Jan 25, 2023 · 307c986 · 307c986
1 parent 8061fca
commit 307c986
Show file tree

Hide file tree

Showing 44 changed files with 366 additions and 613 deletions.
diff --git a/cypress/integration/managed_indices_spec.js b/cypress/integration/managed_indices_spec.js
@@ -217,7 +217,7 @@ describe("Managed indices", () => {
       cy.contains("Changed policy on 1 indices");
 
       // Click back to Managed Indices page by clicking "Managed indices" breadcrumb
-      cy.contains("Managed indices").click();
+      cy.contains("Policy managed indices").click();
 
       // Speed up execution of managed index
       cy.updateManagedIndexConfigStartTime(SAMPLE_INDEX);

diff --git a/cypress/integration/policies_spec.js b/cypress/integration/policies_spec.js
@@ -39,7 +39,7 @@ describe("Policies", () => {
       cy.contains("Continue").click({ force: true });
 
       // Wait for input to load and then type in the policy ID
-      cy.get(`input[placeholder="hot_cold_workflow"]`).type(POLICY_ID, { force: true });
+      cy.get(`input[placeholder="example_policy"]`).type(POLICY_ID, { force: true });
 
       // Wait for default policy JSON to load
       cy.contains("A simple default policy");
@@ -188,5 +188,5 @@ describe("Policies", () => {
       cy.get(`[data-test-subj="viewButton"]`).click({ force: true });
       cy.contains(`View JSON of ${POLICY_ID}`);
     });
-  })
+  });
 });
diff --git a/public/components/ContentPanel/ContentPanel.tsx b/public/components/ContentPanel/ContentPanel.tsx
@@ -15,6 +15,7 @@ interface ContentPanelProps {
   horizontalRuleClassName?: string;
   actions?: React.ReactNode | React.ReactNode[];
   children: React.ReactNode | React.ReactNode[];
+  itemCount?: number;
 }
 
 const renderSubTitleText = (subTitleText: string | JSX.Element): JSX.Element | null => {
@@ -38,13 +39,17 @@ const ContentPanel: React.SFC<ContentPanelProps> = ({
   horizontalRuleClassName = "",
   actions,
   children,
+  itemCount = 0,
 }) => (
   <EuiPanel style={{ paddingLeft: "0px", paddingRight: "0px", ...panelStyles }}>
     <EuiFlexGroup style={{ padding: "0px 10px" }} justifyContent="spaceBetween" alignItems="flexStart">
       <EuiFlexItem>
         {typeof title === "string" ? (
           <EuiTitle size={titleSize}>
-            <h3>{title}</h3>
+            <h3>
+              {title}
+              <span className="panel-header-count"> {itemCount > 0 ? `(${itemCount})` : null} </span>
+            </h3>
           </EuiTitle>
         ) : (
           title

diff --git a/public/components/ContentPanel/__snapshots__/ContentPanel.test.tsx.snap b/public/components/ContentPanel/__snapshots__/ContentPanel.test.tsx.snap
@@ -16,6 +16,12 @@ exports[`<ContentPanel /> spec renders the component 1`] = `
         class="euiTitle euiTitle--large"
       >
         Testing
+        <span
+          class="panel-header-count"
+        >
+           
+           
+        </span>
       </h3>
     </div>
     <div

diff --git a/public/containers/IndexDetail/__snapshots__/IndexDetail.test.tsx.snap b/public/containers/IndexDetail/__snapshots__/IndexDetail.test.tsx.snap
@@ -17,6 +17,12 @@ exports[`<IndexDetail /> spec render the component 1`] = `
           class="euiTitle euiTitle--small"
         >
           Source index details
+          <span
+            class="panel-header-count"
+          >
+             
+             
+          </span>
         </h3>
       </div>
     </div>

diff --git a/...gePolicy/components/ChangeManagedIndices/__snapshots__/ChangeManagedIndices.test.tsx.snap b/...gePolicy/components/ChangeManagedIndices/__snapshots__/ChangeManagedIndices.test.tsx.snap
@@ -16,6 +16,12 @@ exports[`<ChangeManagedIndices /> spec renders the component 1`] = `
         class="euiTitle euiTitle--small"
       >
         Choose managed indices
+        <span
+          class="panel-header-count"
+        >
+           
+           
+        </span>
       </h3>
     </div>
   </div>

diff --git a/public/pages/ChangePolicy/components/NewPolicy/__snapshots__/NewPolicy.test.tsx.snap b/public/pages/ChangePolicy/components/NewPolicy/__snapshots__/NewPolicy.test.tsx.snap
@@ -16,6 +16,12 @@ exports[`<NewPolicy /> spec renders the component 1`] = `
         class="euiTitle euiTitle--small"
       >
         Choose new policy
+        <span
+          class="panel-header-count"
+        >
+           
+           
+        </span>
       </h3>
     </div>
   </div>

diff --git a/public/pages/ChangePolicy/containers/ChangePolicy/__snapshots__/ChangePolicy.test.tsx.snap b/public/pages/ChangePolicy/containers/ChangePolicy/__snapshots__/ChangePolicy.test.tsx.snap
@@ -27,6 +27,12 @@ exports[`<ChangePolicy /> spec renders the component 1`] = `
           class="euiTitle euiTitle--small"
         >
           Choose managed indices
+          <span
+            class="panel-header-count"
+          >
+             
+             
+          </span>
         </h3>
       </div>
     </div>
@@ -221,6 +227,12 @@ exports[`<ChangePolicy /> spec renders the component 1`] = `
           class="euiTitle euiTitle--small"
         >
           Choose new policy
+          <span
+            class="panel-header-count"
+          >
+             
+             
+          </span>
         </h3>
       </div>
     </div>

diff --git a/public/pages/CreateIndex/components/IndexDetail/__snapshots__/IndexDetail.test.tsx.snap b/public/pages/CreateIndex/components/IndexDetail/__snapshots__/IndexDetail.test.tsx.snap
@@ -16,6 +16,12 @@ exports[`<IndexDetail /> spec renders the component 1`] = `
         class="euiTitle euiTitle--small"
       >
         Define index
+        <span
+          class="panel-header-count"
+        >
+           
+           
+        </span>
       </h3>
     </div>
   </div>

diff --git a/...exTemplate/containers/CreateIndexTemplate/__snapshots__/CreateIndexTemplate.test.tsx.snap b/...exTemplate/containers/CreateIndexTemplate/__snapshots__/CreateIndexTemplate.test.tsx.snap
@@ -87,6 +87,12 @@ exports[`<CreateIndexTemplate /> spec it goes to templates page when click cance
             class="euiTitle euiTitle--small"
           >
             Template details
+            <span
+              class="panel-header-count"
+            >
+               
+               
+            </span>
           </h3>
         </div>
       </div>
@@ -273,6 +279,12 @@ exports[`<CreateIndexTemplate /> spec it goes to templates page when click cance
             class="euiTitle euiTitle--small"
           >
             Index settings
+            <span
+              class="panel-header-count"
+            >
+               
+               
+            </span>
           </h3>
         </div>
       </div>

diff --git a/.../CreateIndexTemplate/containers/TemplateDetail/__snapshots__/TemplateDetail.test.tsx.snap b/.../CreateIndexTemplate/containers/TemplateDetail/__snapshots__/TemplateDetail.test.tsx.snap
@@ -66,6 +66,12 @@ exports[`<TemplateDetail /> spec render component 1`] = `
           class="euiTitle euiTitle--small"
         >
           Define template
+          <span
+            class="panel-header-count"
+          >
+             
+             
+          </span>
         </h3>
       </div>
     </div>
@@ -464,6 +470,12 @@ exports[`<TemplateDetail /> spec render component 1`] = `
           class="euiTitle euiTitle--small"
         >
           Index settings
+          <span
+            class="panel-header-count"
+          >
+             
+             
+          </span>
         </h3>
       </div>
     </div>

diff --git a/public/pages/CreatePolicy/components/ConfigurePolicy/ConfigurePolicy.tsx b/public/pages/CreatePolicy/components/ConfigurePolicy/ConfigurePolicy.tsx
@@ -27,7 +27,7 @@ const ConfigurePolicy = ({ isEdit, policyId, policyIdError, onChange }: Configur
         isInvalid={!!policyIdError}
         error={policyIdError}
       >
-        <EuiFieldText isInvalid={!!policyIdError} placeholder="hot_cold_workflow" readOnly={isEdit} value={policyId} onChange={onChange} />
+        <EuiFieldText isInvalid={!!policyIdError} placeholder="example_policy" readOnly={isEdit} value={policyId} onChange={onChange} />
       </EuiFormRow>
     </div>
   </ContentPanel>

diff --git a/...pages/CreatePolicy/components/ConfigurePolicy/__snapshots__/ConfigurePolicy.test.tsx.snap b/...pages/CreatePolicy/components/ConfigurePolicy/__snapshots__/ConfigurePolicy.test.tsx.snap
@@ -16,6 +16,12 @@ exports[`<ConfigurePolicy /> spec renders the component 1`] = `
         class="euiTitle euiTitle--small"
       >
         Name policy
+        <span
+          class="panel-header-count"
+        >
+           
+           
+        </span>
       </h3>
     </div>
   </div>
@@ -66,7 +72,7 @@ exports[`<ConfigurePolicy /> spec renders the component 1`] = `
                 aria-describedby="some_html_id-help-0"
                 class="euiFieldText"
                 id="some_html_id"
-                placeholder="hot_cold_workflow"
+                placeholder="example_policy"
                 type="text"
                 value="some_id"
               />

diff --git a/public/pages/CreatePolicy/components/DefinePolicy/__snapshots__/DefinePolicy.test.tsx.snap b/public/pages/CreatePolicy/components/DefinePolicy/__snapshots__/DefinePolicy.test.tsx.snap
@@ -16,6 +16,12 @@ exports[`<DefinePolicy /> spec renders the component 1`] = `
         class="euiTitle euiTitle--small"
       >
         Define policy
+        <span
+          class="panel-header-count"
+        >
+           
+           
+        </span>
       </h3>
     </div>
     <div

diff --git a/public/pages/CreatePolicy/containers/CreatePolicy/CreatePolicy.test.tsx b/public/pages/CreatePolicy/containers/CreatePolicy/CreatePolicy.test.tsx
@@ -115,7 +115,7 @@ describe("<CreatePolicy /> spec", () => {
 
     await waitFor(() => getByDisplayValue("some_id"));
 
-    expect(getByPlaceholderText("hot_cold_workflow")).toHaveAttribute("readonly");
+    expect(getByPlaceholderText("example_policy")).toHaveAttribute("readonly");
   });
 
   it("shows error for policyId input when clicking create", async () => {
@@ -127,9 +127,9 @@ describe("<CreatePolicy /> spec", () => {
 
     expect(queryByText("Required")).not.toBeNull();
 
-    fireEvent.focus(getByPlaceholderText("hot_cold_workflow"));
-    userEvent.type(getByPlaceholderText("hot_cold_workflow"), `some_policy_id`);
-    fireEvent.blur(getByPlaceholderText("hot_cold_workflow"));
+    fireEvent.focus(getByPlaceholderText("example_policy"));
+    userEvent.type(getByPlaceholderText("example_policy"), `some_policy_id`);
+    fireEvent.blur(getByPlaceholderText("example_policy"));
 
     expect(queryByText("Required")).toBeNull();
   });
@@ -138,9 +138,9 @@ describe("<CreatePolicy /> spec", () => {
     browserServicesMock.policyService.putPolicy = jest.fn().mockResolvedValue({ ok: true, response: { _id: "some_policy_id" } });
     const { getByText, getByTestId, getByPlaceholderText } = renderCreatePolicyWithRouter();
 
-    fireEvent.focus(getByPlaceholderText("hot_cold_workflow"));
-    userEvent.type(getByPlaceholderText("hot_cold_workflow"), `some_policy_id`);
-    fireEvent.blur(getByPlaceholderText("hot_cold_workflow"));
+    fireEvent.focus(getByPlaceholderText("example_policy"));
+    userEvent.type(getByPlaceholderText("example_policy"), `some_policy_id`);
+    fireEvent.blur(getByPlaceholderText("example_policy"));
 
     userEvent.click(getByTestId("createPolicyCreateButton"));
 
@@ -152,9 +152,9 @@ describe("<CreatePolicy /> spec", () => {
     browserServicesMock.policyService.putPolicy = jest.fn().mockResolvedValue({ ok: false, error: "bad policy" });
     const { getByText, getByTestId, getByPlaceholderText } = renderCreatePolicyWithRouter();
 
-    fireEvent.focus(getByPlaceholderText("hot_cold_workflow"));
-    userEvent.type(getByPlaceholderText("hot_cold_workflow"), `some_policy_id`);
-    fireEvent.blur(getByPlaceholderText("hot_cold_workflow"));
+    fireEvent.focus(getByPlaceholderText("example_policy"));
+    userEvent.type(getByPlaceholderText("example_policy"), `some_policy_id`);
+    fireEvent.blur(getByPlaceholderText("example_policy"));
 
     userEvent.click(getByTestId("createPolicyCreateButton"));
 
@@ -165,9 +165,9 @@ describe("<CreatePolicy /> spec", () => {
     browserServicesMock.policyService.putPolicy = jest.fn().mockRejectedValue(new Error("this is an error"));
     const { getByText, getByTestId, getByPlaceholderText } = renderCreatePolicyWithRouter();
 
-    fireEvent.focus(getByPlaceholderText("hot_cold_workflow"));
-    userEvent.type(getByPlaceholderText("hot_cold_workflow"), `some_policy_id`);
-    fireEvent.blur(getByPlaceholderText("hot_cold_workflow"));
+    fireEvent.focus(getByPlaceholderText("example_policy"));
+    userEvent.type(getByPlaceholderText("example_policy"), `some_policy_id`);
+    fireEvent.blur(getByPlaceholderText("example_policy"));
 
     userEvent.click(getByTestId("createPolicyCreateButton"));