fix input specs

appsmithorg · Jan 7, 2025 · 0bb967c · 0bb967c
1 parent 79f2351
commit 0bb967c
Show file tree

Hide file tree

Showing 6 changed files with 12 additions and 7 deletions.
diff --git a/.../cypress/e2e/Regression/ClientSide/Anvil/Widgets/AnvilCurrencyInputWidgetSnapshot_spec.ts b/.../cypress/e2e/Regression/ClientSide/Anvil/Widgets/AnvilCurrencyInputWidgetSnapshot_spec.ts
@@ -5,7 +5,7 @@ import {
 } from "../../../../../support/Objects/ObjectsCore";
 
 // TODO: Enable when issue(github.com/appsmithorg/appsmith/issues/36419) is solved.
-describe.skip(
+describe(
   `${ANVIL_EDITOR_TEST}: Anvil tests for Currency Input Widget`,
   { tags: ["@tag.Anvil", "@tag.Visual"] },
   () => {

diff --git a/app/client/cypress/e2e/Regression/ClientSide/Anvil/Widgets/AnvilInputWidgetSnapshot_spec.ts b/app/client/cypress/e2e/Regression/ClientSide/Anvil/Widgets/AnvilInputWidgetSnapshot_spec.ts
@@ -5,7 +5,7 @@ import {
 } from "../../../../../support/Objects/ObjectsCore";
 
 // TODO: Enable when issue(github.com/appsmithorg/appsmith/issues/36419) is solved.
-describe.skip(
+describe(
   `${ANVIL_EDITOR_TEST}: Anvil tests for Input Widget`,
   { tags: ["@tag.Anvil", "@tag.Visual"] },
   () => {

diff --git a/...ent/cypress/e2e/Regression/ClientSide/Anvil/Widgets/AnvilPhoneInputWidgetSnapshot_spec.ts b/...ent/cypress/e2e/Regression/ClientSide/Anvil/Widgets/AnvilPhoneInputWidgetSnapshot_spec.ts
@@ -5,7 +5,7 @@ import {
 } from "../../../../../support/Objects/ObjectsCore";
 
 // TODO: Enable when issue(github.com/appsmithorg/appsmith/issues/36419) is solved.
-describe.skip(
+describe(
   `${ANVIL_EDITOR_TEST}: Anvil tests for Phone Input Widget`,
   { tags: ["@tag.Anvil", "@tag.Visual"] },
   () => {

diff --git a/app/client/cypress/support/Pages/Anvil/AnvilSnapshot.ts b/app/client/cypress/support/Pages/Anvil/AnvilSnapshot.ts
@@ -173,8 +173,8 @@ export class AnvilSnapshot {
 
   public triggerInputInvalidState = () => {
     this.enterPreviewMode();
-    cy.get("input[aria-required=true]").first().type("123");
-    cy.get("input[aria-required=true]").first().clear();
+    cy.get("input[required]").first().type("123");
+    cy.get("input[required]").first().clear();
     this.exitPreviewMode();
     this.agHelper.GetNClick(this.locators.propertyPaneSidebar);
   };

diff --git a/app/client/packages/design-system/widgets/src/components/FieldLabel/src/FieldLabel.tsx b/app/client/packages/design-system/widgets/src/components/FieldLabel/src/FieldLabel.tsx
@@ -22,7 +22,12 @@ export function FieldLabel(props: LabelProps) {
         className={clsx(styles.label)}
         elementType="label"
       >
-        <Text fontWeight={600} size="caption">
+        <Text
+          fontWeight={600}
+          lineClamp={1}
+          size="caption"
+          title={children?.toString()}
+        >
           {children}
         </Text>
         {Boolean(isRequired) && (

diff --git a/app/client/packages/design-system/widgets/src/components/FieldLabel/src/styles.module.css b/app/client/packages/design-system/widgets/src/components/FieldLabel/src/styles.module.css
@@ -2,7 +2,7 @@
   display: flex;
   align-items: center;
   gap: var(--inner-spacing-1);
-  min-height: var(--sizing-3);
+  height: var(--sizing-3);
   margin-block-end: var(--inner-spacing-3);
 }