Skip to content

Commit 2b1ebcd

Browse files
committed
fix(input): test with input number type letter
Signed-off-by: Alexandre Esteves <[email protected]>
1 parent ebaf0b7 commit 2b1ebcd

File tree

4 files changed

+7
-14
lines changed

4 files changed

+7
-14
lines changed

packages/ods/src/components/datepicker/src/components/ods-datepicker/ods-datepicker.tsx

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -331,7 +331,7 @@ export class OdsDatepicker implements OdsFormElement {
331331
}
332332

333333
private onInput(event: InputEvent): void {
334-
this.showClearable = !(event.target as HTMLInputElement).value;
334+
this.showClearable = !!(event.target as HTMLInputElement).value;
335335
}
336336

337337
render(): FunctionalComponent {

packages/ods/src/components/datepicker/tests/rendering/ods-datepicker.e2e.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -177,7 +177,7 @@ describe('ods-datepicker rendering', () => {
177177
});
178178

179179
it('should render clearable button when type wrong format', async() => {
180-
await setup('<ods-datepicker is-clearable ></ods-datepicker>');
180+
await setup('<ods-datepicker is-clearable></ods-datepicker>');
181181

182182
await el.type('clearable');
183183
await page.waitForChanges();

packages/ods/src/components/input/src/components/ods-input/ods-input.tsx

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -260,13 +260,6 @@ export class OdsInput implements OdsFormElement {
260260
this.shouldUpdateIsInvalidState = false;
261261
}
262262

263-
// console.log('this.internals.validity', this.internals.validity)
264-
// if (!this.internals.validity?.valid && !this.internals.validity?.valueMissing) {
265-
// this.showClearable = true;
266-
// } else {
267-
// this.showClearable = false;
268-
// }
269-
270263
this.odsChange.emit({
271264
name: this.name,
272265
previousValue: typeof this.value === 'number' && isNumeric(previousValue) ? Number(previousValue) : previousValue,

packages/ods/src/components/input/tests/rendering/ods-input.e2e.ts

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -268,11 +268,11 @@ describe('ods-input rendering', () => {
268268
expect(buttonClearable).toBeNull();
269269

270270
// Test E2E launched with chrome. Chrome don't allow to type on input type number
271-
// await page.keyboard.press('q');
272-
// await page.waitForChanges();
273-
// await new Promise(resolve => setTimeout(resolve, 20000));
274-
// buttonClearable = await page.find('ods-input >>> ods-button[icon="xmark"]');
275-
// expect(buttonClearable).not.toBeNull();
271+
await page.keyboard.press('e');
272+
await page.waitForChanges();
273+
274+
buttonClearable = await page.find('ods-input >>> ods-button[icon="xmark"]');
275+
expect(buttonClearable).not.toBeNull();
276276
});
277277
});
278278

0 commit comments

Comments
 (0)