... but need to move tests for #71 back under failing for 3.x due to …

…diff in yaml engine
FasterXML · Nov 23, 2020 · 69ec7d0 · 69ec7d0
1 parent b50ad8b
commit 69ec7d0
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/yaml/src/main/java/com/fasterxml/jackson/dataformat/yaml/YAMLParser.java b/yaml/src/main/java/com/fasterxml/jackson/dataformat/yaml/YAMLParser.java
@@ -441,7 +441,6 @@ protected JsonToken _decodeScalar(ScalarEvent scalar) throws IOException
         final int len = value.length();
         if (!typeTagOptional.isPresent() || typeTagOptional.get().equals("!")) { // no, implicit
             Tag nodeTag = _yamlResolver.resolve(value, scalar.getImplicit().canOmitTagInPlainScalar());
-
             if (nodeTag == Tag.STR) {
                 return JsonToken.VALUE_STRING;
             }

diff --git a/...at/yaml/deser/NumberAltIntRead71Test.java → .../yaml/failing/NumberAltIntRead71Test.java b/...at/yaml/deser/NumberAltIntRead71Test.java → .../yaml/failing/NumberAltIntRead71Test.java
@@ -1,4 +1,4 @@
-package com.fasterxml.jackson.dataformat.yaml.deser;
+package com.fasterxml.jackson.dataformat.yaml.failing;
 
 import java.math.BigInteger;
 
@@ -7,6 +7,10 @@
 
 // [dataformats-text#71]: hex numbers
 // [dataformats-text#233]: also binary, octal (with/without underscores)
+//
+// 23-Nov-2020, tatu: Alas, snakeyaml_engine does not really support
+//   detecting these variants (maybe wrt YAML 1.1 -> 1.2 changes).
+//   So tests fail. Not sure what to do about this.
 public class NumberAltIntRead71Test extends ModuleTestBase
 {
     static class IntHolder {