Snowpack: Help wanted: create tests for internal plugins

[gr2m]

Original Discussion: https://github.com/pikapkg/snowpack/discussions/958#discussioncomment-72307
/cc @FredKSchott @drwpow @ralphtheninja

Snowpack's internal plugins do not currently have good test coverage. We would like to change that 😄

If anyone would like to contribute to Snowpack, comment which one of the plugins below you would like to claim, and create tests similar to what @Akimyou did in https://github.com/pikapkg/snowpack/pull/984. The tests do not need to be complete, we can iterate. Few tests are better than none :)

• [x] [babel-plugin-package-import](https://github.com/pikapkg/snowpack/tree/master/plugins/babel-plugin-package-import)
• [x] [plugin-babel](https://github.com/pikapkg/snowpack/tree/master/plugins/plugin-babel) (@JennieJi #1097 #1199)
• [x] [plugin-build-script](https://github.com/pikapkg/snowpack/tree/master/plugins/plugin-build-script) (@gr2m #1301)
• [x] [plugin-dotenv](https://github.com/pikapkg/snowpack/tree/master/plugins/plugin-dotenv) (@MoonBall #1087)
• [x] [plugin-optimize](https://github.com/pikapkg/snowpack/tree/master/plugins/plugin-optimize) (@gr2m #1300)
• [X] ~plugin-parcel~ No longer supported
• [ ] [plugin-postcss](https://github.com/pikapkg/snowpack/tree/master/plugins/plugin-postcss) (claimed by @thescripted https://github.com/pikapkg/snowpack/issues/1019#issuecomment-704401873)
• [x] [plugin-react-refresh](https://github.com/pikapkg/snowpack/tree/master/plugins/plugin-react-refresh) (@MoonBall https://github.com/snowpackjs/snowpack/pull/1319)
• [X] [plugin-run-script](https://github.com/pikapkg/snowpack/tree/master/plugins/plugin-run-script) (claimed by @FredKSchott https://github.com/pikapkg/snowpack/issues/1019#issuecomment-695882081)
• [x] [plugin-svelte](https://github.com/pikapkg/snowpack/tree/master/plugins/plugin-svelte) (#1241)
• [x] [plugin-vue](https://github.com/pikapkg/snowpack/tree/master/plugins/plugin-vue) (#984)
• [x] [plugin-webpack](https://github.com/pikapkg/snowpack/tree/master/plugins/plugin-webpack) (claimed by @gr2m, #1061)

Once all plugins are tested, close issue and update https://github.com/pikapkg/snowpack/discussions/958#discussioncomment-72307

[MoonBall]

I would like to add test for plugin-webpack.

Edit: @ralphtheninja started this work in https://github.com/pikapkg/snowpack/issues/1012#issuecomment-689182988. So I will contribute to plugin-dotenv.

[gr2m]

@ralphtheninja just to confirm, are you working on test for plugin-webpack?

[ralphtheninja]

@gr2m Nope, I never got started on it.

[gr2m]

no worries, @MoonBall would you like to work on it? If not I'd do it myself, I think it's the most important one to have tests right now

[MoonBall]

@gr2m I started plugin-dotenv. So you do it.

[JennieJi]

Hi @gr2m I would like to try plugin-babel

[gr2m]

Hi @JennieJi, I've signed you up. Thanks for looking into it 👍🏼

[MoonBall]

@gr2m I would like to work on plugin-react-refresh.

[gr2m]

You got it, thank you!

[FredKSchott]

Woo! This is exciting to see! @gr2m got one that I can take a look at? Maybe plugin-run-script & plugin-build-script?

[gr2m]

Sure, just update the issue description for the one you are working on

[thescripted]

I feel pretty comfortable working on tests for plugin-postcss

[gr2m]

Great, all yours @thescripted!

[drwpow]

Just added a test for plugin-svelte in #1241!

[gr2m]

@MoonBall just checking in, no rush: are you still working on / planning to work on plugin-react-refresh?

[gr2m]

I'll work on the last unclaimed one: plugin-optimize.

Wonderful to see this all come together, thanks a bunch everyone 💐

[gr2m]

actually plugin-build-script is up for grabs again ☝🏼

[FredKSchott]

Yup, anyone feel free to grab it! It should be pretty similar to the run-script tests

[gr2m]

I'm looking into plugin-build-script myself now

[gr2m]

I'll close the issue it's no longer actionable for people looking to contribut.

@thescripted @MoonBall let us know in case you won't be able to submit pull requests for the respective tests

[FredKSchott]

woo! awesome job driving this @gr2m!