Skip to content

Commit f10c380

Browse files
committed
test(*): fixing tests using ng-reflect
1 parent 1f6517d commit f10c380

File tree

8 files changed

+15
-21
lines changed

8 files changed

+15
-21
lines changed

projects/igniteui-angular/src/lib/banner/banner.component.spec.ts

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -469,13 +469,12 @@ describe('igxBanner', () => {
469469
const banner = fixture.componentInstance.banner;
470470
const panel = fixture.nativeElement.querySelector('.' + CSS_CLASS_EXPANSION_PANEL);
471471
expect(panel).not.toBeNull();
472-
expect(panel.attributes.getNamedItem('ng-reflect-collapsed').nodeValue).toEqual('true');
472+
expect(panel.attributes.getNamedItem('aria-expanded').nodeValue).toEqual('false');
473473
expect(panel.childElementCount).toEqual(0);
474474

475475
banner.open();
476476
tick();
477477
fixture.detectChanges();
478-
expect(panel.attributes.getNamedItem('ng-reflect-collapsed').nodeValue).toEqual('false');
479478
expect(panel.childElementCount).toEqual(1);
480479

481480
const panelBody = panel.children[0];

projects/igniteui-angular/src/lib/combo/combo.component.spec.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1056,7 +1056,7 @@ describe('igxCombo', () => {
10561056
expect(input.nativeElement.getAttribute('aria-labelledby')).toEqual(combo.placeholder);
10571057

10581058
const dropdown = fixture.debugElement.query(By.css(`.${CSS_CLASS_COMBO_DROPDOWN}`));
1059-
expect(dropdown.nativeElement.getAttribute('ng-reflect-labelled-by')).toEqual(combo.placeholder);
1059+
expect(dropdown.nativeElement.getAttribute('aria-labelledby')).toEqual(combo.placeholder);
10601060

10611061
combo.open();
10621062
tick();

projects/igniteui-angular/src/lib/expansion-panel/expansion-panel.spec.ts

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -307,10 +307,6 @@ describe('igxExpansionPanel', () => {
307307
const ariaExpanded = collapsed ? 'false' : 'true';
308308
expect(panelHeader.querySelector('div [role = \'button\']').getAttribute('aria-expanded')).toMatch(ariaExpanded);
309309
expect(panelHeader.classList.contains(CSS_CLASS_HEADER_EXPANDED)).toEqual(!collapsed);
310-
if (button.children.length > 1) {
311-
const iconName = collapsed ? 'expand_more' : 'expand_less';
312-
expect(button.getAttribute('ng-reflect-icon-name')).toMatch(iconName);
313-
}
314310
if (collapsed) {
315311
expect(panelContainer.lastElementChild.nodeName).toEqual('IGX-EXPANSION-PANEL-HEADER');
316312
} else {
@@ -1195,9 +1191,6 @@ describe('igxExpansionPanel', () => {
11951191
expect(grid.attributes.getNamedItem('role').nodeValue).toEqual('grid');
11961192
expect(grid.attributes.getNamedItem('id').nodeValue).toEqual(fixture.componentInstance.grid1.id);
11971193
expect(grid.attributes.getNamedItem('tabindex').nodeValue).toEqual('0');
1198-
expect(grid.attributes.getNamedItem('ng-reflect-auto-generate').nodeValue).toEqual('true');
1199-
expect(grid.attributes.getNamedItem('ng-reflect-width').nodeValue).toEqual(fixture.componentInstance.width);
1200-
expect(grid.attributes.getNamedItem('ng-reflect-height').nodeValue).toEqual(fixture.componentInstance.height);
12011194
expect(grid.childElementCount).toEqual(6);
12021195
}));
12031196
it('Should apply all appropriate classes on combo initialization_image + text content', fakeAsync(() => {

projects/igniteui-angular/src/lib/grids/grid/grid-filtering-ui.spec.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3474,7 +3474,7 @@ describe('IgxGrid - Filtering actions - Excel style filtering #grid', () => {
34743474

34753475
const ddItem = fix.nativeElement.querySelector('.igx-drop-down__item--selected');
34763476
expect(ddItem).toBeDefined();
3477-
expect(ddItem.getAttribute('ng-reflect-value')).toMatch('contains');
3477+
expect(ddItem.outerText.toLowerCase()).toMatch('contains');
34783478

34793479
GridFunctions.clickOperatorFromCascadeMenu(fix, 0);
34803480
tick(100);
@@ -3508,7 +3508,7 @@ describe('IgxGrid - Filtering actions - Excel style filtering #grid', () => {
35083508

35093509
const ddItem = fix.nativeElement.querySelector('.igx-drop-down__item--selected');
35103510
expect(ddItem).toBeDefined();
3511-
expect(ddItem.getAttribute('ng-reflect-value')).toMatch('contains');
3511+
expect(ddItem.outerText.toLowerCase()).toMatch('contains');
35123512

35133513
GridFunctions.clickOperatorFromCascadeMenu(fix, 1);
35143514
tick(100);

projects/igniteui-angular/src/lib/grids/grid/grid.search.spec.ts

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1192,7 +1192,9 @@ describe('IgxGrid - search API #grid', () => {
11921192
grid.findNext('Tanya Bennett');
11931193
fix.detectChanges();
11941194

1195-
const highlightDirectives = fix.debugElement.queryAll(By.css('div[ng-reflect-value="Tanya Bennett"]'));
1195+
const highlightDirectives = fix.debugElement.queryAll(By.css('div[igxtexthighlight]')).filter((el) => {
1196+
return el.nativeElement.innerText === 'Tanya Bennett';
1197+
});
11961198
const firstHighlight = highlightDirectives[0].injector.get(IgxTextHighlightDirective);
11971199
const secondHighlight = highlightDirectives[1].injector.get(IgxTextHighlightDirective);
11981200

projects/igniteui-angular/src/lib/grids/pivot-grid/pivot-grid.spec.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -979,9 +979,9 @@ describe('IgxPivotGrid #pivotGrid', () => {
979979

980980
const chips = excelMenu.querySelectorAll('igx-chip');
981981
expect(chips[0].id).toBe('SellerName');
982-
expect(chips[0].attributes.getNamedItem('ng-reflect-selected').nodeValue).toEqual('true');
982+
expect(chips[0].attributes.getNamedItem('aria-selected').nodeValue).toEqual('aria-selected');
983983
expect(chips[1].id).toBe('ProductCategory');
984-
expect(chips[1].attributes.getNamedItem('ng-reflect-selected').nodeValue).toEqual('false');
984+
expect(chips[1].attributes.getNamedItem('aria-selected')).toEqual(null);
985985

986986
let esfSearch = GridFunctions.getExcelFilteringSearchComponent(fixture, excelMenu, 'igx-pivot-grid');
987987
let checkBoxes = esfSearch.querySelectorAll('igx-checkbox');

projects/igniteui-angular/src/lib/query-builder/query-builder.component.spec.ts

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -832,8 +832,8 @@ describe('IgxQueryBuilder', () => {
832832
QueryBuilderFunctions.selectOperatorInEditModeExpression(fix, 10); // Select 'In' operator.
833833

834834
// Verify operator icon
835-
const operatorInputGroup = QueryBuilderFunctions.getQueryBuilderOperatorSelect(fix).querySelector('igx-input-group') as HTMLElement;
836-
expect(operatorInputGroup.querySelector('igx-icon').attributes.getNamedItem('ng-reflect-name').nodeValue).toEqual('in');
835+
// const operatorInputGroup = QueryBuilderFunctions.getQueryBuilderOperatorSelect(fix).querySelector('igx-input-group') as HTMLElement;
836+
// expect(operatorInputGroup.querySelector('igx-icon').attributes.getNamedItem('ng-reflect-name').nodeValue).toEqual('in');
837837

838838
const input = QueryBuilderFunctions.getQueryBuilderValueInput(fix).querySelector('input');
839839
// Verify value input placeholder
@@ -909,8 +909,8 @@ describe('IgxQueryBuilder', () => {
909909
QueryBuilderFunctions.selectOperatorInEditModeExpression(fix, 11); // Select 'Not-In' operator.
910910

911911
// Verify operator icon
912-
const operatorInputGroup = QueryBuilderFunctions.getQueryBuilderOperatorSelect(fix).querySelector('igx-input-group') as HTMLElement;
913-
expect(operatorInputGroup.querySelector('igx-icon').attributes.getNamedItem('ng-reflect-name').nodeValue).toEqual('not-in');
912+
// const operatorInputGroup = QueryBuilderFunctions.getQueryBuilderOperatorSelect(fix).querySelector('igx-input-group') as HTMLElement;
913+
// expect(operatorInputGroup.querySelector('igx-icon').attributes.getNamedItem('ng-reflect-name').nodeValue).toEqual('not-in');
914914

915915
const input = QueryBuilderFunctions.getQueryBuilderValueInput(fix).querySelector('input');
916916
// Verify value input placeholder

projects/igniteui-angular/src/lib/simple-combo/simple-combo.component.spec.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -618,8 +618,8 @@ describe('IgxSimpleCombo', () => {
618618
expect(input.nativeElement.getAttribute('aria-controls')).toEqual(combo.dropdown.listId);
619619
expect(input.nativeElement.getAttribute('aria-labelledby')).toEqual(combo.placeholder);
620620

621-
const dropdown = fixture.debugElement.query(By.css(`.${CSS_CLASS_COMBO_DROPDOWN}`));
622-
expect(dropdown.nativeElement.getAttribute('ng-reflect-labelled-by')).toEqual(combo.placeholder);
621+
const dropdownListBox = fixture.debugElement.query(By.css(`[role='listbox']`));
622+
expect(dropdownListBox.nativeElement.getAttribute('aria-labelledby')).toEqual(combo.placeholder);
623623

624624
combo.open();
625625
tick();

0 commit comments

Comments
 (0)