build: remove unused gulp unit-test logic

[devversion]

Removes unused gulp unit-test watch logic. Tests are now
executed and watched with Bazel (except on CI which still uses
gulp for BrowserStack and Saucelabs)

Additionally while being at it, removes the gulp default task as the
recommended tasks are no longer available.

[jelbourn]

My problem with ibazel run approach right now is that it re-opens Chrome every time you make a change, and makes it your active window. In my case, it always pops open right over my IDE. It's super annoying and gets in the way of debugging.

The Google version of the rule has a _local suffixed target that doesn't open any browsers.

[devversion]

@jelbourn I'll take a stab on adding a local target. Regarding your issue with ibazel. I don't really understand how this would happen as per current implementation of ibazel and the karma rules, ibazel should just rebuild the tests and re-run in the exisiting browser.

If this is an issue, then I probably didn't see it because for me, tests always run in chromium headless. did you explicitly opt into non-headless mode somehow?

[jelbourn]

Nope, just e.g.

ibazel run src/cdk/coercion:unit_tests

[josephperrott]

This is something that we should probably ask @alexeagle to put on his backlog for providing automatically in the karma_web_test and karma_web_test_suite rules

[devversion]

Closing this one. I'll resubmit this once we have the script for running local tests with Bazel. We have the targets already and it works with ibazel. All the concerns we talked about seem to have been addressed.

[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.}