Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Master add clone command tests #65

Merged
merged 3 commits into from
Jan 12, 2014

Conversation

MarkEWaite
Copy link
Contributor

Add tests of the CloneCommand implementation.

The tests fail on Windows because they are unable to remove
the temporary directory after the test completes. That is the
pattern which was detected previously in JENKINS-19994.

More work is needed before the Jenkins JGit plugin closes
all the files it opens. These tests pass on Linux, and fail on
Windows with the message that the temp dir used for the
test cannot be deleted.

The CliGit and JGit implementations are different as noted in the
comments of the test methods.

Refer to the comments in the source file for some of the difference
detected by the tests.
Desparately need a unit test oriented way to identify the stack trace
of the code which opened a file so that the test failure can be
assigned to a specific stack trace.
@cloudbees-pull-request-builder

plugins » git-client-plugin #223 SUCCESS
This pull request looks good

@jenkinsadmin
Copy link
Member

Thank you for a pull request! Please check this document for how the Jenkins project handles pull requests

ndeloof added a commit that referenced this pull request Jan 12, 2014
@ndeloof ndeloof merged commit f01fa91 into jenkinsci:master Jan 12, 2014
@MarkEWaite MarkEWaite deleted the master-add-clone-command-tests branch January 13, 2014 13:35
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.

4 participants