From f36edf8fa7f3de4d52df83a008498022f70aa97c Mon Sep 17 00:00:00 2001
From: snss231 <e0543670@u.nus.edu>
Date: Tue, 19 Jul 2022 15:50:42 +0800
Subject: [PATCH 1/2] Add profiles component DOM tests

---
 src/test.ts                                   |   2 +-
 .../auth/profiles/profiles.component.spec.ts  | 106 ++++++++++++++++++
 2 files changed, 107 insertions(+), 1 deletion(-)
 create mode 100644 tests/app/auth/profiles/profiles.component.spec.ts

diff --git a/src/test.ts b/src/test.ts
index c4163e646..290c154b4 100644
--- a/src/test.ts
+++ b/src/test.ts
@@ -1,8 +1,8 @@
 // This file is required by karma.conf.js and loads recursively all the .spec and framework files
+import 'zone.js/dist/zone-testing'; // This import must be at the top
 
 import { getTestBed } from '@angular/core/testing';
 import { BrowserDynamicTestingModule, platformBrowserDynamicTesting } from '@angular/platform-browser-dynamic/testing';
-import 'zone.js/dist/zone-testing';
 
 declare const require: any;
 
diff --git a/tests/app/auth/profiles/profiles.component.spec.ts b/tests/app/auth/profiles/profiles.component.spec.ts
new file mode 100644
index 000000000..22f762678
--- /dev/null
+++ b/tests/app/auth/profiles/profiles.component.spec.ts
@@ -0,0 +1,106 @@
+import { Component, DebugElement } from '@angular/core';
+import { async, ComponentFixture, fakeAsync, TestBed, tick } from '@angular/core/testing';
+import { BrowserAnimationsModule } from '@angular/platform-browser/animations';
+
+import { ProfilesComponent } from '../../../../src/app/auth/profiles/profiles.component';
+import { Profile } from '../../../../src/app/core/models/profile.model';
+import { ErrorHandlingService } from '../../../../src/app/core/services/error-handling.service';
+import { ProfileService } from '../../../../src/app/core/services/profile.service';
+import { SharedModule } from '../../../../src/app/shared/shared.module';
+
+@Component({
+  template: `<app-profiles (selectedProfileEmitter)="onSelected($event)"> </app-profiles>`
+})
+class TestHostComponent {
+  selectedProfile: Profile | undefined;
+  onSelected(profile: Profile) {
+    this.selectedProfile = profile;
+  }
+}
+
+describe('ProfilesComponent', () => {
+  let fixture: ComponentFixture<TestHostComponent>;
+  let debugElement: DebugElement;
+  let nativeElement: HTMLElement;
+  let testHost: TestHostComponent;
+  let profilesEl: HTMLElement;
+
+  const profileService = jasmine.createSpyObj('ProfileService', ['fetchExternalProfiles']);
+  const errorHandlingService = jasmine.createSpyObj('ErrorHandlingService', ['handleError']);
+
+  const testProfiles: Profile[] = [
+    {
+      profileName: 'testProfile1',
+      repoName: 'test-org1/pe'
+    },
+    {
+      profileName: 'testProfile2',
+      repoName: 'test-org2/pe'
+    },
+    {
+      profileName: 'testProfile3',
+      repoName: 'test-org3/pe'
+    }
+  ];
+
+  beforeEach(async(() => {
+    TestBed.configureTestingModule({
+      imports: [BrowserAnimationsModule, SharedModule],
+      declarations: [ProfilesComponent, TestHostComponent],
+      providers: [
+        { provide: ProfileService, useValue: profileService },
+        { provide: ErrorHandlingService, useValue: errorHandlingService }
+      ]
+    }).compileComponents();
+  }));
+
+  beforeEach(fakeAsync(() => {
+    profileService.fetchExternalProfiles.and.returnValue(Promise.resolve(testProfiles));
+
+    fixture = TestBed.createComponent(TestHostComponent);
+    testHost = fixture.componentInstance;
+
+    debugElement = fixture.debugElement;
+    nativeElement = debugElement.nativeElement;
+    profilesEl = <HTMLElement>nativeElement.querySelector('app-profiles');
+
+    fixture.detectChanges(); // onInit()
+    tick(); // wait for profiles to be loaded
+  }));
+
+  it('should display the correct profiles from AppConfig', () => {
+    openMatSelect();
+    const displayedOptions = getOptions();
+
+    displayedOptions.slice(1).forEach((el, i) => {
+      const optionTextEl = <HTMLElement>el.querySelector('.mat-option-text');
+      if (!optionTextEl) {
+        fail('optionTextEl should not be null');
+      }
+      const profileName = optionTextEl.innerText;
+      expect(profileName).toBe(testProfiles[i].profileName);
+    });
+  });
+
+  it('should emit the correct profile when mat-option is clicked', () => {
+    openMatSelect();
+    const displayedOptions = getOptions();
+
+    displayedOptions[3].click();
+    expect(testHost.selectedProfile).toEqual(testProfiles[2]);
+    fixture.detectChanges();
+  });
+
+  function openMatSelect(): void {
+    const select = <HTMLElement>profilesEl.querySelector('.mat-select');
+    if (!select) {
+      fail('Select should not be null');
+    }
+    select.click();
+    fixture.detectChanges();
+  }
+
+  function getOptions(): HTMLElement[] {
+    return Array.from(document.querySelectorAll('.mat-option'));
+  }
+});

From 0908450461cafa2ddb0e54feb1a00e1fb577d226 Mon Sep 17 00:00:00 2001
From: snss231 <e0543670@u.nus.edu>
Date: Mon, 24 Oct 2022 20:59:21 +0800
Subject: [PATCH 2/2] Edit spec description to specify selectProfile

---
 tests/app/auth/profiles/profiles.component.spec.ts | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/tests/app/auth/profiles/profiles.component.spec.ts b/tests/app/auth/profiles/profiles.component.spec.ts
index 22f762678..8883b4ae5 100644
--- a/tests/app/auth/profiles/profiles.component.spec.ts
+++ b/tests/app/auth/profiles/profiles.component.spec.ts
@@ -82,7 +82,7 @@ describe('ProfilesComponent', () => {
     });
   });
 
-  it('should emit the correct profile when mat-option is clicked', () => {
+  it('should emit the correct profile through selectProfile when mat-option is clicked', () => {
     openMatSelect();
     const displayedOptions = getOptions();