-
Notifications
You must be signed in to change notification settings - Fork 99
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
Browser Support #34
Comments
@dareid - Thanks for opening this issue! Just curious - How did you find out about my talk? For others who are viewing this issue, here are the relevant parts of the talk:
That last link is to my sample repo, which has a demo REST API and the same suite of tests written in four different test frameworks (Chakram, SuperTest, Chai-HTTP, Postman). |
Also... a little back-story: I'm actually the one who added browser support to Chai-HTTP. It was pretty easy, actually. I just used Browserify, and made a few tweaks to the code to handle a few browser-specific differences. I'm sure the same will be true for Chakram. |
Your talk triggered a google alert, was very interesting, thanks for considering Chakram :) |
👍 Cool, glad to help. Here's a subsequent PR that improved the browser build for Chai-HTTP |
…ith browser and fix problems with plugin support. Ideally reduce js size as well. #34
any update for browser support? :) |
It has somewhat stalled tbh. There is a very rough implementation available but there are quite a few problems, mainly with plugins. I have raised this issue chaijs/chai-as-promised#145 which needs fixing before full plugin support can be achieved. In addition, the JS produced is too large (for my liking) and the tests I have configured at the moment are struggling in the browser which needs exploring more. So lots more to do. Any help would be appreciated. |
Just wondering if there is any update on this? |
As suggested by @BigstickCarpet - thanks. Presentation @ https://youtu.be/bhrg-7f2e8k?t=1800
The text was updated successfully, but these errors were encountered: