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

Jump to bottom

Running the Test Suite #9929

Closed
cyberduck opened this issue May 8, 2017 · 3 comments
Closed

Running the Test Suite #9929

cyberduck opened this issue May 8, 2017 · 3 comments
Labels
documentation enhancement fixed

Comments

@cyberduck
Copy link
Collaborator

3d620ce created the issue

Hey there, been using Cyberduck for a long time and am preparing to contribute. Unfortunately I've had some trouble running the test suite. After going over the readme and looking into MDEP-187 I realized this wasn't a problem with Cyberduck but stems from the way Maven organizes modules. It would probably help other developers getting started if this were addressed in the readme.

I was able to get the suite running by invoking mvn package in each project that threw the error and then rerunning mvn test -DskipITs until the build was successful, on OS X this list emerged:

  • core
  • core/dylib
  • bonjour
  • importer
  • protocols
  • osx
  • cli/osx

Additionally, there are a handful of tests which seem to be failing as a result of a missing IntegrationTest annotation. Most of these have been corrected as of this morning, but these last two that are still unmarked as of submitting this ticket:

  • S3SearchFeatureTest.search:52 » LoginFailure Listing directory / failed.
  • OneDriveSharingLinkUrlProviderTest>AbstractOneDriveTest.setup
@cyberduck
Copy link
Collaborator Author

@dkocher commented

I will see that we can add better documentation and I must have hit MDEP-187 myself, as the link is marked as visited in my browser (…).

@cyberduck
Copy link
Collaborator Author

@dkocher commented

Added missing integration test annotation in 5d8fd59 and bb4c903.

@cyberduck
Copy link
Collaborator Author

@dkocher commented

Clarify documentation in f9104fe.

@iterate-ch iterate-ch locked as resolved and limited conversation to collaborators Nov 26, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
documentation enhancement fixed
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@cyberduck