minitest-heat 2.1.0 → 2.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 854fe8325934b81a7dd330ce67af14e20789a9233f33296bf3bd9b794c4dac20
-  data.tar.gz: 835c33a68c9efc7f92a9baca779b3844310a74e527f6553f4f12be2967d0691a
+  metadata.gz: de79ea86107b9d5183d3353e2c4cf6b8f5f61601080e781c611f305a4e192afd
+  data.tar.gz: b3deddb67ef5fd2b069d2f3fe3edd754cb91df9322390dfdbe7845cab4ab73a0
 SHA512:
-  metadata.gz: 030dace3e842475ef5dbeb7c108909ec1faed7d4cc0a2a9219704992908a29d2c6320d21e66fce4229960aea2f4538ab33af493bd10d3f6e70e3fcc0910c0504
-  data.tar.gz: 828f273f9e7961f17c4b3a6d35ba6fa4fd020d3075e600e656bbebbb9d8975990e13d6536aec5eb2defb5d2784d4786d3988aa264a27c449230f2723eeedb363
+  metadata.gz: ce4bf462ee43dba7da46d8455caecb242f97c03a596af283c3627d50236562eca481bc6a1fbffa025b48ecfb91692c2d2a527a9a1c55603e241335256aa20b27
+  data.tar.gz: d80070c1d120c17a4fb700980b0107b506c3046ac19463e5e89e9fa0e8ca0e3b614adde6426f1a001c6ef1492df6ce3c33af9a11d95cd63495be5eb064e26808

data/.flayignore

@@ -0,0 +1 @@
+./test/**/*.rb

data/.github/workflows/main.yml

@@ -27,11 +27,21 @@ jobs:
       run: bundle exec bundle-audit check
 
   test:
-    name: Test (Ruby ${{ matrix.ruby }})
+    name: Test (Ruby ${{ matrix.ruby }}, Minitest ${{ matrix.minitest }})
     strategy:
       fail-fast: false
       matrix:
         ruby: ['3.2', '3.3', '3.4', '4.0']
+        minitest: ['~> 5.0', '~> 6.0']
+        include:
+        - ruby: '3.2'
+          minitest: '= 5.22.3'
+        - ruby: '3.2'
+          minitest: '= 6.0.0'
+    env:
+      MINITEST_VERSION: ${{ matrix.minitest }}
+      BUNDLE_FROZEN: 'false'
+      BUNDLE_DEPLOYMENT: 'false'
     runs-on: ubuntu-latest
     steps:
     - uses: actions/checkout@v4
@@ -40,10 +50,37 @@ jobs:
       with:
         ruby-version: ${{ matrix.ruby }}
         bundler: '2.7.2'
-        bundler-cache: true
+        bundler-cache: false
+    - name: Resolve selected Minitest
+      run: |
+        bundle lock --update minitest
+        bundle install
+    - name: Verify selected Minitest
+      run: |
+        bundle exec ruby -rminitest -e '
+          version = Gem.loaded_specs.fetch("minitest").version
+          abort "Wrong Minitest: #{version}" unless Gem::Requirement.new(ENV.fetch("MINITEST_VERSION")).satisfied_by?(version)
+          puts "Ruby #{RUBY_VERSION}, Minitest #{version}"
+        '
     - name: Run tests
       run: bundle exec rake test
 
+  required-tests:
+    # Keep the names required by the repository rules while gating on every
+    # supported Ruby/Minitest combination, including the version boundaries.
+    name: Test (Ruby ${{ matrix.ruby }})
+    needs: test
+    if: ${{ always() }}
+    strategy:
+      matrix:
+        ruby: ['3.2', '3.3', '3.4', '4.0']
+    runs-on: ubuntu-latest
+    env:
+      TEST_RESULT: ${{ needs.test.result }}
+    steps:
+    - name: Require the complete compatibility matrix to pass
+      run: test "$TEST_RESULT" = success
+
   changelog:
     name: Changelog
     runs-on: ubuntu-latest

data/.reek.yml

@@ -0,0 +1,87 @@
+# Reek configuration for minitest-heat
+# https://github.com/troessner/reek/blob/master/docs/defaults.reek.yml
+
+detectors:
+  # --- Disabled: conflicts with standard Ruby patterns ---
+
+  # Flags attr_accessor as a writable attribute. Standard Ruby; no value suppressing.
+  Attribute:
+    enabled: false
+
+  # Flags keyword arguments like `clear_screen: false`. Idiomatic Ruby.
+  BooleanParameter:
+    enabled: false
+
+  # Flags branching on method arguments. Appropriate for configuration and options.
+  ControlParameter:
+    enabled: false
+
+  # Issue#initialize wraps Minitest::Result data — all parameters are necessary.
+  # Map#add takes pathname, line_number, type, and backtrace — all distinct inputs.
+  LongParameterList:
+    max_params: 4
+    exclude:
+      - 'Minitest::Heat::Issue#initialize'
+
+  # --- Threshold adjustments: defaults are too aggressive ---
+
+  # Default max_calls (1) flags every repeated method call. Raise to 3 so only
+  # genuine duplication is flagged, not incidental reuse.
+  DuplicateMethodCall:
+    max_calls: 3
+
+  # Guard clauses like `file_exists?` and mode checks like `json_output?` are
+  # repeated across public methods by design, not duplication.
+  RepeatedConditional:
+    enabled: false
+
+  # Default max_statements (5) flags nearly every non-trivial method. 12 keeps
+  # the check useful while allowing token-building methods that are linear and clear.
+  TooManyStatements:
+    max_statements: 12
+
+  # Location is a comprehensive value object with many accessors.
+  # 30 keeps the check useful for genuinely bloated classes.
+  TooManyMethods:
+    max_methods: 30
+
+  # Flags nil checks (`nil?`, `!x`). Legitimate for optional settings and
+  # backtrace data where nil is a normal value.
+  NilCheck:
+    enabled: false
+
+  # Output classes format data from domain objects — accessing their attributes
+  # is the entire purpose, not a design flaw.
+  FeatureEnvy:
+    enabled: false
+
+  # Flags private methods that don't reference instance state. These methods
+  # logically belong to their class even without direct instance access.
+  UtilityFunction:
+    enabled: false
+
+  # ManualDispatch is legitimate for duck-typing checks (respond_to? + send).
+  ManualDispatch:
+    enabled: false
+
+  # Nested iterators are common in token-building code where lines contain tokens.
+  # Two levels deep is readable and appropriate for this codebase.
+  NestedIterators:
+    max_allowed_nesting: 2
+
+  # Issue wraps a Minitest::Result with classification; 9 ivars reflect the data it needs.
+  # Source has 5 ivars for file reading context — all necessary.
+  TooManyInstanceVariables:
+    max_instance_variables: 9
+
+  # Timer.start! and Timer.stop! are mutating operations with no safe equivalent.
+  # The bang convention here signals state mutation, not a safe/unsafe pair.
+  MissingSafeMethod:
+    enabled: false
+
+  # RuboCop enforces `e` for rescue variables (Naming/RescuedExceptionsVariableName).
+  # Allow `e` to avoid conflicts between the two tools.
+  UncommunicativeVariableName:
+    accept:
+      - e
+      - i

data/.reviewer.yml

@@ -0,0 +1,87 @@
+---
+# Review gem dependencies for security issues
+bundle_audit:
+  name: Bundle Audit
+  description: Review gem dependencies for security issues
+  tags: [security, dependencies, ruby]
+  commands:
+    install: bundle exec gem install bundler-audit
+    prepare: bundle exec bundle-audit update
+    review: bundle exec bundle-audit check --no-update
+
+# Review Ruby syntax and formatting for consistency
+rubocop:
+  name: RuboCop
+  description: Review Ruby syntax and formatting for consistency
+  tags: [ruby, syntax]
+  commands:
+    install: bundle exec gem install rubocop
+    review: bundle exec rubocop --parallel
+    format: bundle exec rubocop --auto-correct
+  files:
+    flag: ''
+    separator: ' '
+    pattern: '*.rb'
+
+# Examine Ruby classes for code smells
+reek:
+  disabled: true
+  name: Reek
+  description: Examine Ruby classes for code smells
+  tags: [ruby, quality]
+  links:
+    home: https://github.com/troessner/reek
+    install: https://github.com/troessner/reek#quickstart
+  max_exit_status: 2
+  commands:
+    install: bundle exec gem install reek
+    review: bundle exec reek --config .reek.yml lib/
+  files:
+    flag: ''
+    separator: ' '
+    pattern: '*.rb'
+
+# Reports most tortured ruby code in a pain report
+flog:
+  disabled: true
+  name: Flog
+  description: Reports your most tortured ruby code in an easy to read pain report.
+  tags: [ruby, quality]
+  links:
+    home: https://ruby.sadi.st/Flog.html
+    install: https://ruby.sadi.st/Flog.html
+  commands:
+    install: bundle exec gem install flog
+    review: bundle exec flog -g lib
+  flags:
+    threshold: 10
+    group:
+    methods-only:
+
+# Review ruby code for structural similarities
+flay:
+  disabled: true
+  name: Flay
+  description: Review ruby code for structural similarities and refactoring opportunities.
+  tags: [ruby, quality]
+  links:
+    home: https://ruby.sadi.st/Flay.html
+    install: https://ruby.sadi.st/Flay.html
+  commands:
+    install: bundle exec gem install flay
+    review: bundle exec flay ./lib
+  flags:
+    liberal:
+    summary:
+
+# Unit tests and coverage
+tests:
+  name: Minitest
+  description: Unit tests and coverage
+  tags: [ruby, tests]
+  commands:
+    review: bundle exec rake test
+  files:
+    review: bundle exec ruby -Itest
+    pattern: '*_test.rb'
+    map_to_tests: minitest

data/.reviewer_history.yml

@@ -0,0 +1,42 @@
+---
+:bundle_audit:
+  :last_status: :passed
+  :last_prepared_at: '2026-02-01 10:15:13 -0700'
+  bundle exec bundle-audit update:
+  - 0.94
+  bundle exec bundle-audit check --no-update:
+  - 0.2
+  - 0.15
+  - 0.23
+  - 0.21
+  - 0.29
+  :last_failed_files:
+:rubocop:
+  :last_status: :passed
+  :last_failed_files:
+  bundle exec rubocop --parallel:
+  - 1.4
+  - 1.11
+  - 1.76
+:tests:
+  :last_status: :passed
+  :last_failed_files:
+  bundle exec rake test:
+  - 0.54
+  - 0.5
+  - 0.66
+:reek:
+  :last_status: :passed
+  :last_failed_files:
+  bundle exec reek --config .reek.yml lib/:
+  - 0.91
+:flog:
+  :last_status: :passed
+  :last_failed_files:
+  bundle exec flog -g lib --threshold 10 --group --methods-only:
+  - 0.38
+:flay:
+  :last_status: :passed
+  :last_failed_files:
+  bundle exec flay ./lib --liberal --summary:
+  - 0.36

data/.rubocop.yml

@@ -7,14 +7,26 @@ AllCops:
   Exclude:
     - 'bin/**/*'
     - 'test/files/source.rb' # An example test file for reading source code
+    - 'test/minitest/contrived_*' # Intentional failures for testing the reporter
 
 # Let's aim for 80, but we don't need to be nagged if we judiciously go over.
 Layout/LineLength:
   Enabled: false
 
-# One case statement in a single method isn't complex.
+# One case statement in a single method isn't complex. Issue classification
+# methods use cascading conditionals that are clear and linear.
 Metrics/CyclomaticComplexity:
-  AllowedMethods: ['case']
+  AllowedMethods: ['case', 'type', 'issue_label']
+
+# Issue#initialize wraps Minitest::Result with keyword args — all are needed.
+# build_issue test helper mirrors those parameters.
+Metrics/ParameterLists:
+  CountKeywordArgs: false
+
+# Classification and token-building methods are linear with many branches.
+# 20 is reasonable for this codebase.
+Metrics/AbcSize:
+  Max: 20
 
 # Development dependencies in gemspec is fine for this gem
 Gemspec/DevelopmentDependencies:
@@ -25,5 +37,24 @@ Metrics/MethodLength:
   CountAsOne: ['array', 'hash', 'heredoc']
   Max: 15
 
+# Test classes naturally grow with coverage (see test/.rubocop.yml for higher limit)
+Metrics/ClassLength:
+  CountAsOne: ['array', 'hash', 'heredoc']
+  Max: 150
+
+Metrics/ModuleLength:
+  CountAsOne: ['array', 'hash', 'heredoc']
+  Max: 150
+
+# Fasterer prefers yield over block.call for performance
+Style/ExplicitBlockArgument:
+  Enabled: false
+
+# Fasterer prefers fetch with block over second argument for performance
+Style/RedundantFetchBlock:
+  Enabled: false
+
+# Compact style (Minitest::Heat::IssueTest) is standard for this project.
+# Nested style breaks when intermediary names are classes, not modules (e.g. Backtrace).
 Style/ClassAndModuleChildren:
   Enabled: false

data/CHANGELOG.md

@@ -1,5 +1,34 @@
 ## [Unreleased]
 
+## [2.2.0] - 2026-09-16
+
+### Added
+- Support Minitest 6 with explicit plugin activation (`Minitest.load :heat`) and compatible test filtering, while retaining automatic discovery on Minitest 5.
+
+### Changed
+- Require Minitest `>= 5.22.3, < 7` and test both major versions in CI. Applications using older Minitest versions must upgrade Minitest before upgrading Heat.
+
+### Fixed
+- Install only one Heat reporter when the plugin is loaded repeatedly, preserving valid JSON output.
+- Normalize relative result paths so test exceptions retain their classification and failure locations when invoked with relative filenames.
+- Return an unsuccessful exit status when Heat cannot record a result, and keep JSON output valid by sending reporter diagnostics to stderr.
+- Return an unsuccessful exit status when an explicit test filter matches no tests, consistent with Minitest; unfiltered empty runs remain successful.
+- Preserve namespaced method names and handle current Ruby quoting in backtrace details.
+- Tests classified as broken now cause an unsuccessful process exit instead of incorrectly reporting success to test runners and CI.
+- JSON output explicitly loads the time library so timestamp formatting works on Ruby 3.2 and 3.3 without relying on another dependency to load it.
+- Updated locked development dependencies for security advisories affecting concurrent-ruby, erb, and json.
+
+## [2.1.1] - 2026-02-01
+
+### Changed
+- Converted ~90 single-expression methods to Ruby 3.0+ endless method syntax
+- Moved dev dependencies from gemspec to Gemfile; changed `add_runtime_dependency` to `add_dependency`
+- Moved constants above `private` in Output::Token for conventional ordering
+- Replaced inline rubocop disables with targeted reek annotations
+- Switched to leading-dot method chains, symbol-to-proc where applicable, and minor style fixes
+- Removed unnecessary `begin`/`end` block in Output#print_tokens rescue
+- Cleaned up test suite: removed redundant tests, fixed silent-pass conditionals, replaced fragile mutation-based setup with fresh instances, added ensure-based cleanup for filesystem tests
+
 ## [2.1.0] - 2026-01-30
 
 ### Added

data/Gemfile

@@ -2,9 +2,21 @@
 
 source 'https://rubygems.org'
 
-# Specify your gem's dependencies in minitest-heat.gemspec
+# Runtime dependencies in minitest-heat.gemspec
 gemspec
 
+gem 'minitest', ENV.fetch('MINITEST_VERSION') if ENV['MINITEST_VERSION']
+
+# Development dependencies
 gem 'bundler-audit', '>= 0.9'
-gem 'minitest', '~> 5.0'
+gem 'debug'
+gem 'flay'
+gem 'flog'
 gem 'rake', '>= 13.0'
+gem 'reek'
+gem 'reviewer'
+gem 'rubocop'
+gem 'rubocop-minitest'
+gem 'rubocop-rake'
+gem 'simplecov'
+gem 'simplecov_json_formatter'

data/Gemfile.lock

@@ -1,36 +1,78 @@
 PATH
   remote: .
   specs:
-    minitest-heat (2.1.0)
-      minitest
+    minitest-heat (2.2.0)
+      minitest (>= 5.22.3, < 7)
 
 GEM
   remote: https://rubygems.org/
   specs:
     ast (2.4.3)
-    awesome_print (1.9.2)
+    bigdecimal (4.0.1)
     bundler-audit (0.9.3)
       bundler (>= 1.2.0)
       thor (~> 1.0)
+    concurrent-ruby (1.3.8)
     date (3.5.1)
     debug (1.11.1)
       irb (~> 1.10)
       reline (>= 0.3.8)
     docile (1.4.1)
-    erb (6.0.1)
+    dry-configurable (1.3.0)
+      dry-core (~> 1.1)
+      zeitwerk (~> 2.6)
+    dry-core (1.2.0)
+      concurrent-ruby (~> 1.0)
+      logger
+      zeitwerk (~> 2.6)
+    dry-inflector (1.3.1)
+    dry-initializer (3.2.0)
+    dry-logic (1.6.0)
+      bigdecimal
+      concurrent-ruby (~> 1.0)
+      dry-core (~> 1.1)
+      zeitwerk (~> 2.6)
+    dry-schema (1.15.0)
+      concurrent-ruby (~> 1.0)
+      dry-configurable (~> 1.0, >= 1.0.1)
+      dry-core (~> 1.1)
+      dry-initializer (~> 3.2)
+      dry-logic (~> 1.6)
+      dry-types (~> 1.8)
+      zeitwerk (~> 2.6)
+    dry-types (1.9.0)
+      bigdecimal (>= 3.0)
+      concurrent-ruby (~> 1.0)
+      dry-core (~> 1.0)
+      dry-inflector (~> 1.0)
+      dry-logic (~> 1.4)
+      zeitwerk (~> 2.6)
+    erb (6.0.7)
+    erubi (1.13.1)
+    flay (2.14.2)
+      erubi (~> 1.10)
+      path_expander (~> 2.0)
+      prism (~> 1.7)
+      sexp_processor (~> 4.0)
+    flog (4.9.4)
+      path_expander (~> 2.0)
+      prism (~> 1.7)
+      sexp_processor (~> 4.8)
     io-console (0.8.2)
     irb (1.16.0)
       pp (>= 0.6.0)
       rdoc (>= 4.0.0)
       reline (>= 0.4.2)
-    json (2.18.0)
+    json (2.21.2)
     language_server-protocol (3.17.0.5)
     lint_roller (1.1.0)
+    logger (1.7.0)
     minitest (5.27.0)
     parallel (1.27.0)
     parser (3.3.10.1)
       ast (~> 2.4.1)
       racc
+    path_expander (2.0.1)
     pp (0.6.3)
       prettyprint
     prettyprint (0.2.0)
@@ -45,9 +87,19 @@ GEM
       erb
       psych (>= 4.0.0)
       tsort
+    reek (6.5.0)
+      dry-schema (~> 1.13)
+      logger (~> 1.6)
+      parser (~> 3.3.0)
+      rainbow (>= 2.0, < 4.0)
+      rexml (~> 3.1)
     regexp_parser (2.11.3)
     reline (0.6.3)
       io-console (~> 0.5)
+    reviewer (0.1.5)
+      rainbow
+      slop
+    rexml (3.4.4)
     rubocop (1.84.0)
       json (~> 2.3)
       language_server-protocol (~> 3.17.0.2)
@@ -70,30 +122,35 @@ GEM
       lint_roller (~> 1.1)
       rubocop (>= 1.72.1)
     ruby-progressbar (1.13.0)
+    sexp_processor (4.17.5)
     simplecov (0.22.0)
       docile (~> 1.1)
       simplecov-html (~> 0.11)
       simplecov_json_formatter (~> 0.1)
     simplecov-html (0.13.2)
     simplecov_json_formatter (0.1.4)
+    slop (4.10.1)
     stringio (3.2.0)
     thor (1.5.0)
     tsort (0.2.0)
     unicode-display_width (3.2.0)
       unicode-emoji (~> 4.1)
     unicode-emoji (4.2.0)
+    zeitwerk (2.7.4)
 
 PLATFORMS
   arm64-darwin-24
   ruby
 
 DEPENDENCIES
-  awesome_print
   bundler-audit (>= 0.9)
   debug
-  minitest (~> 5.0)
+  flay
+  flog
   minitest-heat!
   rake (>= 13.0)
+  reek
+  reviewer
   rubocop
   rubocop-minitest
   rubocop-rake

data/README.md

@@ -20,12 +20,22 @@ Or install it yourself as:
 
     $ gem install minitest-heat
 
-And depending on your usage, you may need to require Minitest Heat in your test suite:
+Heat supports Ruby 3.2 and newer with Minitest `>= 5.22.3, < 7`.
+
+Minitest 5 discovers Heat automatically when the gem is available and plugin loading
+is enabled. Existing Minitest 5 setups do not need to change.
+
+Minitest 6 requires explicit plugin activation. Add this to your `test_helper.rb`:
 
 ```ruby
-require 'minitest/heat'
+require 'minitest/autorun'
+Minitest.load :heat
 ```
 
+`require 'minitest/heat'` loads Heat's library and configuration API; it does not
+activate the reporter on Minitest 6. Loading the Heat plugin more than once installs
+only one reporter.
+
 ## Prioritizing Your Work
 
 Minitest Heat surfaces the most impactful problems first so you can fix what matters without scrolling through noise.
@@ -145,6 +155,24 @@ FORCE_SLOWS=true       # No errors or skipped tests, just slow tests
 
 So to see the full context of a test suite, `IMPLODE=true bundle exec rake` will work its magic.
 
+### Testing Minitest compatibility
+
+The development lockfile retains Minitest 5.27.0. To select another supported version
+in a disposable checkout, use the same runner as CI:
+
+```sh
+export MINITEST_VERSION='~> 6.0'
+export BUNDLE_FROZEN=false BUNDLE_DEPLOYMENT=false
+bundle lock --update minitest
+bundle install
+CI=true bundle exec rake test
+```
+
+Use `~> 5.0` for the latest Minitest 5, or `= 5.22.3` / `= 6.0.0` for the supported
+boundaries. These commands update that checkout's lockfile; do not commit those
+version-selection changes. CI tests the latest 5.x and 6.x on Ruby 3.2–4.0 and both
+Minitest boundaries on Ruby 3.2. All compatibility jobs must pass before release.
+
 ## Contributing
 Bug reports and pull requests are welcome on GitHub at https://github.com/garrettdimon/minitest-heat. This project is intended to be a safe, welcoming space for collaboration, and contributors are expected to adhere to the [code of conduct](https://github.com/[USERNAME]/minitest-heat/blob/master/CODE_OF_CONDUCT.md).
 

data/favicon.svg

@@ -0,0 +1,3 @@
+<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 100 100">
+  <text y="0.9em" font-size="90">🔥</text>
+</svg>

data/lib/minitest/heat/backtrace/line_count.rb

@@ -15,7 +15,7 @@ module Minitest
         end
 
         def earliest_project_location
-          locations.rindex { |element| element.project_file? }
+          locations.rindex(&:project_file?)
         end
 
         def max_location

data/lib/minitest/heat/backtrace/line_parser.rb

@@ -12,8 +12,8 @@ module Minitest
         def self.read(raw_text)
           return nil if raw_text.nil? || raw_text.empty?
 
-          raw_pathname, raw_line_number, raw_container = raw_text.to_s.split(':')
-          raw_container = raw_container&.delete_prefix('in `')&.delete_suffix("'")
+          raw_pathname, raw_line_number, raw_container = raw_text.to_s.split(':', 3)
+          raw_container = raw_container&.sub(/\Ain [`']/, '')&.delete_suffix("'")
 
           ::Minitest::Heat::Location.new(
             pathname: raw_pathname,

data/lib/minitest/heat/backtrace.rb

@@ -21,9 +21,7 @@ module Minitest
       # Determines if the raw backtrace has values in it
       #
       # @return [Boolean] true if there's no backtrace or it's empty
-      def empty?
-        raw_backtrace.empty?
-      end
+      def empty? = raw_backtrace.empty?
 
       # All lines of the backtrace converted to Backtrace::LineParser's
       #
@@ -38,30 +36,22 @@ module Minitest
       #   files at the beginning
       #
       # @return [Array<Location>] the sorted backtrace lines from the project
-      def recently_modified_locations
-        @recently_modified_locations ||= project_locations.sort_by(&:mtime).reverse
-      end
+      def recently_modified_locations = (@recently_modified_locations ||= project_locations.sort_by(&:mtime).reverse)
 
       # All entries from the backtrace that are files within the project
       #
       # @return [Array<Location>] the backtrace lines from within the project
-      def project_locations
-        @project_locations ||= locations.select(&:project_file?)
-      end
+      def project_locations = (@project_locations ||= locations.select(&:project_file?))
 
       # All entries from the backtrace within the project tests
       #
       # @return [Array<Location>] the backtrace lines from within the tests
-      def test_locations
-        @test_locations ||= project_locations.select(&:test_file?)
-      end
+      def test_locations = (@test_locations ||= project_locations.select(&:test_file?))
 
       # All source code entries from the backtrace (i.e. excluding tests)
       #
       # @return [Array<Location>] the backtrace lines from within the source code
-      def source_code_locations
-        @source_code_locations ||= project_locations.select(&:source_code_file?)
-      end
+      def source_code_locations = (@source_code_locations ||= project_locations.select(&:source_code_file?))
     end
   end
 end