Skip to content

Fix Rails 7.1 fixture_path deprecation warning - #499

Open
MattRogish wants to merge 1 commit into
collectiveidea:mainfrom
MattRogish:fix-feature-path-deprecation
Open

MattRogish wants to merge 1 commit into
collectiveidea:mainfrom
MattRogish:fix-feature-path-deprecation

Conversation

@MattRogish

Copy link
Copy Markdown
Contributor

On a clean checkout, when running bundle exec spec I got the following:

Deprecation Warnings:

Rails 7.1 has deprecated the singular fixture_path in favour of an array.You should migrate to plural:

If you need more of the backtrace for any of these deprecations to
identify where to make the necessary changes, you can configure
config.raise_errors_for_deprecations!, and it will turn the
deprecation warnings into errors, giving you the full backtrace.

1 deprecation warning total

I made fixture_paths= / fall back to fixture_path=. rspec has officially made this change upstream, so presumably this warning will persist and/or this functionality will break at some point in the future?

Use `fixture_paths=` (plural, array) on Rails 7.1+ and fall back to
`fixture_path=` (singular) on older versions.
@MattRogish

Copy link
Copy Markdown
Contributor Author

I guess maybe the right thing to do is bump rspec and see what happens but that feels like a decision above my pay grade, hence the duct-tape patch above.

@parndt

parndt commented Apr 27, 2026

Copy link
Copy Markdown
Collaborator

I think upgrading rspec sounds like a good idea if you're willing to give it a go 😄

@stale

stale Bot commented Jun 28, 2026

Copy link
Copy Markdown

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale Bot added the stale label Jun 28, 2026
@parndt parndt removed the stale label Jun 29, 2026

This branch has not been deployed

No deployments
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