-
Notifications
You must be signed in to change notification settings - Fork 25
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
Development Environment Configuration #204
Conversation
tpendragon
commented
Jul 30, 2019
- Fix Solr configuration to use 7.x locally (like CircleCI)
- Add documentation on dependencies to run tests locally.
- Fix periodically failing spec.
- Add local coverage generation.
6ed4564
to
e3cd440
Compare
README.md
Outdated
## Dependencies | ||
|
||
* [ImageMagick](https://www.imagemagick.org) | ||
* On a mac, do `brew install imagemagick --with-little-cms2 --with-openjpeg` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As of Homebrew 2.0.0, homebrew-core accepts no options and uses both of these by default.
spec/units/derivatives_spec.rb
Outdated
@@ -40,6 +40,12 @@ class CustomFile < ActiveFedora::Base | |||
end | |||
end | |||
describe "#{method}=" do | |||
around do |obj| |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Isn't this save/restore usually done as follows?
around(:each) do |example|
..
example.run
..
end
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
example.run
is definitely better. Rubocop complains about (:each)
.
Following the updated README (without imagemagick options) gave me a fully passing suite. Once that's addressed, 🚢. |
Old cache had some bad bundled things in it.
e3cd440
to
1de44f8
Compare
@botimer Sorry, this should be ready now. Was lost in coverage fixing. |