Disable Minitest::Reporters for RubyMine

[amomchilov]

Workaround for this issue.

[andyw8]

I think this the git stuff is better suited for people's personal ~/.gitignore.

[paracycle]

I think we'd rather just ignore the whole .idea folder

[amomchilov]

Do you think I should do that here at all, or leave it to each dev to configure as @andyw8 suggested?

[andyw8]

(this is also what's recommended on https://git-scm.com/docs/gitignore#:~:text=Patterns%20which%20a,is%20used%20instead. )

[paracycle]

I'd rather not special case RubyMine here. I'd be open to changing the default reporter, though, since our custom reporter doesn't work that well.

[amomchilov]

This is the only workaround for now, unless we don't use Minitest::Reporters at all. :(

https://blog.jetbrains.com/ruby/2021/04/improved-minitest-support-action-required/

This is a really poor DX IMO, so I opened an issue, hopefully they can address it: https://youtrack.jetbrains.com/issue/RUBY-33007/Using-Minitest-should-not-require-changes-to-testhelper.rb

[paracycle]

Let's try not using it at all, and see what happens?

[amomchilov]

Left is without, right is with.

I see 2 slight benefits to using this SpecReporter:

1. The passing tests are summarized more tersely with simple green dots (though with less detail, like the name and runtime)
2. There's a backtrace filter applied

The stock one is more detailed, and perhaps better overall, but that's subjective

[paracycle]

Can we make this to be just:

Suggested change

	unless ENV["RM_INFO"]
	Minitest::Reporters.use!(SpecReporter.new(color: true), ENV, backtrace_filter)
	end
	Minitest.backtrace_filter = backtrace_filter

[amomchilov]

Can you explain the reasoning? Do we not want to use SpecReporter?

[paracycle]

Yes, that's what I've been trying to say in the other thread:

I'd be open to changing the default reporter, though, since our custom reporter doesn't work that well.

and

Let's try not using it at all, and see what happens?