tools.nrepl

Two deftests with same name cause reduced test coverage

Details

  • Type: Defect Defect
  • Status: Closed Closed
  • Priority: Trivial Trivial
  • Resolution: Completed
  • Affects Version/s: None
  • Fix Version/s: 0.2.4
  • Component/s: None
  • Labels:
    None

Description

There are two deftest's with the name switch-ns in the same namespace. When this happens, the first deftest's tests are ignored, and never run. It is best if all tests are given unique names so this does not happen.

Found using a pre-release version of the Eastwood Clojure lint tool.

Activity

Hide
Andy Fingerhut added a comment -

Patch nrepl-47-v1.diff simply renames one of the deftest's named switch-ns to switch-ns-2.

Show
Andy Fingerhut added a comment - Patch nrepl-47-v1.diff simply renames one of the deftest's named switch-ns to switch-ns-2.
Hide
Chas Emerick added a comment -

Thanks, fix applied.

Show
Chas Emerick added a comment - Thanks, fix applied.

People

Vote (0)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved: