Fix printing of async test summary.

cichli commented 10 years ago

Fixes #61.

The results from async tests are now only printed if any were actually run.
Additionally, the printed summary for the async test results will now only include statistics for the async tests themselves - previously, the sync and async tests were added together.

Example output without async tests:

Testing core-async-tutorial.core-test 

FAIL in (core-async-tutorial.core-test/fail-test) (:)
expected: false
  actual: false

Ran 3 tests containing 3 assertions.
Testing complete: 1 failures, 0 errors.

Example output with async tests:

Testing core-async-tutorial.core-test 

FAIL in (core-async-tutorial.core-test/fail-test) (:)
expected: false
  actual: false

Testing core-async-tutorial.async-test 

Ran 3 tests containing 3 assertions.
Testing complete: 1 failures, 0 errors.
Waiting on 2 asynchronous tests to complete.

Testing core-async-tutorial.async-test (async)

FAIL in (core-async-tutorial.async-test/async-fail-test) (:)
expected: false
  actual: false

Ran 2 tests containing 2 assertions.
Testing complete: 1 failures, 0 errors.

cichli commented 10 years ago

Happy to update so the final printed results are for async + sync; not sure if you intended for that to be the case or not.

cemerick commented 10 years ago

Thanks for working on this! Yes, the final report should be a full summary of both flavours.

cichli commented 10 years ago

:+1: No problem. New example outputs (without and with async tests):

Testing core-async-tutorial.core-test 

FAIL in (core-async-tutorial.core-test/fail-test) (:)
expected: false
  actual: false

Ran 3 tests containing 3 assertions.
Testing complete: 1 failures, 0 errors.

Testing core-async-tutorial.core-test 

FAIL in (core-async-tutorial.core-test/fail-test) (:)
expected: false
  actual: false

Testing core-async-tutorial.async-test 

Ran 3 tests containing 3 assertions.
Testing complete: 1 failures, 0 errors.
Waiting on 2 asynchronous tests to complete.

Testing core-async-tutorial.async-test (async)

FAIL in (core-async-tutorial.async-test/async-fail-test) (:)
expected: false
  actual: false

Ran 5 tests containing 5 assertions.
Testing complete: 2 failures, 0 errors.

cemerick commented 9 years ago

Thanks!

cemerick / clojurescript.test

Fix printing of async test summary. #62