Skip to content

Commit b11f2f8

Browse files
committed
Merge branch '2.15'
2 parents 0681ada + 7fe5f5a commit b11f2f8

File tree

8 files changed

+19
-14
lines changed

8 files changed

+19
-14
lines changed

avro/src/main/java/tools/jackson/dataformat/avro/deser/AvroParserImpl.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -669,5 +669,5 @@ protected final JsonToken setNumber(double v) {
669669
return JsonToken.VALUE_NUMBER_FLOAT;
670670
}
671671

672-
protected abstract JsonToken setString(String str);
672+
protected abstract JsonToken setString(String str) throws IOException;
673673
}

avro/src/main/java/tools/jackson/dataformat/avro/deser/JacksonAvroParserImpl.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1034,7 +1034,7 @@ public boolean checkInputEnd() throws IOException {
10341034
*/
10351035

10361036
@Override
1037-
protected JsonToken setString(String str) {
1037+
protected JsonToken setString(String str) throws IOException {
10381038
_textBuffer.resetWithString(str);
10391039
return JsonToken.VALUE_STRING;
10401040
}

avro/src/main/java/tools/jackson/dataformat/avro/deser/ScalarDefaults.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@ public StringDefaults(String name, String v) {
5353
}
5454

5555
@Override
56-
public JsonToken readValue(AvroReadContext parent, AvroParserImpl parser) {
56+
public JsonToken readValue(AvroReadContext parent, AvroParserImpl parser) throws IOException {
5757
return parser.setString(_defaults);
5858
}
5959
}

avro/src/test/java/tools/jackson/dataformat/avro/RoundtripTest.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22

33
import tools.jackson.core.StreamReadConstraints;
44
import tools.jackson.core.StreamWriteFeature;
5-
5+
import tools.jackson.core.exc.StreamConstraintsException;
66
import tools.jackson.databind.*;
77

88
public class RoundtripTest extends MapTest
@@ -105,8 +105,8 @@ public void testCharSequencesLowStringLimit() throws Exception
105105
try {
106106
mapper.reader(CHARSEQ_SCHEMA)
107107
.forType(CharSeqBean.class).readValue(avroData);
108-
fail("expected IllegalStateException");
109-
} catch (IllegalStateException ise) {
108+
fail("expected StreamConstraintsException");
109+
} catch (StreamConstraintsException ise) {
110110
assertEquals("String length (3) exceeds the maximum length (1)", ise.getMessage());
111111
}
112112
}

cbor/src/test/java/tools/jackson/dataformat/cbor/parse/ParserNumbersTest.java

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,7 @@
77

88
import tools.jackson.core.*;
99
import tools.jackson.core.JsonParser.NumberType;
10+
import tools.jackson.core.exc.StreamConstraintsException;
1011
import tools.jackson.dataformat.cbor.*;
1112
import tools.jackson.dataformat.cbor.testutil.ThrottledInputStream;
1213

@@ -363,9 +364,9 @@ public void testVeryBigDecimalType() throws IOException {
363364
try (CBORParser parser = cborParser(b)) {
364365
try {
365366
parser.nextToken();
366-
fail("expected NumberFormatException");
367-
} catch (NumberFormatException nfe) {
368-
assertEquals("Number length (4153) exceeds the maximum length (1000)", nfe.getMessage());
367+
fail("expected StreamConstraintsException");
368+
} catch (StreamConstraintsException e) {
369+
assertEquals("Number length (4153) exceeds the maximum length (1000)", e.getMessage());
369370
}
370371
}
371372
}

cbor/src/test/java/tools/jackson/dataformat/cbor/seq/ReadTreesTest.java

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,9 @@
33
import java.util.List;
44

55
import tools.jackson.core.StreamReadConstraints;
6+
import tools.jackson.core.exc.StreamConstraintsException;
67
import tools.jackson.databind.*;
8+
79
import tools.jackson.dataformat.cbor.CBORFactory;
810
import tools.jackson.dataformat.cbor.CBORTestBase;
911

@@ -81,7 +83,7 @@ public void testReadTreeSequenceLowStringLimit() throws Exception
8183
try {
8284
it.nextValue();
8385
fail("expected IllegalStateException");
84-
} catch (IllegalStateException ise) {
86+
} catch (StreamConstraintsException ise) {
8587
assertEquals("String length (2) exceeds the maximum length (1)", ise.getMessage());
8688
}
8789
}

smile/src/test/java/tools/jackson/dataformat/smile/async/SimpleStringArrayTest.java

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,7 @@
66
import tools.jackson.core.JsonGenerator;
77
import tools.jackson.core.JsonToken;
88
import tools.jackson.core.StreamReadConstraints;
9+
import tools.jackson.core.exc.StreamConstraintsException;
910
import tools.jackson.databind.ObjectWriter;
1011

1112
import tools.jackson.dataformat.smile.SmileFactory;
@@ -130,8 +131,8 @@ public void testLongAsciiStringsLowStringLimit() throws IOException
130131
assertToken(JsonToken.VALUE_STRING, r.nextToken());
131132
try {
132133
r.currentText();
133-
fail("expected IllegalStateException");
134-
} catch (IllegalStateException ise) {
134+
fail("expected StreamConstraintsException");
135+
} catch (StreamConstraintsException ise) {
135136
assertEquals("String length (98) exceeds the maximum length (10)", ise.getMessage());
136137
}
137138
}

smile/src/test/java/tools/jackson/dataformat/smile/parse/NumberParsingTest.java

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@
55
import java.math.BigInteger;
66

77
import tools.jackson.core.*;
8+
import tools.jackson.core.exc.StreamConstraintsException;
89
import tools.jackson.core.exc.StreamReadException;
910

1011
import tools.jackson.dataformat.smile.BaseTestForSmile;
@@ -400,8 +401,8 @@ public void testVeryBigDecimal() throws IOException
400401
assertToken(JsonToken.VALUE_NUMBER_FLOAT, p.nextToken());
401402
try {
402403
p.getNumberType();
403-
fail("expected NumberFormatException");
404-
} catch (NumberFormatException nfe) {
404+
fail("expected StreamConstraintsException");
405+
} catch (StreamConstraintsException nfe) {
405406
assertEquals("Number length (4153) exceeds the maximum length (1000)", nfe.getMessage());
406407
}
407408
}

0 commit comments

Comments
 (0)