Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(@angular-devkit/build-angular): add a base href to karma debug context #19119

Merged

Commits on Oct 19, 2020

  1. fix(@angular-devkit/build-angular): add a base href to karma debug co…

    …ntext
    
    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 angular#19116
    IxquitilisSaid committed Oct 19, 2020
    Configuration menu
    Copy the full SHA
    24854df View commit details
    Browse the repository at this point in the history