fix(@ngtools/webpack): fix Angular 5 lazy routes in es2015

[filipesilva]

Fix #7689

[hansl]

Do we need the changes here? This is only for Angular 5, isn't it?

[filipesilva]

It's still possible to turn off the new compiler (--no-experimental-angular-compiler) so this plugin should still work with Angular 5.

[hansl]

The commit is not about fixing the warning, it's about fixing the resolution of lazy routes in Angular 5, isn't it?

[filipesilva]

Fixed commit message.

[petebacondarwin]

where are the tests? :-P

[hansl]

@petebacondarwin we test against Angular 5.0.0 already, with a lazy route :-)

Actually scratch that. We don't have lazy routes testing in nightly. We should. I'll make a PR for that.

[filipesilva]

@hansl we run this test in nightly, and it tests lazy routes https://github.com/angular/angular-cli/blob/master/tests/e2e/tests/build/rebuild.ts.

[petebacondarwin]

@filipesilva - If you already have tests for this then why were they not failing already? Or perhaps they were, which is what led to this PR?

[filipesilva]

@petebacondarwin because this only failed when lazy routes were being used with es2015. We test the two but separately.

[angular-automatic-lock-bot]

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

_{This action has been performed automatically by a bot.}