Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrade webpack to 2.2 #24

Closed
wants to merge 1 commit into from
Closed

Upgrade webpack to 2.2 #24

wants to merge 1 commit into from

Conversation

RSO
Copy link
Contributor

@RSO RSO commented Feb 7, 2017

This PR is pretty much the result of me banging my head against the wall while trying to get the tests running. All tests are supposed to show up green now!

Seeing how this plugin seems to be pretty much set in terms of what it does, I figured a major release would be the way to go., unless someone can figure out a pretty way to support both versions of webpack next to each other.

Should fix #22

@rtibbles
Copy link

rtibbles commented Feb 8, 2017

This saved my build - gets my vote!

@RSO
Copy link
Contributor Author

RSO commented Feb 16, 2017

NOTE: I just discovered a strange problem in our own testsuite on Webpack2.2. It seems that second/third runs don't properly rewire resources. I have no clue what is going on, and we decided to actually move away from rewire, so I won't be able to update this PR. I hope that somebody else will be able to figure out what's going on!

@RSO RSO closed this Feb 16, 2017
@nadavsinai
Copy link

@RSO thanks for your work, I'm curious what are you using instead of rewire?

@RSO
Copy link
Contributor Author

RSO commented Mar 6, 2017

Not using anything anymore, we've decided that unit-tests like this add less value to our codebase in comparison to the pain they add. In some cases we found other ways around rewire, for instance in cases where we had to mock window.location =, we decide to create a redirectHelper that actually returns a sinon.spy() when called in the test environment.

In other cases we decided that mocking was just simply not the way to go.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Not working with Webpack 2 beta
3 participants