Remove old unnecessary SkipTest

[kjordahl]

Removes an old SkipTest for a test that doesn't no longer fails (ref: https://svn.enthought.com/enthought/ticket/1618).

Cherry-picked from #189.

[enbuilder]

Merged build triggered. Test FAILed.

[enbuilder]

Merged build started. Test FAILed.

[cfarrow]

LGTM

[enbuilder]

Merged build finished. Test FAILed.

[enbuilder]

Test FAILed.
Refer to this link for build results: http://jenkins.enthought.com:8080/job/Chaco_PR_builder/27/

Build Log
last 5 lines

[...truncated 13 lines...]
x86_64,Windows completed with result FAILURE
x86,Windows completed with result FAILURE
x86_64,Linux completed with result UNSTABLE
x86,Linux completed with result UNSTABLE

[cfarrow]

I take back the LGTM. Please update CHANGES.txt.

[enbuilder]

Merged build triggered. Test FAILed.

[enbuilder]

Merged build started. Test FAILed.

[kjordahl]

@cfarrow Done, but this seems like a pretty minor, non-user-facing change. I'm not sure what the standard for including in the changelog is. There are a few other testing-related PRs merged recently, perhaps they can be mentioned in a single line in CHANGES.txt (and of course testing needs a lot more work).

[cfarrow]

Thanks.

It does seem like a minor issue, but I'm not sure where to draw that line. I see no problem with merging all testing-related PRs on a single line, however.

LGTM (for real).

[coveralls]

Coverage remained the same when pulling f953b2a on test/remove-skiptest into 94dd348 on master.

[enbuilder]

Merged build finished. Test FAILed.

[enbuilder]

Test FAILed.
Refer to this link for build results: http://jenkins.enthought.com:8080/job/Chaco_PR_builder/28/

Build Log
last 5 lines

[...truncated 18 lines...]
x86_64,Linux appears to be cancelled
x86_64,Linux completed with result ABORTED
Aborted
Aborted by anonymous