Merge branch '2.19'

Author: cowtowncoder

.github/workflows/cifuzz.yml

@@ -30,7 +30,7 @@ jobs:
         dry-run: false
         language: jvm
     - name: Upload Crash
-      uses: actions/upload-artifact@4cec3d8aa04e39d1a68397de0c4cd6fb9dce8ec1 # v4.6.1
+      uses: actions/upload-artifact@ea165f8d65b6e75b540449e92b4886f43607fa02 # v4.6.2
       if: failure() && steps.build.outcome == 'success'
       with:
         name: artifacts

.github/workflows/codeql-analysis.yml

@@ -27,12 +27,12 @@ jobs:
 
     # Initializes the CodeQL tools for scanning.
     - name: Initialize CodeQL
-      uses: github/codeql-action/init@b56ba49b26e50535fa1e7f7db0f4f7b4bf65d80d # v3.28.10
+      uses: github/codeql-action/init@1b549b9259bda1cb5ddde3b41741a82a2d15a841 # v3.28.13
       with:
         languages: ${{ matrix.language }}
 
     - name: Autobuild
-      uses: github/codeql-action/autobuild@b56ba49b26e50535fa1e7f7db0f4f7b4bf65d80d # v3.28.10
+      uses: github/codeql-action/autobuild@1b549b9259bda1cb5ddde3b41741a82a2d15a841 # v3.28.13
 
     - name: Perform CodeQL Analysis
-      uses: github/codeql-action/analyze@b56ba49b26e50535fa1e7f7db0f4f7b4bf65d80d # v3.28.10
+      uses: github/codeql-action/analyze@1b549b9259bda1cb5ddde3b41741a82a2d15a841 # v3.28.13

src/test/java/tools/jackson/databind/records/RecordWithReadOnlyTest.java

@@ -17,7 +17,7 @@ record RecordWithReadOnly(int id, @JsonProperty(access = Access.READ_ONLY) Strin
     }
 
     record RecordWithReadOnlyNamedProperty(int id,
-                                           @JsonProperty(value = "name", access = Access.READ_ONLY) String name) {
+            @JsonProperty(value = "name", access = Access.READ_ONLY) String name) {
     }
 
     record RecordWithReadOnlyAccessor(int id, String name) {
@@ -29,8 +29,9 @@ public String name() {
         }
     }
 
-    record RecordWithReadOnlyComponentOverriddenAccessor(int id, @JsonProperty(access = Access.READ_ONLY) String name) {
-
+    record RecordWithReadOnlyComponentOverriddenAccessor(int id,
+            @JsonProperty(access = Access.READ_ONLY) String name)
+    {
         // @JsonProperty on overridden method is not automatically inherited by overriding method
         @Override
         public String name() {
@@ -88,12 +89,14 @@ public void testSerializeReadOnlyNamedProperty() throws Exception {
     }
 
     /**
-     * Currently documents a bug where a property was NOT ignored during deserialization if given an explicit name.
+     * Currently documents a bug where a property was NOT ignored during deserialization
+     * if given an explicit name.
      * Also reproducible in 2.14.x.
      */
     @Test
     public void testDeserializeReadOnlyNamedProperty() throws Exception {
-        RecordWithReadOnlyNamedProperty value = MAPPER.readValue(a2q("{'id':123,'name':'Bob'}"), RecordWithReadOnlyNamedProperty.class);
+        RecordWithReadOnlyNamedProperty value = MAPPER.readValue(a2q("{'id':123,'name':'Bob'}"),
+                RecordWithReadOnlyNamedProperty.class);
         assertEquals(new RecordWithReadOnlyNamedProperty(123, "Bob"), value); // BUG: should be `null` instead of "Bob"
     }
 
@@ -120,7 +123,8 @@ public void testDeserializeReadOnlyAccessor() throws Exception {
 
     /*
     /**********************************************************************
-    /* Test methods, JsonProperty.access=READ_ONLY component, but accessor method was overridden without re-annotating with JsonProperty.access=READ_ONLY
+    /* Test methods, JsonProperty.access=READ_ONLY component, but accessor
+     * method was overridden without re-annotating with JsonProperty.access=READ_ONLY
     /**********************************************************************
      */
 
@@ -153,7 +157,8 @@ public void testSerializeReadOnlyPrimitiveTypeProperty() throws Exception {
 
     @Test
     public void testDeserializeReadOnlyPrimitiveTypeProperty() throws Exception {
-        RecordWithReadOnlyPrimitiveType value = MAPPER.readValue(a2q("{'id':123,'name':'Bob'}"), RecordWithReadOnlyPrimitiveType.class);
+        RecordWithReadOnlyPrimitiveType value = MAPPER.readValue(a2q("{'id':123,'name':'Bob'}"),
+                RecordWithReadOnlyPrimitiveType.class);
         assertEquals(new RecordWithReadOnlyPrimitiveType(0, "Bob"), value);
     }
 
@@ -171,7 +176,8 @@ public void testSerializeReadOnlyAllProperties() throws Exception {
 
     @Test
     public void testDeserializeReadOnlyAllProperties() throws Exception {
-        RecordWithReadOnlyAll value = MAPPER.readValue(a2q("{'id':123,'name':'Bob'}"), RecordWithReadOnlyAll.class);
+        RecordWithReadOnlyAll value = MAPPER.readValue(a2q("{'id':123,'name':'Bob'}"),
+                RecordWithReadOnlyAll.class);
         assertEquals(new RecordWithReadOnlyAll(0, null), value);
     }
 
@@ -183,7 +189,8 @@ public void testSerializeReadOnlyAllProperties_WithNoArgConstructor() throws Exc
 
     @Test
     public void testDeserializeReadOnlyAllProperties_WithNoArgConstructor() throws Exception {
-        RecordWithReadOnlyAllAndNoArgConstructor value = MAPPER.readValue(a2q("{'id':123,'name':'Bob'}"), RecordWithReadOnlyAllAndNoArgConstructor.class);
+        RecordWithReadOnlyAllAndNoArgConstructor value = MAPPER.readValue(a2q("{'id':123,'name':'Bob'}"),
+                RecordWithReadOnlyAllAndNoArgConstructor.class);
         assertEquals(new RecordWithReadOnlyAllAndNoArgConstructor(0, null), value);
     }
 }