Project

General

Profile

Actions
Copy link

Defect #33562

closed

Some tests in ApplicationHelperTest are declared as private

Added by Go MAEDA over 4 years ago. Updated over 4 years ago.

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

0%

Estimated time:
Resolution:
Fixed
Affected version:

Description

The attached patch fixes that the following two test methods are never executed.

  • test_export_csv_encoding_select_tag_should_return_nil_when_general_csv_encoding_is_UTF8
  • test_export_csv_encoding_select_tag_should_have_two_option_when_general_csv_encoding_is_not_UTF8

This is because a private keyword (source:tags/4.1.1/test/helpers/application_helper_test.rb#L1891) is placed before those two methods in r17490.

Files

fix-unreachable-test-methods.patch (1.91 KB) fix-unreachable-test-methods.patch Go MAEDA, 2020-06-06 08:55

Related issues

Related to Redmine - Defect #31929: MarkdownFormatterTest#test_should_support_underlined_text is declared as privateClosedGo MAEDA

Actions
Actions
Copy link
 #1

Updated by Go MAEDA over 4 years ago

  • Target version set to Candidate for next major release
Actions
Copy link
 #2

Updated by Mischa The Evil over 4 years ago

  • Target version changed from Candidate for next major release to 4.2.0

LGTM. I don't see why this has to be postponed until the next major release (5.x.x). I think it can safely go into the next minor (4.2.0) release (hence, it could even go into 4.1.2/4.0.8, but I don't think that the significance of this fix warrants such a back-port).

FWIW: this is not the first case where this happened (see #31929), which brings two questions to my mind:
  1. Are there currently more, undetected cases like this?
  2. Is this something that, in the future, can be handled by (a custom) RuboCop (cop) now that's being used?
Actions
Copy link
 #3

Updated by Mischa The Evil over 4 years ago

  • Related to Defect #31929: MarkdownFormatterTest#test_should_support_underlined_text is declared as private added
Actions
Copy link
 #4

Updated by Go MAEDA over 4 years ago

Mischa The Evil wrote:

LGTM. I don't see why this has to be postponed until the next major release (5.x.x). I think it can safely go into the next minor (4.2.0) release (hence, it could even go into 4.1.2/4.0.8, but I don't think that the significance of this fix warrants such a back-port).

Thank you for reviewing the patch. My understanding is that the next major release is 4.2.0, and the next minor release is 4.1.2.

  1. Are there currently more, undetected cases like this?

I have just checked the following files and no other cases are found.

Files

Actions
Copy link
 #5

Updated by Go MAEDA over 4 years ago

  • Subject changed from Test methods that will never be executed in ApplicationHelperTest to Some tests in ApplicationHelperTest are declared as private
  • Assignee set to Go MAEDA
  • Resolution set to Fixed

Committed the patch.

Actions
Copy link
 #6

Updated by Go MAEDA over 4 years ago

  • Status changed from New to Closed
Actions
Copy link
 #7

Updated by Mischa The Evil over 4 years ago

Go MAEDA wrote:

[...] My understanding is that the next major release is 4.2.0, and the next minor release is 4.1.2.

That's why I mentioned it explicitly as there seems to be some confusion about this in the community.
FWIW: AFAIK Redmine follows the (semver) major.minor.patch scheme. So major releases are 3.x.x, 4.x.x, 5.x.x; minor releases are 4.1.x, 4.2.x, 4.3.x and 4.1.1, 4.1.2, 4.1.3 are patch releases.

This is documented in the (currently somewhat outdated) ReleaseManagement wiki page.

Actions
Copy link

Also available in: Atom PDF