From cd62ee1c9bf2523760fe4d6389c97b27984eb507 Mon Sep 17 00:00:00 2001
From: Erin Hall <halle@mitre.org>
Date: Thu, 2 Nov 2023 16:29:47 -0400
Subject: [PATCH] improved code coverage for help component

---
 nav-app/src/app/help/help.component.spec.ts | 94 ++++++++++++++++++++-
 1 file changed, 92 insertions(+), 2 deletions(-)

diff --git a/nav-app/src/app/help/help.component.spec.ts b/nav-app/src/app/help/help.component.spec.ts
index eaf5dc0ed..c1fd40439 100755
--- a/nav-app/src/app/help/help.component.spec.ts
+++ b/nav-app/src/app/help/help.component.spec.ts
@@ -1,15 +1,22 @@
 import { ComponentFixture, TestBed, waitForAsync } from '@angular/core/testing';
-import { MatDialogModule, MAT_DIALOG_DATA, MatDialogRef } from '@angular/material/dialog';
+import { MatDialogModule, MAT_DIALOG_DATA, MatDialogRef, MatDialog } from '@angular/material/dialog';
 import { HelpComponent } from './help.component';
 import { HttpClientModule, HttpClient } from '@angular/common/http';
 import { MarkdownService, MarkdownModule } from 'ngx-markdown';
+import { Renderer2 } from '@angular/core';
+import { BrowserAnimationsModule } from '@angular/platform-browser/animations';
+
 describe('HelpComponent', () => {
     let component: HelpComponent;
     let fixture: ComponentFixture<HelpComponent>;
+    let markdownService: MarkdownService;
+    let dialog: MatDialog;
+    let renderer: Renderer2;
+    let mockMarkdownElement: any
 
     beforeEach(waitForAsync(() => {
         TestBed.configureTestingModule({
-            imports: [HttpClientModule, MatDialogModule, MarkdownModule.forRoot({ loader: HttpClient })],
+            imports: [HttpClientModule, MatDialogModule, MarkdownModule.forRoot({ loader: HttpClient }), BrowserAnimationsModule],
             declarations: [HelpComponent],
             providers: [
                 {
@@ -21,17 +28,100 @@ describe('HelpComponent', () => {
                     useValue: {},
                 },
                 MarkdownService,
+                {
+                    provide: Renderer2,
+                    useValue: {
+                        listen: jasmine.createSpy('listen').and.returnValue(() => { }),
+                    },
+                },
             ],
         }).compileComponents();
+        dialog = TestBed.inject(MatDialog);
+        markdownService = TestBed.inject(MarkdownService);
+        renderer = TestBed.inject(Renderer2);
+
+
     }));
 
     beforeEach(() => {
         fixture = TestBed.createComponent(HelpComponent);
         component = fixture.componentInstance;
+        renderer = fixture.componentRef.injector.get<Renderer2>(Renderer2)
+        mockMarkdownElement = {
+            element: {
+                nativeElement: document.createElement('div')
+            }
+        };
+        component['markdownElement'] = mockMarkdownElement
+        spyOn(renderer, 'listen').and.callFake((elem, eventName, callback) => {
+            return () => { }
+        });
+        fixture.detectChanges();
+        spyOn(component, 'scrollTo').and.callThrough();
+        spyOn(dialog, 'open').and.callThrough();
         fixture.detectChanges();
     });
 
     it('should create', () => {
         expect(component).toBeTruthy();
     });
+
+    describe('ngOnDestroy', () => {
+        it('should remove event listeners if any', () => {
+            component['listenObj'] = jasmine.createSpy();
+            component.ngOnDestroy();
+            expect(component['listenObj']).toHaveBeenCalled();
+        });
+    });
+
+    describe('scrollTo', () => {
+        it('should call scrollIntoView when element exists', () => {
+            const mockElement = document.createElement('div');
+            spyOn(document, 'querySelector').and.returnValue(mockElement);
+            spyOn(mockElement, 'scrollIntoView');
+            component.scrollTo('toc');
+            expect(document.querySelector).toHaveBeenCalledWith('.toc');
+            expect(mockElement.scrollIntoView).toHaveBeenCalledWith({
+                behavior: 'smooth',
+                block: 'start',
+                inline: 'nearest',
+            });
+        });
+
+        it('should not call scrollIntoView when element does not exist', () => {
+            spyOn(document, 'querySelector').and.returnValue(null);
+            const scrollSpy = spyOn(window, 'scrollTo');
+            component.scrollTo('toc');
+            expect(document.querySelector).toHaveBeenCalledWith('.toc');
+            expect(scrollSpy).not.toHaveBeenCalled();
+        });
+    });
+
+    describe('openLayerDialog', () => {
+        it('should open the dialog with LayerInformationComponent', () => {
+            component.openLayerDialog();
+            expect(dialog.open).toHaveBeenCalled();
+        });
+    });
+
+    describe('onMarkdownLoad', () => {
+        it('should set up click listener on markdown element', () => {
+            component.onMarkdownLoad(null);
+            expect(renderer.listen).toHaveBeenCalled();
+        });
+    });
+
+    describe('MarkdownService renderer overrides', () => {
+        it('should override heading renderer', () => {
+            const mockHeading = 'Heading';
+            const level = 1;
+            markdownService.renderer.heading(mockHeading, level, null, null);
+            expect(component.headingAnchors.length).toBeGreaterThan(0);
+        });
+
+        it('should override html renderer', () => {
+            const mockHtml = '<div></div>';
+            markdownService.renderer.html(mockHtml);
+        });
+    });
 });