Fix .addWatchFile() dependencies failing to invalidate in watch mode

[tivac]

This PR contains:

• bugfix
• feature
• refactor
• documentation
• other

Are tests included?

• yes (bugfixes and features will not be merged without tests)
• not yet

Breaking Changes?

• yes (breaking changes will not be merged unless absolutely necessary)
• no

List any relevant issue numbers:

Fixes #3111

Description

Fixes dependency invalidation in watch mode when using this.addWatchFile() from a plugin. Code was refactored in #3081 to use a for...of but the return was kept from the previous .forEach(...) version, so as soon as a module w/o any matching transformDependencies is found iteration halts immediately.

TODO

• Figure out how to write a test for this

[codecov]

Codecov Report

Merging #3112 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master    #3112   +/-   ##
=======================================
  Coverage   89.28%   89.28%           
=======================================
  Files         165      165           
  Lines        5731     5731           
  Branches     1740     1740           
=======================================
  Hits         5117     5117           
  Misses        380      380           
  Partials      234      234

Impacted Files	Coverage Δ
src/watch/index.ts	85.32% <100%> (ø)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ed34bc5...aaba1dd. Read the comment docs.

[lukastaegert]

I added a test using your repro as a basis. It I did not mess up, we can merge this.

[tivac]

Thanks for the test help and the merge @lukastaegert! 👍🏻