Closed composerinteralia closed 1 year ago
Looks like we're almost there, but getting an error from spring that it's not able to find the application config:
lib/spring/configuration.rb:31:in 'application_root_path': Spring::MissingApplication (Spring::MissingApplication)
as far as I can see, there is only a couple of tests failing related to Spring. I don't personally use Spring, so I could not care less. can we skip those two? and move forward until someone has the time to investigate?
I'd like to see this merged first and then rails 7 added to the mix to ensure its fully supported.
thanks
I agree it'd be nice to get CI working against Rails 7.
In my experience skipping tests is basically the same as deleting them, and I'd rather not delete the spring tests. Maybe we could merge this with a failing build? That'd still be better than no build at all, which is what we currently have.
Tests passed on Travis, and they pass locally on both MacOS and Linux with all the same versions of Ruby and gems. I have no idea what is going on here! 😢
I'm still totally stumped. In https://github.com/thoughtbot/factory_bot_rails/pull/396/commits/1e0de31f6053ffe2ddc86a7ed029cb20781d6b68 I can see that Spring.project_root_path.join("config/application.rb").exist?
returns true
, and so I wouldn't expect to see this spring error. But once we run bin/rails test
with spring enabled that's exactly the error we hit.
@composerinteralia it could be something like Github Actions does not set a current path
quite unusual to think of a process without $PWD
but it looks like it ...
maybe catch the error in place? fallback to a good default ?
I'm debugging in CI, since that's the only place it's failing. I'm not planning to merge any of these latest commits.
In my latest adventure, I really want to see this full error message: https://github.com/rails/spring/blob/v2.1.0/lib/spring/errors.rb#L25-L31, but I don't know how convince cucumber to show it to me. It's showing me the error class, not the full message.
I'll suggest we move the attempt to solve the spring issue to a later PR, disable the two tests. and hope someone will provide more insight on this issue later.
Spring is not included as a default gem from Rails 7, so I think we can skip tests with Spring. https://github.com/rails/rails/commit/21c9732fae9ce0fff7adf0da610d26eeedfc3076
yes, lets bring back this PR to something simple, and skip the few spring related issues.
Pulled from https://github.com/thoughtbot/factory_bot/blob/23162d4348ed8b316d36091b0fc734c174be75c3/.github/workflows/build.yml
As with factory_bot, we're no longer running against Ruby head, since we don't have a great way to do allowed failures.
Closes #395