Skip to content

Fix rspec deprecation warnings#138

Merged
JoeSouthan merged 1 commit intomasterfrom
fix-rspec-deprecation-warnings
Dec 3, 2025
Merged

Fix rspec deprecation warnings#138
JoeSouthan merged 1 commit intomasterfrom
fix-rspec-deprecation-warnings

Conversation

@jamiecobbett
Copy link
Contributor

These tests were generating warnings:

The implicit block expectation syntax is deprecated, you should pass a block rather than an argument to `expect` to use the provided block expectation matcher or the matcher must implement `supports_value_expectations?`. e.g  `expect { value }.to raise Exception with message matching /missing \[:a, :c\]/` not `expect(value).to raise Exception with message matching /missing \[:a, :c\]/`

The implicit block expectation syntax is deprecated, you should pass a block rather than an argument to `expect` to use the provided block expectation matcher or the matcher must implement `supports_value_expectations?`. e.g  `expect { value }.to raise Exception with message matching /missing \[:b\]/` not `expect(value).to raise Exception with message matching /missing \[:b\]/`

These tests were generating warnings:

    The implicit block expectation syntax is deprecated, you should pass a block rather than an argument to `expect` to use the provided block expectation matcher or the matcher must implement `supports_value_expectations?`. e.g  `expect { value }.to raise Exception with message matching /missing \[:a, :c\]/` not `expect(value).to raise Exception with message matching /missing \[:a, :c\]/`

    The implicit block expectation syntax is deprecated, you should pass a block rather than an argument to `expect` to use the provided block expectation matcher or the matcher must implement `supports_value_expectations?`. e.g  `expect { value }.to raise Exception with message matching /missing \[:b\]/` not `expect(value).to raise Exception with message matching /missing \[:b\]/`

I had to mess around with the subject to satisfy rubocop.
@jamiecobbett jamiecobbett force-pushed the fix-rspec-deprecation-warnings branch from 7ed1490 to 99d5c62 Compare July 7, 2025 09:55
@JoeSouthan JoeSouthan merged commit 466fe54 into master Dec 3, 2025
16 checks passed
@jamiecobbett
Copy link
Contributor Author

👴 👍

@JoeSouthan JoeSouthan deleted the fix-rspec-deprecation-warnings branch December 3, 2025 16:03
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