Fixes tests: resets current directory in gemcutter After block. #485

Merged
merged 1 commit into from Nov 4, 2012

Conversation

Projects
None yet
2 participants
@mildmojo
Contributor

mildmojo commented Nov 4, 2012

The Before was making a temp directory and chdir'ing into it and the
After was removing the temp directory, but it never changed back
out of the (now missing) temp directory. Subsequent tests failed
in the C getcwd function because CWD no longer existed.

Fixed with @jaredonline. Issue on branch #463.

Fixes tests: resets current directory in gemcutter After block.
The Before was making a temp directory and chdir'ing into it and the
After was removing the temp directory, but it never changed back
out of the (now missing) temp directory. Subsequent tests failed
in the C `getcwd` function because CWD no longer existed.

Fixed with @jaredonline. Issue on branch #463.

cmeiklejohn added a commit that referenced this pull request Nov 4, 2012

Merge pull request #485 from mildmojo/subscription_steps_fixed
Fixes tests: resets current directory in gemcutter After block.

@cmeiklejohn cmeiklejohn merged commit c4f9f07 into rubygems:subscription_steps Nov 4, 2012

@cmeiklejohn

This comment has been minimized.

Show comment
Hide comment
@cmeiklejohn

cmeiklejohn Nov 4, 2012

Contributor

👍

Contributor

cmeiklejohn commented Nov 4, 2012

👍

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