ClojureScript

Add a :pprint option to the default reporter in cljs.test

Details

  • Type: Enhancement Enhancement
  • Status: Closed Closed
  • Priority: Minor Minor
  • Resolution: Completed
  • Affects Version/s: 0.0-3269
  • Fix Version/s: None
  • Component/s: None
  • Labels:

Description

Now that cljs.pprint has landed, cljs.test could report failures and exceptions with it. The exact API is TBD.

  1. cljs_1282_v02.patch
    28/Jun/15 5:13 PM
    3 kB
    Sebastian Bensusan
  2. cljs_1282_v03.patch
    13/Jul/15 5:10 PM
    3 kB
    Sebastian Bensusan
  3. cljs_1282.patch
    22/Jun/15 4:00 PM
    3 kB
    Sebastian Bensusan

Activity

Hide
Sebastian Bensusan added a comment -

The :cljs.test/pprint reporter is implemented as a small deviation from the :cljs.test/default reporter.

The API is equivalent to that of custom reporters:

(run-tests
  (empty-env :cljs.test/pprint)
  'my-test.namespace)
Show
Sebastian Bensusan added a comment - The :cljs.test/pprint reporter is implemented as a small deviation from the :cljs.test/default reporter. The API is equivalent to that of custom reporters:
(run-tests
  (empty-env :cljs.test/pprint)
  'my-test.namespace)
Hide
David Nolen added a comment -

Looks good mostly but small typo: `print-comparisson` instead of `print-comparison`.

Show
David Nolen added a comment - Looks good mostly but small typo: `print-comparisson` instead of `print-comparison`.
Hide
Sebastian Bensusan added a comment -

Added the rebased file with the typo corrected.

Show
Sebastian Bensusan added a comment - Added the rebased file with the typo corrected.
Hide
Sebastian Bensusan added a comment -

Rebased the patch.

Show
Sebastian Bensusan added a comment - Rebased the patch.

People

Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: