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

Convert from mocha to rspec-mocks #158

Merged
merged 1 commit into from
Jan 18, 2021

Conversation

ekohl
Copy link
Member

@ekohl ekohl commented Aug 27, 2020

This is recommended by puppetlabs_spec_helper and avoids a deprecation warning.

@ekohl ekohl force-pushed the convert-to-rspec-mocks branch from afbadc4 to d91e1c5 Compare August 27, 2020 16:38
This is recommended by puppetlabs_spec_helper and avoids a deprecation
warning. bbed0b8 already started the
conversion, but didn't update the existing fact tests. Because the
mock_with call was incorrect (needs to be done before require), this
didn't show.
@ekohl ekohl force-pushed the convert-to-rspec-mocks branch from d91e1c5 to 8d0dff0 Compare August 27, 2020 18:00
@raphink raphink merged commit 8a84db5 into voxpupuli:master Jan 18, 2021
@raphink raphink added enhancement New feature or request maintenance labels Jan 18, 2021
@ekohl ekohl deleted the convert-to-rspec-mocks branch April 26, 2021 15:49
simondeziel pushed a commit to simondeziel/puppet-systemd that referenced this pull request Jun 16, 2021
This is recommended by puppetlabs_spec_helper and avoids a deprecation
warning. bbed0b8 already started the
conversion, but didn't update the existing fact tests. Because the
mock_with call was incorrect (needs to be done before require), this
didn't show.
simondeziel pushed a commit to simondeziel/puppet-systemd that referenced this pull request Jun 16, 2021
This is recommended by puppetlabs_spec_helper and avoids a deprecation
warning. bbed0b8 already started the
conversion, but didn't update the existing fact tests. Because the
mock_with call was incorrect (needs to be done before require), this
didn't show.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request maintenance
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants