-
Notifications
You must be signed in to change notification settings - Fork 62
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
Mocha Testing #59
Comments
Thank you! I will try to fix this when I get a bit of time this weekend. |
Please let me know when you have it fixed. The fix you sent me doesn't seem to work. Thanks. |
I have the same error. The odd part is that it worked for a while and then seems to have failed. I'm unsure why. |
@lawrencetvo, @iggi- import Multiselect from 'react-bootstrap-multiselect' try let Multiselect;
if (typeof(window) !== 'undefined') {
Multiselect = require('react-bootstrap-multiselect');
} idk, seems this repository no longer maintained, I probably will use this alternative: https://jedwatson.github.io/react-select/ |
I should definitely keep a list of alternatives in the readme. I keep meaning to update this repo one weekend, but have been so busy lately. There are definitely some good pure react libs out there (this one is just a wrapper around an existing jQuery library). I will try to update this lib soon though (and add some unit tests). |
When I run test cases got error from react-bootstrap-multiselect. I am using enzyme, mocha and chai.
` |
Issue is fixed when I add |
@lawrencetvo Could you please look into #80? |
When I try to test the npm module
react-bootstrap-multiselect
with mocha and enzyme, I get the following error:where the error is thrown when I try to import MasterTable. MasterTable tries to render the react-bootstrap-multiselect.
Link to StackOverflow post http://stackoverflow.com/questions/40406149/testing-react-bootstrap-multiselect-with-mocha
The text was updated successfully, but these errors were encountered: