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

Replace Rake's testtask with Minitest's #257

Closed
wants to merge 2 commits into from

Conversation

teoljungberg
Copy link
Contributor

This is available since Minitest v5.16.0 and the following commit:

minitest/minitest@73692f9

This is available since Minitest v5.16.0 and the following commit:

minitest/minitest@73692f9
@teoljungberg
Copy link
Contributor Author

I could add a version lock to the gemfile to denote that we have an expected version of minitest. But since there are no other version constraints I wanted to ask first.

@teoljungberg
Copy link
Contributor Author

We are also not running bundler on CI, should that be done first as a separate PR?

@casperisfine
Copy link
Contributor

What is supposed to be the benefit?

@teoljungberg
Copy link
Contributor Author

@casperisfine

What is supposed to be the benefit?

Minitest's test task has an easier time passing in options to minitest itself, compared to rake's task.

It also generates more tasks to get the slowests tests etc, if that is of interest in a future refactoring.

That being said - they have the same output which is running tests. I don't mind if we decide to ditch this in favor of rake's test task.

@teoljungberg
Copy link
Contributor Author

As for the "we are not running bundler on CI" - that means we won't get access to tools such as #253 on CI.

But that's a separate issue that I can push a PR for.

@teoljungberg
Copy link
Contributor Author

It looks like bundler is run on CI, as per:

#258

But then the cache on CI needs to be bumped due to this upgrade change and we do not have that new version of minitest on the CI runs.

@teoljungberg teoljungberg mentioned this pull request Feb 7, 2023
@teoljungberg
Copy link
Contributor Author

It's not, minitest 5.16 (where the new test task is introduced) does not support ruby 2.5 as per the CI configuration.

conflicting requirements for the Ruby version:
    In Gemfile:
      minitest (~> 5.16) was resolved to 5.17.0, which depends on
        Ruby (< 4.0, >= 2.6)
  
      zeitwerk was resolved to 2.6.6, which depends on
        Ruby (>= 2.5)
  
    Current Ruby version:
      Ruby (= 2.5.9)

Therefore with the current Ruby 2.5 and above CI policy - this is not a doable addition.

@teoljungberg teoljungberg deleted the use-minitests-test-task branch February 7, 2023 20:10
@fxn
Copy link
Owner

fxn commented Feb 7, 2023

Thanks anyway for looking into this!

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