Fix Deployment Scale Button "minus" (#1728)

Signed-off-by: vshakirova <vshakirova@mirantis.com>

src/renderer/components/+workloads-deployments/deployment-scale-dialog.test.tsx

@@ -128,33 +128,45 @@ describe("<DeploymentScaleDialog />", () => {
     const initReplicas = 1;
 
     deploymentApi.getReplicas = jest.fn().mockImplementationOnce(async () => initReplicas);
-    const { getByTestId } = render(<DeploymentScaleDialog />);
+    const component = render(<DeploymentScaleDialog />);
 
     DeploymentScaleDialog.open(dummyDeployment);
     await waitFor(async () => {
-      const desiredScale = await getByTestId("desired-scale");
+      expect(await component.getByTestId("desired-scale")).toHaveTextContent(`${initReplicas}`);
-
+      expect(await component.getByTestId("current-scale")).toHaveTextContent(`${initReplicas}`);
-      expect(desiredScale).toHaveTextContent(`${initReplicas}`);
+      expect((await component.baseElement.querySelector("input").value)).toBe(`${initReplicas}`);
     });
-    const up = await getByTestId("desired-replicas-up");
+    const up = await component.getByTestId("desired-replicas-up");
-    const down = await getByTestId("desired-replicas-down");
+    const down = await component.getByTestId("desired-replicas-down");
 
     fireEvent.click(up);
-    expect(await getByTestId("desired-scale")).toHaveTextContent(`${initReplicas + 1}`);
+    expect(await component.getByTestId("desired-scale")).toHaveTextContent(`${initReplicas + 1}`);
+    expect(await component.getByTestId("current-scale")).toHaveTextContent(`${initReplicas}`);
+    expect((await component.baseElement.querySelector("input").value)).toBe(`${initReplicas + 1}`);
+
     fireEvent.click(down);
-    expect(await getByTestId("desired-scale")).toHaveTextContent("1");
+    expect(await component.getByTestId("desired-scale")).toHaveTextContent(`${initReplicas}`);
-    // edge case, desiredScale must > 0
+    expect(await component.getByTestId("current-scale")).toHaveTextContent(`${initReplicas}`);
-    fireEvent.click(down);
+    expect((await component.baseElement.querySelector("input").value)).toBe(`${initReplicas}`);
-    fireEvent.click(down);
+
-    expect(await getByTestId("desired-scale")).toHaveTextContent("1");
+    // edge case, desiredScale must = 0
-    const times = 120;
+    let times = 10;
+
+    for (let i = 0; i < times; i++) {
+      fireEvent.click(down);
+    }
+    expect(await component.getByTestId("desired-scale")).toHaveTextContent("0");
+    expect((await component.baseElement.querySelector("input").value)).toBe("0");
+
+    // edge case, desiredScale must = 100 scaleMax (100)
+    times = 120;
 
-    // edge case, desiredScale must < scaleMax (100)
     for (let i = 0; i < times; i++) {
       fireEvent.click(up);
     }
-    expect(await getByTestId("desired-scale")).toHaveTextContent("100");
+    expect(await component.getByTestId("desired-scale")).toHaveTextContent("100");
+    expect((component.baseElement.querySelector("input").value)).toBe("100");
+    expect(await component.getByTestId("warning"))
+      .toHaveTextContent("High number of replicas may cause cluster performance issues");
   });
-
 });
-

src/renderer/components/+workloads-deployments/deployment-scale-dialog.tsx

@@ -91,7 +91,7 @@ export class DeploymentScaleDialog extends Component<Props> {
   };
   
   desiredReplicasDown = () => {
-    this.desiredReplicas > 1 && this.desiredReplicas--;
+    this.desiredReplicas > 0 && this.desiredReplicas--;
   };
 
   renderContents() {
@@ -124,7 +124,7 @@ export class DeploymentScaleDialog extends Component<Props> {
           </div>
         </div>
         {warning &&
-        <div className="warning">
+        <div className="warning" data-testid="warning">
           <Icon material="warning"/>
           <Trans>High number of replicas may cause cluster performance issues</Trans>
         </div>