Apply correct nesting structure for nested tests

[vinistock]

Motivation

We were creating a flat hierarchy for nested tests, but that's not very accurate. We need to ensure that test items are nested similarly to tests, so that if you run the outer group, it works as expected.

For example

class MyTest < Minitest::Test
  class OuterGroup < Minitest::Test
    class InnerGroup < Minitest::Test
      def test_something; end
    end
  end
end

In this case, running OuterGroup should run the InnerGroup as well.

Implementation

I started keeping a flat stack of parents, so that we can always add the items to the appropriate nesting. I used nil to represent classes that are not tests, so that the logic for popping the stack can be simpler.

Automated Tests

Added tests.

[vinistock]

• Apply correct nesting structure for nested tests #3497 👈 (View in Graphite)
• main

---

How to use the Graphite Merge Queue

Add the label graphite-merge to this PR to add it to the merge queue.

You must have a Graphite account in order to use the merge queue. Sign up using this link.

An organization admin has enabled the Graphite Merge Queue in this repository.

Please do not merge from GitHub as this will restart CI on PRs being processed by the merge queue.

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