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

Adds support for cljs testing and ->styled tests #26

Merged
merged 1 commit into from
Oct 12, 2017
Merged

Adds support for cljs testing and ->styled tests #26

merged 1 commit into from
Oct 12, 2017

Conversation

brianium
Copy link
Contributor

@roman01la sorry for the delay - I've been busy

This is my first crack at introducing an environment for testing ClojureScript aspects of cljss.

It includes tests for ->styled to start, and includes a lein test-all task for running all tests, as well as a doo setup to test ClojureScript.

I am attempting to get phantomjs installed via Circle CI as well.

Please let me know if you have comments/questions :)

@brianium
Copy link
Contributor Author

brianium commented Oct 10, 2017

It looks like I can't see what is happening in Circle CI - so I can't tell if what I did works - any help you can offer would be welcome :)

@roman01la
Copy link
Collaborator

@brianium Have you seen headless Chrome? It seems like a good new replacement for PhantomJS https://github.com/GoogleChrome/puppeteer

@roman01la roman01la merged commit b72eea1 into clj-commons:master Oct 12, 2017
@roman01la
Copy link
Collaborator

All tests are passed! :) CI link https://circleci.com/gh/roman01la/cljss

@brianium
Copy link
Contributor Author

@roman01la awesome! thank you! I actually used headless Chrome and puppeteer this week for a small application. Definitely an awesome tool 👍

For the time being - it looks like we are at the mercy of the doo library. In terms of headless browsers, it looks like it only supports phantomjs, slimer, and has some alpha support for electron.

@brianium brianium deleted the styled-tests branch October 12, 2017 10:23
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.

2 participants