Switch to run() over setUp() and tearDown() #24

Merged
merged 5 commits into from Mar 6, 2013

Conversation

Projects
None yet
2 participants
Owner

Fluxx commented Feb 23, 2013

Changes the Exam case to use run() with custom setup and teardown functionality, as opposed to piggy backing on testunit's default setUp and tearDown hooks being called.

This has a few advantages:

  1. It simplifies the code
  2. It allows users of Exam to define their own setUp or tearDown methods in their test cases without needing to call super to ensure the exam versions ran.
  3. Simplifies the API a test runner must implement to be compatible with Exam to just run().

This PR also has some further code cleanup and readability improvements.

Collaborator

tkaemming commented Mar 6, 2013

+1

@Fluxx Fluxx added a commit that referenced this pull request Mar 6, 2013

@Fluxx Fluxx Merge pull request #24 from Fluxx/switch_to_run
Switch to run() over setUp() and tearDown()
3e10e8c

@Fluxx Fluxx merged commit 3e10e8c into master Mar 6, 2013

1 check passed

default The Travis build passed
Details

Fluxx deleted the switch_to_run branch Mar 6, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment