Skip to content

Commit dabe16a

Browse files
author
John Hudson
committed
Test reproducing SmileParser getValueAsString() issue with JsonToken.FIELD_NAME and tentative bugfix.
1 parent d12d0c3 commit dabe16a

File tree

2 files changed

+4
-1
lines changed

2 files changed

+4
-1
lines changed

smile/src/main/java/com/fasterxml/jackson/dataformat/smile/SmileParser.java

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1173,6 +1173,9 @@ public String getValueAsString() throws IOException
11731173
if (_currToken == JsonToken.VALUE_STRING) {
11741174
return _textBuffer.contentsAsString();
11751175
}
1176+
if (_currToken == JsonToken.FIELD_NAME) {
1177+
return currentName();
1178+
}
11761179
if (_currToken == null || _currToken == JsonToken.VALUE_NULL || !_currToken.isScalarValue()) {
11771180
return null;
11781181
}

smile/src/test/java/com/fasterxml/jackson/dataformat/smile/parse/BasicParserTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -231,8 +231,8 @@ public void testSimpleObject() throws IOException
231231
assertToken(JsonToken.START_OBJECT, p.nextToken());
232232

233233
assertToken(JsonToken.FIELD_NAME, p.nextToken());
234-
assertEquals("a", p.currentName());
235234
assertEquals("a", p.getText());
235+
assertEquals("a", p.getValueAsString());
236236
assertToken(JsonToken.VALUE_NUMBER_INT, p.nextToken());
237237
assertEquals(8, p.getIntValue());
238238

0 commit comments

Comments
 (0)