From 718f0be23c88a7a00ee9d983b774f7b838d61d25 Mon Sep 17 00:00:00 2001 From: Shivangi917 Date: Wed, 23 Jul 2025 03:00:53 -0400 Subject: [PATCH] Add whitespace error and fix unit test for checkbox --- projects/aca-content/assets/i18n/en.json | 3 +- .../no-whitespace.validator.spec.ts | 50 +++++++++ .../validators/no-whitespace.validator.ts | 37 +++++++ .../search-input-control.component.html | 1 - .../search-input-control.component.spec.ts | 8 +- .../search-input-control.component.ts | 11 +- .../search-input/search-input.component.html | 18 +-- .../search-input/search-input.component.scss | 1 - .../search-input.component.spec.ts | 104 +++++++++++------- .../search-input/search-input.component.ts | 12 +- 10 files changed, 180 insertions(+), 65 deletions(-) create mode 100644 projects/aca-content/folder-rules/src/rule-details/validators/no-whitespace.validator.spec.ts create mode 100644 projects/aca-content/folder-rules/src/rule-details/validators/no-whitespace.validator.ts diff --git a/projects/aca-content/assets/i18n/en.json b/projects/aca-content/assets/i18n/en.json index 12f368ab4..92c10df21 100644 --- a/projects/aca-content/assets/i18n/en.json +++ b/projects/aca-content/assets/i18n/en.json @@ -575,7 +575,8 @@ "FOLDERS": "Folders", "LIBRARIES": "Libraries", "HINT": "Search input must have at least 2 alphanumeric characters.", - "REQUIRED": "Search term is required." + "REQUIRED": "Search input is required.", + "WHITESPACE": "Search input cannot be only whitespace." }, "SORT": { "SORTING_OPTION": "Sort by", diff --git a/projects/aca-content/folder-rules/src/rule-details/validators/no-whitespace.validator.spec.ts b/projects/aca-content/folder-rules/src/rule-details/validators/no-whitespace.validator.spec.ts new file mode 100644 index 000000000..4454834f2 --- /dev/null +++ b/projects/aca-content/folder-rules/src/rule-details/validators/no-whitespace.validator.spec.ts @@ -0,0 +1,50 @@ +/*! + * Copyright © 2005-2025 Hyland Software, Inc. and its affiliates. All rights reserved. + * + * Alfresco Example Content Application + * + * This file is part of the Alfresco Example Content Application. + * If the software was purchased under a paid Alfresco license, the terms of + * the paid license agreement will prevail. Otherwise, the software is + * provided under the following open source license terms: + * + * The Alfresco Example Content Application is free software: you can redistribute it and/or modify + * it under the terms of the GNU Lesser General Public License as published by + * the Free Software Foundation, either version 3 of the License, or + * (at your option) any later version. + * + * The Alfresco Example Content Application is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU Lesser General Public License for more details. + * + * You should have received a copy of the GNU Lesser General Public License + * from Hyland Software. If not, see . + */ + +import { FormControl } from '@angular/forms'; +import { noWhitespaceValidator } from './no-whitespace.validator'; + +describe('noWhitespaceValidator', () => { + const validatorFn = noWhitespaceValidator(); + + it('should return null for valid non-whitespace input', () => { + const control = new FormControl('valid input'); + expect(validatorFn(control)).toBeNull(); + }); + + it('should return error for input with only spaces', () => { + const control = new FormControl(' '); + expect(validatorFn(control)).toEqual({ whitespace: true }); + }); + + it('should return error for empty string', () => { + const control = new FormControl(''); + expect(validatorFn(control)).toBeNull(); + }); + + it('should return null for input with leading and trailing spaces but valid content inside', () => { + const control = new FormControl(' valid '); + expect(validatorFn(control)).toBeNull(); + }); +}); diff --git a/projects/aca-content/folder-rules/src/rule-details/validators/no-whitespace.validator.ts b/projects/aca-content/folder-rules/src/rule-details/validators/no-whitespace.validator.ts new file mode 100644 index 000000000..75d6060b0 --- /dev/null +++ b/projects/aca-content/folder-rules/src/rule-details/validators/no-whitespace.validator.ts @@ -0,0 +1,37 @@ +/*! + * Copyright © 2005-2025 Hyland Software, Inc. and its affiliates. All rights reserved. + * + * Alfresco Example Content Application + * + * This file is part of the Alfresco Example Content Application. + * If the software was purchased under a paid Alfresco license, the terms of + * the paid license agreement will prevail. Otherwise, the software is + * provided under the following open source license terms: + * + * The Alfresco Example Content Application is free software: you can redistribute it and/or modify + * it under the terms of the GNU Lesser General Public License as published by + * the Free Software Foundation, either version 3 of the License, or + * (at your option) any later version. + * + * The Alfresco Example Content Application is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU Lesser General Public License for more details. + * + * You should have received a copy of the GNU Lesser General Public License + * from Hyland Software. If not, see . + */ + +import { AbstractControl, ValidationErrors, ValidatorFn } from '@angular/forms'; + +export const noWhitespaceValidator = (): ValidatorFn => { + return (control: AbstractControl): ValidationErrors | null => { + const rawValue = control.value; + if (rawValue === null || rawValue === '') { + return null; + } + + const trimmedValue = rawValue.toString().trim(); + return trimmedValue.length === 0 ? { whitespace: true } : null; + }; +}; diff --git a/projects/aca-content/src/lib/components/search/search-input-control/search-input-control.component.html b/projects/aca-content/src/lib/components/search/search-input-control/search-input-control.component.html index 41309ba2e..9ba7ab4fb 100644 --- a/projects/aca-content/src/lib/components/search/search-input-control/search-input-control.component.html +++ b/projects/aca-content/src/lib/components/search/search-input-control/search-input-control.component.html @@ -20,7 +20,6 @@ id="app-control-input" [formControl]="searchFieldFormControl" (keydown.enter)="searchSubmit()" - (blur)="onBlur()" [placeholder]="'SEARCH.INPUT.PLACEHOLDER' | translate" autocomplete="off" diff --git a/projects/aca-content/src/lib/components/search/search-input-control/search-input-control.component.spec.ts b/projects/aca-content/src/lib/components/search/search-input-control/search-input-control.component.spec.ts index 3a09fb55d..384a68aa1 100644 --- a/projects/aca-content/src/lib/components/search/search-input-control/search-input-control.component.spec.ts +++ b/projects/aca-content/src/lib/components/search/search-input-control/search-input-control.component.spec.ts @@ -30,13 +30,11 @@ import { NO_ERRORS_SCHEMA } from '@angular/core'; describe('SearchInputControlComponent', () => { let fixture: ComponentFixture; let component: SearchInputControlComponent; - beforeEach(() => { TestBed.configureTestingModule({ imports: [AppTestingModule, SearchInputControlComponent], schemas: [NO_ERRORS_SCHEMA] }); - fixture = TestBed.createComponent(SearchInputControlComponent); component = fixture.componentInstance; fixture.detectChanges(); @@ -54,6 +52,7 @@ describe('SearchInputControlComponent', () => { it('should not emit submit event if form is invalid', () => { component.searchTerm = ''; spyOn(component.submit, 'emit'); + component.searchSubmit(); expect(component.submit.emit).not.toHaveBeenCalled(); @@ -63,14 +62,12 @@ describe('SearchInputControlComponent', () => { let emittedSearchTerm = ''; component.searchChange.subscribe((searchTerm) => (emittedSearchTerm = searchTerm)); component.searchTerm = 'mock-search-term'; - expect(emittedSearchTerm).toBe('mock-search-term'); }); it('should emit searchChange event on clear', () => { let emittedSearchTerm: string = null; component.searchChange.subscribe((searchTerm) => (emittedSearchTerm = searchTerm)); - component.clear(); expect(emittedSearchTerm).toBe(''); }); @@ -78,18 +75,15 @@ describe('SearchInputControlComponent', () => { it('should clear searchTerm', () => { component.searchTerm = 'c'; fixture.detectChanges(); - component.clear(); expect(component.searchTerm).toBe(''); }); it('should check if searchTerm has a length less than 2', () => { expect(component.isTermTooShort()).toBe(false); - component.searchTerm = 'd'; fixture.detectChanges(); expect(component.isTermTooShort()).toBe(true); - component.searchTerm = 'dd'; fixture.detectChanges(); expect(component.isTermTooShort()).toBe(false); diff --git a/projects/aca-content/src/lib/components/search/search-input-control/search-input-control.component.ts b/projects/aca-content/src/lib/components/search/search-input-control/search-input-control.component.ts index d5ddbb57a..87512c2a0 100644 --- a/projects/aca-content/src/lib/components/search/search-input-control/search-input-control.component.ts +++ b/projects/aca-content/src/lib/components/search/search-input-control/search-input-control.component.ts @@ -31,6 +31,7 @@ import { MatFormFieldModule } from '@angular/material/form-field'; import { MatInputModule } from '@angular/material/input'; import { FormControl, FormsModule, ReactiveFormsModule, Validators } from '@angular/forms'; import { takeUntilDestroyed } from '@angular/core/rxjs-interop'; +import { noWhitespaceValidator } from 'projects/aca-content/folder-rules/src/rule-details/validators/no-whitespace.validator'; @Component({ imports: [CommonModule, TranslatePipe, MatButtonModule, MatIconModule, MatFormFieldModule, MatInputModule, FormsModule, ReactiveFormsModule], @@ -65,7 +66,7 @@ export class SearchInputControlComponent implements OnInit { @ViewChild('searchInput', { static: true }) searchInput: ElementRef; - searchFieldFormControl = new FormControl('', [Validators.required]); + searchFieldFormControl = new FormControl('', [Validators.required, noWhitespaceValidator()]); get searchTerm(): string { return this.searchFieldFormControl.value.replace('text:', 'TEXT:'); @@ -83,16 +84,14 @@ export class SearchInputControlComponent implements OnInit { } openDropdown() { - setTimeout(() => { - this.searchInput.nativeElement.focus(); - }, 0); + this.searchInput.nativeElement.focus(); } searchSubmit() { this.searchFieldFormControl.markAsTouched(); const trimmedTerm = this.searchTerm?.trim(); - if (this.searchFieldFormControl.valid && trimmedTerm?.length > 0) { + if (this.searchFieldFormControl.valid && trimmedTerm) { this.submit.emit(trimmedTerm); } } @@ -107,6 +106,6 @@ export class SearchInputControlComponent implements OnInit { } isTermTooShort() { - return !!(this.searchTerm && this.searchTerm.length < 2); + return !!(this.searchTerm.trim() && this.searchTerm.trim().length < 2); } } diff --git a/projects/aca-content/src/lib/components/search/search-input/search-input.component.html b/projects/aca-content/src/lib/components/search/search-input/search-input.component.html index 5f647124f..983879f3d 100644 --- a/projects/aca-content/src/lib/components/search/search-input/search-input.component.html +++ b/projects/aca-content/src/lib/components/search/search-input/search-input.component.html @@ -36,14 +36,16 @@ (submit)="onSearchSubmit($event)" (searchChange)="onSearchChange($event)" /> -
- - {{ 'SEARCH.INPUT.HINT' | translate }} - - - {{ 'SEARCH.INPUT.REQUIRED' | translate }} - -
+ + {{ 'SEARCH.INPUT.HINT' | translate }} + + + {{ 'SEARCH.INPUT.REQUIRED' | translate }} + + + {{ 'SEARCH.INPUT.WHITESPACE' | translate }} +
{ let fixture: ComponentFixture; let component: SearchInputComponent; let store: jasmine.SpyObj>; + let unitTestingUtils: UnitTestingUtils; + let loader: HarnessLoader; + + function getFirstError(): string { + const error = unitTestingUtils.getByDirective(MatError); + return error?.nativeElement.textContent.trim(); + } + + async function openMenu() { + const menu = await loader.getHarness(MatMenuHarness); + await menu.open(); + return menu; + } + + async function getCheckbox(id: string) { + const overlayLoader = TestbedHarnessEnvironment.documentRootLoader(fixture); + return overlayLoader.getHarness(MatCheckboxHarness.with({ selector: `#${id}` })); + } + + async function uncheckAllCheckboxes() { + const checkboxIds = ['libraries', 'folder', 'content']; + for (const id of checkboxIds) { + try { + const checkbox = await getCheckbox(id); + if (await checkbox.isChecked()) { + await checkbox.uncheck(); + fixture.detectChanges(); + } + } catch (err) { + fail(`Checkbox with id ${id} not found`); + } + } + } beforeEach(async () => { const storeSpy = jasmine.createSpyObj>('Store', ['dispatch', 'pipe']); await TestBed.configureTestingModule({ - imports: [AppTestingModule, ReactiveFormsModule, SearchInputComponent, SearchInputControlComponent], - providers: [{ provide: Store, useValue: storeSpy }], - schemas: [NO_ERRORS_SCHEMA] + imports: [AppTestingModule, SearchInputComponent], + providers: [{ provide: Store, useValue: storeSpy }] }).compileComponents(); fixture = TestBed.createComponent(SearchInputComponent); @@ -53,21 +86,13 @@ describe('SearchInputComponent', () => { store = TestBed.inject(Store) as jasmine.SpyObj>; store.pipe.and.returnValue(of([])); fixture.detectChanges(); + unitTestingUtils = new UnitTestingUtils(fixture.debugElement); + loader = TestbedHarnessEnvironment.loader(fixture); }); - function getFirstError(): string { - const error = fixture.debugElement.query(By.directive(MatError)); - return error?.nativeElement.textContent.trim(); - } + it('should show required error when field is empty and touched', async () => { + await openMenu(); - function openSearchContainer(): void { - const menuButton = fixture.debugElement.query(By.css('.app-search-container')); - menuButton?.nativeElement.click(); - fixture.detectChanges(); - } - - it('should show required error when field is empty and touched', () => { - openSearchContainer(); component.searchInputControl.searchFieldFormControl.setValue(''); component.searchInputControl.searchFieldFormControl.markAsTouched(); fixture.detectChanges(); @@ -75,47 +100,50 @@ describe('SearchInputComponent', () => { expect(getFirstError()).toBe('SEARCH.INPUT.REQUIRED'); }); - it('should not show error when field has value', () => { - openSearchContainer(); - component.searchInputControl.searchFieldFormControl.setValue('test'); + it('should not show error when field has value', async () => { + await openMenu(); + + component.searchInputControl.searchFieldFormControl.setValue('not giving up'); component.searchInputControl.searchFieldFormControl.markAsTouched(); fixture.detectChanges(); - const error = fixture.debugElement.query(By.directive(MatError)); + const error = unitTestingUtils.getByDirective(MatError); expect(error).toBeNull(); }); - it('should not show error when field is untouched', () => { - openSearchContainer(); + it('should not show error when field is untouched', async () => { + await openMenu(); + component.searchInputControl.searchFieldFormControl.setValue(''); component.searchInputControl.searchFieldFormControl.markAsUntouched(); fixture.detectChanges(); - const error = fixture.debugElement.query(By.directive(MatError)); + const error = unitTestingUtils.getByDirective(MatError); expect(error).toBeNull(); }); - it('should dispatch SearchByTermAction when libraries are checked and term is new', () => { - spyOn(component as any, 'isLibrariesChecked').and.returnValue(true); - spyOn(component as any, 'isFoldersChecked').and.returnValue(false); - spyOn(component as any, 'isFilesChecked').and.returnValue(false); - - component.searchedWord = 'test'; - component.onSearchSubmit('Enter'); + it('should dispatch action when Libraries checkbox selected and term is entered', async () => { + await openMenu(); + const checkbox = await getCheckbox('libraries'); + await checkbox.check(); + fixture.detectChanges(); + component.onSearchSubmit({ target: { value: 'happy faces only' } }); expect(store.dispatch).toHaveBeenCalled(); }); - it('should not dispatch SearchByTermAction when no checkboxes are selected and term is empty', () => { + it('should not dispatch SearchByTermAction when no checkboxes are selected and term is empty', async () => { store.dispatch.calls.reset(); - spyOn(component as any, 'isLibrariesChecked').and.returnValue(false); - spyOn(component as any, 'isFoldersChecked').and.returnValue(false); - spyOn(component as any, 'isFilesChecked').and.returnValue(false); + await openMenu(); + await uncheckAllCheckboxes(); + + expect(component.searchOptions.every((option) => !option.value)).toBeTrue(); component.searchedWord = ''; - component.onSearchSubmit({ target: { value: '' } }); + fixture.detectChanges(); + component.onSearchSubmit({ target: { value: '' } }); expect(store.dispatch).not.toHaveBeenCalled(); }); }); diff --git a/projects/aca-content/src/lib/components/search/search-input/search-input.component.ts b/projects/aca-content/src/lib/components/search/search-input/search-input.component.ts index 9390bd215..9a9c13322 100644 --- a/projects/aca-content/src/lib/components/search/search-input/search-input.component.ts +++ b/projects/aca-content/src/lib/components/search/search-input/search-input.component.ts @@ -73,6 +73,7 @@ export class SearchInputComponent implements OnInit, OnDestroy { has400LibraryError = false; hasLibrariesConstraint = false; searchOnChange: boolean; + isTrimmedWordEmpty = false; searchedWord: string = null; searchOptions: Array = [ @@ -177,10 +178,15 @@ export class SearchInputComponent implements OnInit, OnDestroy { */ onSearchSubmit(event: any) { const searchTerm = event.target ? (event.target as HTMLInputElement).value : event; - if (searchTerm) { - this.searchedWord = searchTerm; + const trimmedTerm = searchTerm.trim(); - this.searchByOption(); + if (trimmedTerm) { + this.searchedWord = trimmedTerm; + if (this.isLibrariesChecked() && this.searchInputControl.isTermTooShort()) { + return; + } else { + this.searchByOption(); + } } else { this.notificationService.showError('APP.BROWSE.SEARCH.EMPTY_SEARCH'); }