Project

General

Profile

Actions

Patch #32432

closed

Avoid class name overlap that causes TypeError on `rake test:system test`

Added by Akira Matsuda over 4 years ago. Updated over 4 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Code cleanup/refactoring
Target version:
Start date:
Due date:
% Done:

0%

Estimated time:

Description

Running test via `rake test:system test` errs with TypeError because some test class names are overlapping.


$ bundle e rake test:system test
Mercurial test repository NOT FOUND. Skipping functional tests !!!
rake aborted!
TypeError: superclass mismatch for class IssuesTest
.../redmine/test/integration/issues_test.rb:22:in `<top (required)>'
.../activesupport-5.2.3/lib/active_support/dependencies.rb:291:in `require'
.../activesupport-5.2.3/lib/active_support/dependencies.rb:291:in `block in require'
.../activesupport-5.2.3/lib/active_support/dependencies.rb:257:in `load_dependency'
.../activesupport-5.2.3/lib/active_support/dependencies.rb:291:in `require'
.../railties-5.2.3/lib/rails/test_unit/runner.rb:50:in `block in load_tests'
.../railties-5.2.3/lib/rails/test_unit/runner.rb:50:in `each'
.../railties-5.2.3/lib/rails/test_unit/runner.rb:50:in `load_tests'
.../railties-5.2.3/lib/rails/test_unit/runner.rb:39:in `run'
.../railties-5.2.3/lib/rails/test_unit/runner.rb:35:in `rake_run'
.../railties-5.2.3/lib/rails/test_unit/testing.rake:16:in `block in <top (required)>'
.../redmine/bin/bundle:4:in `load'
.../redmine/bin/bundle:4:in `<main>'
Tasks: TOP => test
(See full trace by running task with --trace)

The attached patch fixes this error with a minimal patch that appends "System" only to the class names that overlap with integration tests.


Files

Actions #1

Updated by Marius BÄ‚LTEANU over 4 years ago

  • Description updated (diff)

Please use textile formatting instead of markdown.

Actions #2

Updated by Go MAEDA over 4 years ago

  • Category set to Code cleanup/refactoring
  • Target version set to 4.1.0

Confirmed the error. Setting the target version to 4.1.0.

Actions #3

Updated by Go MAEDA over 4 years ago

  • Status changed from New to Closed
  • Assignee set to Go MAEDA

Committed the patch. Thank you for your contribution.

Actions

Also available in: Atom PDF