Skip to content

Restructure test classes for better RubyMine compatibility#912

Open
juniper-shopify wants to merge 1 commit into
mainfrom
juni/Restructure_test_classes_for_better_RubyMine_compatibility
Open

Restructure test classes for better RubyMine compatibility#912
juniper-shopify wants to merge 1 commit into
mainfrom
juni/Restructure_test_classes_for_better_RubyMine_compatibility

Conversation

@juniper-shopify
Copy link
Copy Markdown
Contributor

@juniper-shopify juniper-shopify commented Jun 2, 2026

Test classes defined like this

module Roast
  module Cogs
    class Agent < Cog
      class ConfigTest < ActiveSupport::TestCase
        ...

which re-open the Agent class (it's a class, not a module) result in some undesirable output from the RubyMine test runner.

Restructuring test classes to use the short-hand class name-spacing notation:

module Roast
  module Cogs
    class Agent::ConfigTest < ActiveSupport::TestCase
      ...

resolves the problem, since the class is not re-opened.

Copy link
Copy Markdown
Contributor Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@juniper-shopify juniper-shopify marked this pull request as ready for review June 2, 2026 20:42
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.

1 participant