Watch mode overwrites the test file about karma-rollup-preprocessor HOT 9 CLOSED

@jlmakes friendly ping!

from karma-rollup-preprocessor.

Hey! I wanted to leave a note that I was seeing the same behavior of an endless loop of re-writing and watching the tests file when autoWatch: true, singleRun: false. I also implemented a quick fix of downgrading to 7.0.7 and am investigating locally.

from karma-rollup-preprocessor.

I've looked into this one and here is what I found.

• The changes made for #72 introduced this issue.

• rollup.watch is used instead of rollup.rollup. rollup.watch also writes directly to the file system, resulting in this issue.

• I think that the root cause of #72 is in @rollup/plugin-typescript when using rollup.rollup (i.e. watchMode is false)

• I created an issue in @rollup/plugin-typescript
  rollup/plugins#1354

• I also created a pull request in @rollup/plugin-typescript
  rollup/plugins#1356.

Once rollup/plugins#1354 is fixed and released, the fix for #72 could be reverted and everything should work as expected when using the official Typescript plugin.

from karma-rollup-preprocessor.

@jlmakes would you consider releasing a new version with this fix? The current v7.0.8 has this issue...

from karma-rollup-preprocessor.

@jlmakes any chance you could have a look please? :)

from karma-rollup-preprocessor.

We hit this issue as well and downgrading resolved the problem. (Internal project, no link to share)

from karma-rollup-preprocessor.

I see, that's no good! Roping @YonatanKra into this conversation. Let's see if we can get this fixed, thanks for the report.

from karma-rollup-preprocessor.

The pull request for @rollup/plugin-typescript was approved and released as part of version 10.0.1.

I created a pull request #76 to revert the changes that introduced this issue and added a basic test case.

@jlmakes

from karma-rollup-preprocessor.

@jlmakes LMK if you think the change can be integrated

from karma-rollup-preprocessor.