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

Empty change to check how unit tests run #50

Closed
wants to merge 3 commits into from
Closed

Empty change to check how unit tests run #50

wants to merge 3 commits into from

Conversation

suztomo
Copy link
Contributor

@suztomo suztomo commented Sep 11, 2020

Empty change to check how unit tests run

@suztomo
Copy link
Contributor Author

suztomo commented Sep 12, 2020

Build And Test / unitTests (8) failed https://github.com/GoogleCloudPlatform/spring-cloud-gcp/pull/50/checks?check_run_id=1103888830

[ERROR] Failed to execute goal on project spring-cloud-gcp-kotlin-app-sample: Could not resolve dependencies for project
 com.google.cloud:spring-cloud-gcp-kotlin-app-sample:jar:2.0.0-SNAPSHOT: Failed to collect dependencies at 
org.springframework.boot:spring-boot-starter-data-jpa:jar:2.4.0-SNAPSHOT -> org.hibernate:hibernate-core:jar:5.4.20.Final
 -> org.glassfish.jaxb:jaxb-runtime:jar:2.3.3 -> com.sun.istack:istack-commons-runtime:jar:3.0.11: Failed to read artifact 
descriptor for com.sun.istack:istack-commons-runtime:jar:3.0.11: Could not transfer artifact com.sun.istack:istack-commons-
runtime:pom:3.0.11 from/to central (https://repo.maven.apache.org/maven2): 
/home/runner/.m2/repository/com/sun/istack/istack-commons-runtime/3.0.11/istack-commons-runtime-3.0.11.pom.part (No
 such file or directory) -> [Help 1]

@suztomo
Copy link
Contributor Author

suztomo commented Sep 12, 2020

2nd run of "Build And Test / unitTests (8)" passed mvn install but failed to compile test https://github.com/GoogleCloudPlatform/spring-cloud-gcp/runs/1105851346

[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 01:04 min (Wall Clock)
[INFO] Finished at: 2020-09-12T13:18:48Z
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:3.8.1:testCompile (default-testCompile) on project spring-cloud-gcp-data-datastore: Compilation failure
[ERROR] /home/runner/work/spring-cloud-gcp/spring-cloud-gcp/spring-cloud-gcp-data-datastore/src/test/java/org/springframework/cloud/gcp/data/datastore/core/convert/TestDatastoreItemCollections.java:[110,31] no suitable method found for add(java.util.List<capture#1 of ?>)
[ERROR]     method java.util.Collection.add(java.util.List<java.lang.Byte>) is not applicable
[ERROR]       (argument mismatch; java.util.List<capture#2 of ?> cannot be converted to java.util.List<java.lang.Byte>)
[ERROR]     method java.util.List.add(java.util.List<java.lang.Byte>) is not applicable
[ERROR]       (argument mismatch; java.util.List<capture#2 of ?> cannot be converted to java.util.List<java.lang.Byte>)
[ERROR] -> [Help 1]

@suztomo
Copy link
Contributor Author

suztomo commented Sep 14, 2020

@dzou (I see you committed the repository few weeks ago) Do you happen to know why the compilation error did not surface when you made the change?

@dzou
Copy link
Contributor

dzou commented Sep 14, 2020

@dzou (I see you committed the repository few weeks ago) Do you happen to know why the compilation error did not surface when you made the change?

Hmm I'm not sure. The build sometimes breaks when changes are introduced upstream in our Spring dependencies (since we depend on snapshot versions) but this seems unrelated... Let me see if it breaks locally for me.

@dzou
Copy link
Contributor

dzou commented Sep 14, 2020

Ah yes, the breakage is happening locally for me too. That is odd; it probably must be a breaking API change upstream in Spring which somehow changed the types.

@suztomo
Copy link
Contributor Author

suztomo commented Sep 14, 2020

@dzou Thank you for checking!

@dzou
Copy link
Contributor

dzou commented Sep 14, 2020

@suztomo - I created an issue for us to fix this, thanks for reporting this Tomo.

@suztomo suztomo reopened this Sep 15, 2020
@suztomo suztomo closed this Sep 15, 2020
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.

None yet

2 participants