fix: text.forEach is not a function

[Ty3uK]

Fixed with code written by Cleod9

[GreenGremlin]

For me, this was happening because I was using webpack-split-by-path and it was pulling css-loader out of the entry chunk. I had to tell webpack-split-by-path to ignore css-loader.

See my fix here: Module build failed: TypeError: text.forEach is not a function #131.

[Ty3uK]

I was using split-by-name-webpack-plugin and with my config webpack pulling to another chunk only angular and it's libraries.

[bebraw]

Can you rebase? This could really use a test.

[jsf-clabot]

All committers have signed the CLA.

[codecov]

Codecov Report

Merging #134 into master will decrease coverage by 0.22%.
The diff coverage is 50%.

@@            Coverage Diff             @@
##           master     #134      +/-   ##
==========================================
- Coverage   90.38%   90.16%   -0.23%     
==========================================
  Files           6        6              
  Lines         364      366       +2     
  Branches       77       78       +1     
==========================================
+ Hits          329      330       +1     
- Misses         35       36       +1

Impacted Files	Coverage Δ
loader.js	89.53% <50%> (-0.95%)	⬇️

---

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 a284f3a...836de37. Read the comment docs.

[bebraw]

Any chance for a test?

[Ty3uK]

Sorry, but i don't understand how i can test empty object into text variable and pass it through Code Coverage. Never worked with this stuff.

[michael-ciniawsky]

@Ty3uK Please add a comment to the code, so I in case this causes any regressions I can revert it :) Could you at least manually test it a bit and report ?

[joshwiens]

Given the amount of effort going into getting testing up to snuff across the organization, i'm not inclined to accept anything that doesn't have a test case. In this case, both a positive & negative test given how clearly defined the issue is.

[michael-ciniawsky]

We are getting close to a new major release and therefore I will close this for now, feel free to open a new PR, if you still get this regression with npm i -D extract-text-webpack-plugin@beta