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 },