Skip to content

Commit ae94245

Browse files
IxquitilisSaidalan-agius4
authored andcommitted
fix(@angular-devkit/build-angular): add a base href to karma debug context
This commits adds a base href value in the karma context iframe used to run unit tests where a unit test throws: No base href set. Please provide a value for the APP_BASE_HREF token or add a base element to the document. Even if the application is fine. This is because the index.html from Angular CLI contains a base href value, but not the Karma context iframe. So when adding a unit test with a testing module that imports a NgModule, for example AppModule, which itself imports RouterModule, the unit test used to throw an error (regression appeared in router 3.1). That could be solved by either adding `RouterTestingModule` to the testing module, or by adding a provider `{ provide: APP_BASE_HREF, useValue: '/' }`, but required to understand the issue. This solves the issue in a transparent way: developers won't even encounter the problem anymore. Closes #19116
1 parent 4d5b9a8 commit ae94245

File tree

1 file changed

+1
-0
lines changed

1 file changed

+1
-0
lines changed

packages/angular_devkit/build_angular/src/webpack/plugins/karma-debug.html

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@
99
<head>
1010
%X_UA_COMPATIBLE%
1111
<title>Karma DEBUG RUNNER</title>
12+
<base href="/">
1213
<link href="favicon.ico" rel="icon" type="image/x-icon" />
1314
<meta http-equiv="Content-Type" content="text/html; charset=UTF-8" />
1415
<meta name="viewport" content="width=device-width, initial-scale=1, maximum-scale=1, user-scalable=no" />

0 commit comments

Comments
 (0)