Skip to content
This repository has been archived by the owner on May 11, 2022. It is now read-only.

Change to a static config and drop confstruct #630

Merged
merged 1 commit into from
May 28, 2019
Merged

Conversation

jcoyne
Copy link
Contributor

@jcoyne jcoyne commented May 22, 2019

Confstruct is not well maintained and it's 'accept anything' API can lead to misconfiguration

@jcoyne jcoyne force-pushed the static_config branch 2 times, most recently from 1042e93 to 0c64a3b Compare May 22, 2019 15:17
@coveralls
Copy link

coveralls commented May 22, 2019

Coverage Status

Coverage decreased (-0.001%) to 90.529% when pulling fcaa1f7 on static_config into c095bac on master.

Confstruct is not well maintained and it's 'accept anything' API can lead to misconfiguration
allow(ActiveFedora).to receive(:fedora).and_return(double('frepo').as_null_object) # must be used in per-request context: :each not :all
end

def unstub_config
Dor::Config.pop!
puts 'this does nothning'
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

typo: nothning

@mjgiarlo mjgiarlo merged commit b503f69 into master May 28, 2019
@mjgiarlo mjgiarlo deleted the static_config branch May 28, 2019 20:42
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants