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

Replace ert with buttercup #5

Open
bbatsov opened this issue Jul 16, 2017 · 3 comments
Open

Replace ert with buttercup #5

bbatsov opened this issue Jul 16, 2017 · 3 comments

Comments

@bbatsov
Copy link
Member

bbatsov commented Jul 16, 2017

This allows you to have much more expressive tests (similar to what you get from jasmine/rspec) and simplifies the travis integration a lot.

https://github.com/jorgenschaefer/emacs-buttercup

The project doesn't have many tests yet, so it's better to do something like this sooner rather than later IMO.

@plexus
Copy link
Collaborator

plexus commented Jul 16, 2017

Interesting. I just introduced Cask + ert-runner for a.el and quite liked the experience. While I quite like rspec, I always found that doing rspec-like things outside of Ruby never really felt right. I'm not a fan of Clojure's expectations for instance.

Could you point to a project that uses buttercup and has CI set up?

@bbatsov
Copy link
Member Author

bbatsov commented Jul 22, 2017

Interesting. I just introduced Cask + ert-runner for a.el and quite liked the experience. While I quite like rspec, I always found that doing rspec-like things outside of Ruby never really felt right. I'm not a fan of Clojure's expectations for instance.

My point is mostly that ert's API is so basic that you end up with some pretty hard to read to tests, especially if they are non-trivial. And things like mocking are not even supported - you have to resort to things like no-flet quite often with it.

Could you point to a project that uses buttercup and has CI set up?

CIDER uses it. The switch simplified both the CI setup and the tests themselves.

@MalloZup
Copy link

MalloZup commented Aug 5, 2019

autogenerated with https://github.com/MalloZup/doghub: issue inactive since 450 days. Please update the issue or close it

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

No branches or pull requests

3 participants