Skip to content

Conversation

@jleclear
Copy link

@jleclear jleclear commented Oct 2, 2025

No description provided.

amilligan and others added 30 commits October 25, 2021 12:12
Authored-by: Adam Milligan <[email protected]>
Authored-by: Adam Milligan <[email protected]>
This is mostly useful for matchers like have_received that have to be
called after the action and are not called on the subject.  So, e.g.:

  it { should satisfy { expect(foo).to have_received(:wibble) } }

vs.

  it "calls the method" do
    subject.call
    expect(foo).to have_received(:wibble)
  end

Authored-by: Adam Milligan <[email protected]>
Version 2.2.0 changes the module to be included.

Authored-by: Adam Milligan <[email protected]>
Authored-by: Adam Milligan <[email protected]>
Authored-by: Adam Milligan <[email protected]>
Version 1.47 of rubocop has changes to the Lint/MissingSuper cop.  The
new behavior no longer finds an error with the super-less initializer.
This reverts commit 96e86b9.
amilligan and others added 15 commits May 2, 2025 09:14
- Updated ActiveStorage::Service::Configurator to use proper Rails 8 approach
- Removed explicit Rails version constraint for better compatibility
- Updated dependencies to latest versions
- Added new matchers and helpers from upstream
- Convert double quotes to single quotes where appropriate
- Fix indentation and layout issues
- Use Hash#key? instead of Hash#has_key?
- Fix module/class nesting style
- Remove trailing empty lines
- Add supports_block_expectations to send_rollbar_report matcher
- Fixes 'You must pass an argument rather than a block to expect' error
- Allows matcher to work with expect { ... }.to syntax in Rails 8
- Store index calls under :index key instead of :create key
- Fixes tests expecting Doubles::Elasticsearch::Client.calls[:index] to work
- This was causing nil values when tests expected arrays
@jleclear jleclear changed the title Sync with upstream Sync with upstream and Rails 8 changes Oct 2, 2025
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.

3 participants