-
Notifications
You must be signed in to change notification settings - Fork 464
feat: Update and adapt to GraalVM for JDK 23 #2069
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
Merged
Merged
Changes from all commits
Commits
Show all changes
91 commits
Select commit
Hold shift + click to select a range
dceff5b
build(deps): update dependency com.google.cloud:native-image-shared-c…
renovate-bot e2c988e
disable testDisconnectShouldNotWaitToReadResponse for JDK >= 23
diegomarquezp d355740
fix assumption code
diegomarquezp 2fbd4cb
fix assumption ii
diegomarquezp e525298
Revert "fix assumption ii"
diegomarquezp 42958ef
Revert "fix assumption code"
diegomarquezp a74d12c
Revert "disable testDisconnectShouldNotWaitToReadResponse for JDK >= 23"
diegomarquezp e648262
Merge branch 'main' into renovate/com.google.cloud-native-image-share…
diegomarquezp 7599092
chore: use JUnit4 in tests
diegomarquezp 84f3f4f
use at least 1 Junit38 test
diegomarquezp c9eb627
Revert "use at least 1 Junit38 test"
diegomarquezp 74d8487
fix net http response tests
diegomarquezp dbdba31
annotate tests with @RunWith(JUnit4.class)
diegomarquezp 2ba8ac0
initialize static members in static block
diegomarquezp 22e0f17
use static blocks to initialize test classes
diegomarquezp 4c4d719
format
diegomarquezp 905c8f3
set up class initialization
diegomarquezp d78c642
convert google-http-client-test to JUnit4
diegomarquezp f3b785e
Merge branch 'renovate/com.google.cloud-native-image-shared-config-1.…
diegomarquezp e99ea38
use java.nio for tests
diegomarquezp 9faa908
Merge branch 'renovate/com.google.cloud-native-image-shared-config-1.…
diegomarquezp e7bd4e1
run with JUnit 4 explicitly
diegomarquezp 3121f60
Merge branch 'renovate/com.google.cloud-native-image-shared-config-1.…
diegomarquezp 73ebee2
use static block
diegomarquezp 8205011
Merge branch 'renovate/com.google.cloud-native-image-shared-config-1.…
diegomarquezp 06acc4a
Revert "use static block"
diegomarquezp c388a04
Reapply "use static block"
diegomarquezp 6b0a708
initialize IS_WINDOWS at build time
diegomarquezp 9455b69
Merge branch 'renovate/com.google.cloud-native-image-shared-config-1.…
diegomarquezp c70dbd1
prevent cryptic message when initializing FileDataStoreFactory
diegomarquezp 3b27435
Merge branch 'renovate/com.google.cloud-native-image-shared-config-1.…
diegomarquezp 4e507a0
fix static initialization in AbstractDatastoreFactory
diegomarquezp 3f6483d
test no IS_WINDOWS
diegomarquezp e646c47
Revert "test no IS_WINDOWS"
diegomarquezp 70b2b15
ruling out logger initialization
diegomarquezp 0d0f4d7
add test output
diegomarquezp 937aabf
safe initialization of ID_PATTERN
diegomarquezp c9fb4bf
remove provided scope depdendency
diegomarquezp 7b5d600
convert appengine transport to JUnit 4
diegomarquezp 25c6523
Merge branch 'renovate/com.google.cloud-native-image-shared-config-1.…
diegomarquezp 0b346a1
add clirr differences
diegomarquezp c5f76a1
include appengine classes in runtime
diegomarquezp 7e35ed1
add native config
diegomarquezp 8e90738
add reflect-config for appengine
diegomarquezp bf74443
use JUnit4 in GSON tests
diegomarquezp 818da9a
fix clirr
diegomarquezp d6f991c
port abstract json parser test to JUnit4
diegomarquezp 1322e42
remove unused imports
diegomarquezp 3b8c44c
update clirr again
diegomarquezp bf0c6b4
add native config to gson module
diegomarquezp 411304b
convert jackson module to JUnit4
diegomarquezp fff197f
Merge branch 'renovate/com.google.cloud-native-image-shared-config-1.…
diegomarquezp dfb48fe
Merge branch 'renovate/com.google.cloud-native-image-shared-config-1.…
diegomarquezp f787ea4
make jackson factory instantiation native friendly
diegomarquezp 505f931
config test-only build-time initialization
diegomarquezp 94aab38
adapt protocol buffers module to JUnit 4
diegomarquezp 9ad8269
Merge branch 'renovate/com.google.cloud-native-image-shared-config-1.…
diegomarquezp 35543d3
initialize test classes at test time
diegomarquezp 715fc0d
convert xml module to JUnit4
diegomarquezp de7a0b5
Merge branch 'renovate/com.google.cloud-native-image-shared-config-1.…
diegomarquezp dd335e8
add test time initialization entries to xml module
diegomarquezp 6209970
format
diegomarquezp 3d4c01f
restore debug setup
diegomarquezp 448d178
Merge branch 'renovate/com.google.cloud-native-image-shared-config-1.…
diegomarquezp 17665a1
add junit test time initialization flags
diegomarquezp b68b0a7
add junit to gson deps
diegomarquezp 4929c9c
Merge branch 'support-graal-jdk-23-ii' of https://github.com/googleap…
diegomarquezp f3cff46
add junit test-scoped dependency to jackson2 module
diegomarquezp 048a340
add jackson factory to class initialization
diegomarquezp a3c6923
Merge branch 'support-graal-jdk-23-ii' of https://github.com/googleap…
diegomarquezp e75875b
add app engine feature
diegomarquezp eeb9352
try setup time feature registration
diegomarquezp 32e38d6
Revert "try setup time feature registration"
diegomarquezp 700b79d
Revert "add app engine feature"
diegomarquezp 5ca0f1b
add app engine core lib for testing
diegomarquezp 50034e7
add test-scoped dep in test module
diegomarquezp 2407435
use -Pnative-deps
diegomarquezp 8382068
add conflicting dep in native-deps profile for appengine
diegomarquezp d5b3946
include appengine in graalvm tests
diegomarquezp debfc15
restore static blocks in google-http-client
diegomarquezp 245c570
Merge branch 'support-graal-jdk-23-ii' of https://github.com/googleap…
diegomarquezp ce1ef7a
restore static blocks in gson module
diegomarquezp 4c29e49
restore static blocks in jackson
diegomarquezp da3f2c6
restore static blocks in protobuf
diegomarquezp 03dd1b2
format
diegomarquezp c7bfc68
restore static initailization in xml module
diegomarquezp 960fcbd
Merge branch 'main' into support-graal-jdk-23-ii
diegomarquezp f357d9a
restore static block in jackson module
diegomarquezp e600d33
format
diegomarquezp cfa4c27
Merge branch 'support-graal-jdk-23-ii' of https://github.com/googleap…
diegomarquezp ca885f6
remove app engine from test
diegomarquezp File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
47 changes: 47 additions & 0 deletions
47
...-INF/native-image/com.google.http-client/google-http-client-appengine/reflect-config.json
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,47 @@ | ||
[ | ||
{ | ||
"name": "com.google.apphosting.api.DatastorePb$DeleteRequest", | ||
"methods": [ | ||
{ | ||
"name": "<init>", | ||
"parameterTypes": [] | ||
} | ||
] | ||
}, | ||
{ | ||
"name": "com.google.apphosting.api.DatastorePb$GetRequest", | ||
"methods": [ | ||
{ | ||
"name": "<init>", | ||
"parameterTypes": [] | ||
} | ||
] | ||
}, | ||
{ | ||
"name": "com.google.apphosting.api.DatastorePb$NextRequest", | ||
"methods": [ | ||
{ | ||
"name": "<init>", | ||
"parameterTypes": [] | ||
} | ||
] | ||
}, | ||
{ | ||
"name": "com.google.apphosting.api.DatastorePb$PutRequest", | ||
"methods": [ | ||
{ | ||
"name": "<init>", | ||
"parameterTypes": [] | ||
} | ||
] | ||
}, | ||
{ | ||
"name": "com.google.apphosting.api.DatastorePb$Query", | ||
"methods": [ | ||
{ | ||
"name": "<init>", | ||
"parameterTypes": [] | ||
} | ||
] | ||
} | ||
] |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
1 change: 1 addition & 0 deletions
1
.../native-image/com.google.http-client/google-http-client-appengine/native-image.properties
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1 @@ | ||
Args=--initialize-at-build-time=org.junit.runner.RunWith,java.lang.annotation.Annotation \ |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
1 change: 1 addition & 0 deletions
1
...A-INF/native-image/com.google.http-client/google-http-client-gson/native-image.properties
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1 @@ | ||
Args=--initialize-at-build-time=org.junit.runner.RunWith,java.lang.annotation.Annotation |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
9 changes: 4 additions & 5 deletions
9
...F/native-image/com.google.http-client/google-http-client-jackson2/native-image.properties
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,7 +1,6 @@ | ||
Args=--initialize-at-build-time=com.google.api.client.json.jackson2.JacksonFactoryTest \ | ||
--initialize-at-build-time=com.google.api.client.json.jackson2.JacksonGeneratorTest \ | ||
--initialize-at-build-time=com.fasterxml.jackson.core.io.SerializedString \ | ||
--initialize-at-build-time=com.fasterxml.jackson.core.io.CharTypes \ | ||
--initialize-at-build-time=com.fasterxml.jackson.core.JsonFactory \ | ||
--initialize-at-build-time=com.fasterxml.jackson.core.io.JsonStringEncoder \ | ||
--initialize-at-build-time=com.google.api.client.util.StringUtils | ||
--initialize-at-build-time=com.google.api.client.json.jackson2.JacksonFactory \ | ||
--initialize-at-build-time=com.google.api.client.util.StringUtils \ | ||
--initialize-at-build-time=com.fasterxml.jackson.core \ | ||
--initialize-at-build-time=org.junit.runner.RunWith,java.lang.annotation.Annotation |
1 change: 1 addition & 0 deletions
1
...F/native-image/com.google.http-client/google-http-client-protobuf/native-image.properties
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1 @@ | ||
Args=--initialize-at-build-time=org.junit.runner.RunWith,java.lang.annotation.Annotation |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,75 @@ | ||
<?xml version="1.0" encoding="UTF-8"?> | ||
diegomarquezp marked this conversation as resolved.
Show resolved
Hide resolved
|
||
<!-- see http://www.mojohaus.org/clirr-maven-plugin/examples/ignored-differences.html --> | ||
<!-- | ||
[ERROR] 4001: com.google.api.client.test.json.AbstractJsonFactoryTest: Removed junit.framework.Test from the set of implemented interfaces | ||
[ERROR] 5001: com.google.api.client.test.json.AbstractJsonFactoryTest: Removed junit.framework.Assert from the list of superclasses | ||
[ERROR] 5001: com.google.api.client.test.json.AbstractJsonFactoryTest: Removed junit.framework.TestCase from the list of superclasses | ||
[WARNING] 6002: com.google.api.client.test.json.AbstractJsonFactoryTest: Value of field BOOLEAN_TYPE_EMPTY is no longer a compile-time constant | ||
[WARNING] 6002: com.google.api.client.test.json.AbstractJsonFactoryTest: Value of field BOOLEAN_TYPE_EMPTY_OUTPUT is no longer a compile-time constant | ||
[WARNING] 6002: com.google.api.client.test.json.AbstractJsonFactoryTest: Value of field BOOLEAN_TYPE_FALSE is no longer a compile-time constant | ||
[WARNING] 6002: com.google.api.client.test.json.AbstractJsonFactoryTest: Value of field BOOLEAN_TYPE_NULL is no longer a compile-time constant | ||
[WARNING] 6002: com.google.api.client.test.json.AbstractJsonFactoryTest: Value of field BOOLEAN_TYPE_NULL_OUTPUT is no longer a compile-time constant | ||
[WARNING] 6002: com.google.api.client.test.json.AbstractJsonFactoryTest: Value of field BOOLEAN_TYPE_TRUE is no longer a compile-time constant | ||
[WARNING] 6002: com.google.api.client.test.json.AbstractJsonFactoryTest: Value of field BOOLEAN_TYPE_WRONG is no longer a compile-time constant | ||
[ERROR] 7004: com.google.api.client.test.json.AbstractJsonFactoryTest: In method 'public AbstractJsonFactoryTest(java.lang.String)' the number of arguments has changed | ||
[ERROR] 4001: com.google.api.client.test.json.AbstractJsonGeneratorTest: Removed junit.framework.Test from the set of implemented interfaces | ||
[ERROR] 5001: com.google.api.client.test.json.AbstractJsonGeneratorTest: Removed junit.framework.Assert from the list of superclasses | ||
[ERROR] 5001: com.google.api.client.test.json.AbstractJsonGeneratorTest: Removed junit.framework.TestCase from the list of superclasses | ||
[ERROR] 4001: com.google.api.client.test.json.AbstractJsonParserTest: Removed junit.framework.Test from the set of implemented interfaces [ERROR] 5001: com.google.api.client.test.json.AbstractJsonParserTest: Removed junit.framework.Assert from the list of superclasses [ERROR] 5001: com.google.api.client.test.json.AbstractJsonParserTest: Removed junit.framework.TestCase from the list of superclasses | ||
--> | ||
<differences> | ||
<difference> | ||
<!-- migration to JUnit 4 --> | ||
<differenceType>4001</differenceType> | ||
<className>com/google/api/client/test/util/store/AbstractDataStoreFactoryTest</className> | ||
<to>junit/framework/Test</to> | ||
</difference> | ||
<difference> | ||
<!-- migration to JUnit 4 --> | ||
<differenceType>4001</differenceType> | ||
<className>com/google/api/client/test/json/AbstractJsonFactoryTest</className> | ||
<to>junit/framework/Test</to> | ||
</difference> | ||
<difference> | ||
<!-- migration to JUnit 4 --> | ||
<differenceType>4001</differenceType> | ||
<className>com/google/api/client/test/json/AbstractJsonGeneratorTest</className> | ||
<to>junit/framework/Test</to> | ||
</difference> | ||
<difference> | ||
<!-- migration to JUnit 4 --> | ||
<differenceType>4001</differenceType> | ||
<className>com/google/api/client/test/json/AbstractJsonParserTest</className> | ||
<to>junit/framework/Test</to> | ||
</difference> | ||
<difference> | ||
<!-- migration to JUnit 4 --> | ||
<differenceType>5001</differenceType> | ||
<className>com/google/api/client/test/util/store/AbstractDataStoreFactoryTest</className> | ||
<to>junit/framework/*</to> | ||
</difference> | ||
<difference> | ||
<!-- migration to JUnit 4 --> | ||
<differenceType>5001</differenceType> | ||
<className>com/google/api/client/test/json/AbstractJsonFactoryTest</className> | ||
<to>junit/framework/*</to> | ||
</difference> | ||
<difference> | ||
<!-- migration to JUnit 4 --> | ||
<differenceType>5001</differenceType> | ||
<className>com/google/api/client/test/json/AbstractJsonGeneratorTest</className> | ||
<to>junit/framework/*</to> | ||
</difference> | ||
<difference> | ||
<!-- migration to JUnit 4 --> | ||
<differenceType>5001</differenceType> | ||
<className>com/google/api/client/test/json/AbstractJsonParserTest</className> | ||
<to>junit/framework/*</to> | ||
</difference> | ||
<difference> | ||
<!-- migration to JUnit 4 --> | ||
<differenceType>7004</differenceType> | ||
<className>com/google/api/client/test/json/AbstractJsonFactoryTest</className> | ||
<method>*</method> | ||
</difference> | ||
</differences> |
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.