fix(highlight): skip active highlight if index is -1 (#10030)

* fix(highlight): skip active highlight if index is -1
* test(highlight): add activateIfNecessary test

Co-authored-by: Galina Edinakova <[email protected]>
Co-authored-by: igdmdimitrov <[email protected]>
Co-authored-by: Konstantin Dinev <[email protected]>

Author: 4 people

projects/igniteui-angular/src/lib/directives/text-highlight/text-highlight.directive.spec.ts

@@ -286,6 +286,19 @@ describe('IgxHighlight', () => {
         expect(spans.length).toBe(4);
         expect(activeSpans.length).toBe(1);
     });
+
+    it('Should not throw error when active highlight is not set', () => {
+        const fix = TestBed.createComponent(HighlightLoremIpsumComponent);
+        fix.detectChanges();
+
+        const component: HighlightLoremIpsumComponent = fix.debugElement.componentInstance;
+        component.highlight.row = undefined;
+        component.highlight.column = undefined;
+        component.highlightText('a');
+
+        expect(() => component.highlight.activateIfNecessary()).not.toThrowError();
+    });
+
 });
 
 @Component({

projects/igniteui-angular/src/lib/directives/text-highlight/text-highlight.directive.ts

@@ -340,7 +340,7 @@ export class IgxTextHighlightDirective implements AfterViewInit, AfterViewChecke
     public activateIfNecessary(): void {
         const group = IgxTextHighlightDirective.highlightGroupsMap.get(this.groupName);
 
-        if (group.column === this.column && group.row === this.row && compareMaps(this.metadata, group.metadata)) {
+        if (group.index >= 0 && group.column === this.column && group.row === this.row && compareMaps(this.metadata, group.metadata)) {
             this.activate(group.index);
         }
     }