From ad7179650750023f270328d0fcaefba1f4a7ce02 Mon Sep 17 00:00:00 2001
From: Samuel <samuel@soundmarker.com>
Date: Wed, 8 Apr 2020 10:53:25 +0200
Subject: [PATCH] Edit Community - Assign Roles/Groups (Angular) - feedback

---
 .../edit-comcol-page/comcol-role/comcol-role.component.html | 2 +-
 .../comcol-role/comcol-role.component.spec.ts               | 6 +++---
 2 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/src/app/shared/comcol-forms/edit-comcol-page/comcol-role/comcol-role.component.html b/src/app/shared/comcol-forms/edit-comcol-page/comcol-role/comcol-role.component.html
index 26460d5367..a50845889e 100644
--- a/src/app/shared/comcol-forms/edit-comcol-page/comcol-role/comcol-role.component.html
+++ b/src/app/shared/comcol-forms/edit-comcol-page/comcol-role/comcol-role.component.html
@@ -15,7 +15,7 @@
           </div>
 
           <a *ngIf="group"
-             href="{{editGroupLink$ | async}}">
+             routerLink="{{editGroupLink$ | async}}">
             {{group.name}}
           </a>
 
diff --git a/src/app/shared/comcol-forms/edit-comcol-page/comcol-role/comcol-role.component.spec.ts b/src/app/shared/comcol-forms/edit-comcol-page/comcol-role/comcol-role.component.spec.ts
index ab5696312b..4750da34bb 100644
--- a/src/app/shared/comcol-forms/edit-comcol-page/comcol-role/comcol-role.component.spec.ts
+++ b/src/app/shared/comcol-forms/edit-comcol-page/comcol-role/comcol-role.component.spec.ts
@@ -12,6 +12,7 @@ import { ComcolRole } from './comcol-role';
 import { of as observableOf } from 'rxjs/internal/observable/of';
 import { RemoteData } from '../../../../core/data/remote-data';
 import { Group } from '../../../../core/eperson/models/group.model';
+import { RouterTestingModule } from '@angular/router/testing';
 
 describe('ComcolRoleComponent', () => {
 
@@ -34,10 +35,9 @@ describe('ComcolRoleComponent', () => {
 
     TestBed.configureTestingModule({
       imports: [
-        TranslateModule.forRoot(),
         SharedModule,
-      ],
-      declarations: [
+        RouterTestingModule.withRoutes([]),
+        TranslateModule.forRoot(),
       ],
       providers: [
         { provide: GroupDataService, useValue: groupService },