Skip to content
This repository has been archived by the owner on Apr 23, 2020. It is now read-only.

Make Watai compatible with Node v0.10 #90

Merged
merged 14 commits into from
Sep 4, 2013
Merged

Make Watai compatible with Node v0.10 #90

merged 14 commits into from
Sep 4, 2013

Conversation

Ndpnt
Copy link
Collaborator

@Ndpnt Ndpnt commented Sep 4, 2013

  • Make config sanity checks compatible with Node ≥0.10 and also retrocompatible with Node 0.8
  • Remove obsolete path.basename's argument.
  • Add SucceedingSuite in test's resources to ensure having a succeeding suite in tests which is not internet dependent.
  • Refactor some tests condition with SucceedingSuite instead of FailingSuite for safety.

@MattiSG
Copy link
Owner

MattiSG commented Sep 4, 2013

Seems good. I'll rebase and merge. Thanks! :)

MattiSG and others added 14 commits September 4, 2013 15:27
Since node v0.9.6, basename and extname ignore trailing slashes.
The condition of `should not do anything if passed empty options` test is inverted to prevent having an error code returned due to test failure.
The condition of `should not do anything if passed empty options` test is inverted to prevent having an error code returned due to test failure
Pessimistic approach, we'll see what happens with Node updates (not tested on v0.11, yet saying it won't work, to at least warn users that it's ntested)
@MattiSG MattiSG merged commit 054e4dc into v06 Sep 4, 2013
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants