e2e test should exit with -1 if devServerTarget failed

[robertIsaac]

• I'd be willing to implement this feature (contributing guide)

Description

right now if the devServerTarget of the e2e fails, the e2e stay hanging forever (till it timeout in the CI)
it wastes our resources a lot in our CI, because normally it takes 45 mins to complete, we notice the error usually when it pass 1 hour

Motivation

it will save our resources in the CI

Suggested Implementation

to check if the devServerTarget has errors, if it does to exit with -1 immediately

Alternate Implementations

nothing

additional information

this happen with us because of this bug in Angular angular/angular-cli#27167
there is nothing we could do to make sure the server will actually serve, because we could easily add the build step before the e2e to make sure the code is valid, but it doesn't guarantee the serve will actually work

[robertIsaac]

hi @FrozenPandaz
it seems in native angular setting watch to false will make the server exit, but with nx it doesn't
supporting the same implementation as Angular for watch will fix the problem
minimum reproduce is to create a new nx project using angular, remove something to break the application and run nx serve --no-watch
in native angular it will fail and exit with -1
in nx angular it will fail and stuck

if you don't have the time, just point me to where to make the change and I will create the PR