Clojure

clojure.test/test-vars runs :each fixtures for vars without :test metadata

Details

  • Type: Defect Defect
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Completed
  • Affects Version/s: Release 1.6
  • Fix Version/s: Release 1.6
  • Component/s: None
  • Labels:
    None
  • Patch:
    Code and Test
  • Approval:
    Ok

Description

The patch for CLJ-866 introduced a bug with :each fixtures and non-test vars: the fixtures are invoked for every var, not just ones with :test metadata.

Patch: tcrawley-fixtures-with-non-test-vars-2014-02-14.diff

Screened by: Alex Miller

Activity

Hide
Alex Miller added a comment -

yeah, I noticed that too. I was going to mention it to Stu the next time we talked.

Show
Alex Miller added a comment - yeah, I noticed that too. I was going to mention it to Stu the next time we talked.
Hide
Andy Fingerhut added a comment -

The patch for this ticket has been committed: https://github.com/clojure/clojure/commit/919a7100ddf327d73bc2d50d9ee1411d4a0e8921

but the ticket has not yet been closed.

Show
Andy Fingerhut added a comment - The patch for this ticket has been committed: https://github.com/clojure/clojure/commit/919a7100ddf327d73bc2d50d9ee1411d4a0e8921 but the ticket has not yet been closed.

People

Vote (0)
Watch (2)

Dates

  • Created:
    Updated:
    Resolved: