clean up

Author: siltomato

src/SIL.XForge.Scripture/ClientApp/src/app/shared/sf-tab-group/tab-group-header/tab-group-header.component.ts

@@ -39,19 +39,19 @@ import { TabScrollButtonComponent } from './tab-scroll-button/tab-scroll-button.
   templateUrl: './tab-group-header.component.html',
   styleUrls: ['./tab-group-header.component.scss'],
   imports: [
-    TabScrollButtonComponent,
     CdkDropList,
-    NgFor,
-    TabHeaderComponent,
     CdkDrag,
+    NgFor,
+    NgIf,
     NgTemplateOutlet,
-    MatMenuTrigger,
+    AsyncPipe,
     MatIcon,
     MatMenu,
-    NgIf,
     MatMenuItem,
-    AsyncPipe,
-    TranslocoModule
+    MatMenuTrigger,
+    TranslocoModule,
+    TabHeaderComponent,
+    TabScrollButtonComponent
   ]
 })
 export class TabGroupHeaderComponent implements OnChanges, OnInit, AfterViewInit, OnDestroy {

src/SIL.XForge.Scripture/ClientApp/src/app/translate/editor/lynx/insights/lynx-insight-editor-objects/lynx-insight-editor-objects.component.spec.ts

@@ -1,6 +1,7 @@
 import { Component, DestroyRef, NO_ERRORS_SCHEMA, ViewChild } from '@angular/core';
 import { ComponentFixture, fakeAsync, flush, TestBed, tick } from '@angular/core/testing';
 import { Delta } from 'quill';
+import { LynxInsightFilter, LynxInsightType } from 'realtime-server/lib/esm/scriptureforge/models/lynx-insight';
 import { BehaviorSubject } from 'rxjs';
 import { anything, instance, mock, verify, when } from 'ts-mockito';
 import { ActivatedBookChapterService, RouteBookChapter } from 'xforge-common/activated-book-chapter.service';
@@ -384,8 +385,8 @@ class TestEnvironment {
   private filteredInsightsSubject: BehaviorSubject<LynxInsight[]>;
   private displayStateSubject: BehaviorSubject<LynxInsightDisplayState>;
   private activatedBookChapterSubject: BehaviorSubject<RouteBookChapter | undefined>;
-  private filterSubject: BehaviorSubject<any>;
-  private filteredInsightCountsByTypeSubject: BehaviorSubject<any>;
+  private filterSubject: BehaviorSubject<LynxInsightFilter>;
+  private filteredInsightCountsByTypeSubject: BehaviorSubject<Record<LynxInsightType, number>>;
   private taskRunningStatusSubject: BehaviorSubject<boolean>;
 
   constructor(args: TestEnvArgs = {}) {

src/SIL.XForge.Scripture/ClientApp/src/app/translate/editor/lynx/insights/lynx-insights-panel/lynx-insights-panel.component.spec.ts

@@ -124,6 +124,7 @@ describe('LynxInsightsPanelComponent', () => {
     ],
     schemas: [CUSTOM_ELEMENTS_SCHEMA],
     providers: [
+      provideNoopAnimations(),
       { provide: LynxInsightStateService, useMock: mockLynxInsightStateService },
       { provide: ActivatedProjectService, useMock: mockActivatedProjectService },
       { provide: ActivatedBookChapterService, useMock: mockActivatedBookChapterService },
@@ -134,7 +135,7 @@ describe('LynxInsightsPanelComponent', () => {
       {
         provide: EDITOR_INSIGHT_DEFAULTS,
         useValue: {
-          filter: { types: ['info', 'warning', 'error', provideNoopAnimations()], scope: 'chapter' },
+          filter: { types: ['info', 'warning', 'error'], scope: 'chapter' },
           sortOrder: 'severity',
           queryParamName: 'insight',
           actionOverlayApplyPrimaryActionChord: { altKey: true, shiftKey: true, key: 'Enter' },