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

DEPRECATION WARNING: TestFixtures#fixture_path is deprecated and will be removed in Rails 7.2 #2661

Closed
jasnow opened this issue Mar 19, 2023 · 6 comments · Fixed by #2664
Closed

Comments

@jasnow
Copy link
Contributor

jasnow commented Mar 19, 2023

FYI: Getting

DEPRECATION WARNING: TestFixtures#fixture_path is deprecated 
and will be removed in Rails 7.2. Use #fixture_paths instead. (called from block in 
<module:FixtureSupport> at /usr/local/rvm/gems/ruby-3.2.1@ruby3.2-railsedge/
gems/rspec-rails-6.0.1/lib/rspec/rails/fixture_support.rb:24)

while running "rake" command.

@pirj
Copy link
Member

pirj commented Mar 19, 2023

Would you like to send a PR to fix that, @jasnow ?

@jasnow
Copy link
Contributor Author

jasnow commented Mar 19, 2023

I will be patient.

@jasnow jasnow closed this as completed Mar 19, 2023
@JonRowe JonRowe reopened this Mar 19, 2023
@jasnow
Copy link
Contributor Author

jasnow commented Mar 20, 2023

Example: Run "rake" on this repo: https://github.com/jasnow/tasks_edge

@pirj
Copy link
Member

pirj commented Mar 20, 2023

I really meant if you like, please send a PR to fix this, @jasnow . No hidden meaning.

@jasnow
Copy link
Contributor Author

jasnow commented Mar 20, 2023

This newsletter mentioned this deprecation: https://world.hey.com/this.week.in.rails/testfixtures-fixture_path-deprecation-findermethods-find-support-for-composite-primary-key-values-87e6e69a

@JonRowe
Copy link
Member

JonRowe commented Mar 21, 2023

Thanks for reporting, we don't need any further citation, just someone to find the time to fix it, I reopened so people can see this is an unfixed issue we'd like someone to tackle.

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 a pull request may close this issue.

3 participants