Skip to content

Don't fail test if artifact's directory does not exist yet#660

Open
manandbytes wants to merge 1 commit into
eclipse-lemminx:masterfrom
manandbytes:gav-preconditions
Open

Don't fail test if artifact's directory does not exist yet#660
manandbytes wants to merge 1 commit into
eclipse-lemminx:masterfrom
manandbytes:gav-preconditions

Conversation

@manandbytes
Copy link
Copy Markdown
Contributor

Follow up to #651, DownloadArtifactsTest should not fail when executed in isolation, like:

./mvnw clean
./mvnw -Dtest=org.eclipse.lemminx.extensions.maven.participants.hover.DownloadArtifactsTest test
...
[INFO] Results:
[INFO]
[ERROR] Errors:
[ERROR]   DownloadArtifactsTest.testDownloadArtifactOnHover:104->deleteRecursively:83 » NoSuchFile /sources/eclipse-lemminx/lemminx-maven/lemminx-maven/target/.lemminx-maven/org/glassfish/jersey/project/2.19
[ERROR]   DownloadArtifactsTest.testDownloadNonCentralArtifactOnHover:123->deleteRecursively:83 » NoSuchFile /sources/eclipse-lemminx/lemminx-maven/lemminx-maven/target/.lemminx-maven/com/github/goxr3plus/java-stream-player/9.0.4
[INFO]
[ERROR] Tests run: 2, Failures: 0, Errors: 2, Skipped: 0

[1] #651

Follow up to eclipse-lemminx#651, DownloadArtifactsTest should not fail when executed
in isolation, like:

    ./mvnw clean
    ./mvnw -Dtest=org.eclipse.lemminx.extensions.maven.participants.hover.DownloadArtifactsTest test
    ...
    [INFO] Results:
    [INFO]
    [ERROR] Errors:
    [ERROR]   DownloadArtifactsTest.testDownloadArtifactOnHover:104->deleteRecursively:83 » NoSuchFile /sources/eclipse-lemminx/lemminx-maven/lemminx-maven/target/.lemminx-maven/org/glassfish/jersey/project/2.19
    [ERROR]   DownloadArtifactsTest.testDownloadNonCentralArtifactOnHover:123->deleteRecursively:83 » NoSuchFile /sources/eclipse-lemminx/lemminx-maven/lemminx-maven/target/.lemminx-maven/com/github/goxr3plus/java-stream-player/9.0.4
    [INFO]
    [ERROR] Tests run: 2, Failures: 0, Errors: 2, Skipped: 0

[1] eclipse-lemminx#651
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.

1 participant