Skip to content

Refactored tests#983

Merged
ErisDS merged 1 commit intoTryGhost:masterfrom
gotdibbs:Issue362
Oct 8, 2013
Merged

Refactored tests#983
ErisDS merged 1 commit intoTryGhost:masterfrom
gotdibbs:Issue362

Conversation

@gotdibbs
Copy link
Contributor

@gotdibbs gotdibbs commented Oct 8, 2013

Fixes #362

  • There is no need to set the viewport on functional tests anymore
    unless something other
    than the default of 1280x1024 is desired.
  • There is no need to invoke casper.run to trigger test.done
    anymore for functional tests.
  • Each test works independently of the rest; registration is handled
    once for the lifetime of the test run and then login/logout can be
    invoked automatically as desired.
  • Mocha tests all utilize predefined, more realistic fixtures when
    appropriate.
  • Renamed old api tests that were really model tests as appropraite.
  • Added example api test for posts.

Fixes TryGhost#362

- There is no need to set the viewport on functional tests anymore
unless something other
than the default of 1280x1024 is desired.
- There is no need to invoke `casper.run` to trigger `test.done`
anymore for functional tests.
- Each test works independently of the rest; registration is handled
once for the lifetime of the test run and then login/logout can be
invoked automatically as desired.
- Mocha tests all utilize predefined, more realistic fixtures when
appropriate.
- Renamed old api tests that were really model tests as appropraite.
- Added example api test for posts.
ErisDS added a commit that referenced this pull request Oct 8, 2013
@ErisDS ErisDS merged commit 53dc1b4 into TryGhost:master Oct 8, 2013
@gotdibbs gotdibbs deleted the Issue362 branch October 8, 2013 14:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Infra: Test Fixtures

2 participants