Fix some linting and testing issues

ChrisMBarr · Oct 5, 2024 · 4430228 · 4430228
1 parent 805780c
commit 4430228
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 2 deletions.
diff --git a/src/app/app.component.spec.ts b/src/app/app.component.spec.ts
@@ -1,6 +1,6 @@
 import { DOCUMENT } from '@angular/common';
 import { ComponentFixture, TestBed } from '@angular/core/testing';
-import { RouterTestingModule } from '@angular/router/testing';
+import { provideRouter, RouterLink, RouterLinkActive, RouterOutlet } from '@angular/router';
 
 import { AboutComponent } from './about/about.component';
 import { AppComponent } from './app.component';
@@ -16,8 +16,9 @@ describe('AppComponent', () => {
 
   beforeEach(() => {
     TestBed.configureTestingModule({
-      imports: [RouterTestingModule],
+      imports: [RouterOutlet, RouterLink, RouterLinkActive],
       declarations: [AppComponent, HelpComponent, AboutComponent, ConvertComponent, DonateButtonComponent, DragAndDropFilesDirective],
+      providers: [provideRouter([])],
     });
 
     fixture = TestBed.createComponent(AppComponent);

diff --git a/src/app/convert/download-display/download-display.component.spec.ts b/src/app/convert/download-display/download-display.component.spec.ts
@@ -1,3 +1,4 @@
+/* eslint-disable @typescript-eslint/no-deprecated -- TS selects the wrong overload and says it deprecated. It's not */
 import { ComponentFixture, fakeAsync, TestBed, tick } from '@angular/core/testing';
 import { By } from '@angular/platform-browser';
 import * as fileSaver from 'file-saver';

diff --git a/src/app/convert/download-display/download-display.component.ts b/src/app/convert/download-display/download-display.component.ts
@@ -15,6 +15,7 @@ export class DownloadDisplayComponent {
 
   onClickDownloadFiles(): void {
     for (const outputFile of this.outputFileList) {
+      // eslint-disable-next-line @typescript-eslint/no-deprecated -- TS selects the wrong overload and says it deprecated. It's not
       fileSaver.saveAs(
         new File(['\ufeff' + outputFile.outputContent], outputFile.fileName, {
           type: 'text/plain',
@@ -36,6 +37,7 @@ export class DownloadDisplayComponent {
         type: 'blob',
       })
       .then((zipContent: Blob) => {
+        // eslint-disable-next-line @typescript-eslint/no-deprecated -- TS selects the wrong overload and says it deprecated. It's not
         fileSaver.saveAs(
           zipContent,
           `LyricConverter (${this.outputFileList.length.toString()} files).zip`,