Incorrect Symbol Lookup with Hoisted Text Span

[rmarrowstone]

I was playing with capturing and hoisting spans and discovered that the text capture/hoist implementation does not capture the current symbol tables. While I think the likelihood anyone encounters this in production is quite low, I wanted to report it for awareness.

The test below shows my expectation and points out which assertion currently fails:

    @Test
    public void hoistingTextSpanAlsoHoistsSymbolTable() throws IOException {
        String textWithSymbolTables =
                "$ion_1_0 $ion_symbol_table::{ symbols:[\"bar\"] } { foo1: $10 }" +
                "$ion_1_0 $ion_symbol_table::{ symbols:[\"baz\"] } { foo2: $10 }";

        IonReader reader = IonReaderBuilder.standard().build(textWithSymbolTables);
        SeekableReader seekableReader = Facets.asFacet(SeekableReader.class, reader);

        Assertions.assertEquals(IonType.STRUCT, reader.next());
        Span span = seekableReader.currentSpan();
        reader.stepIn();
        Assertions.assertEquals(IonType.SYMBOL, reader.next());
        Assertions.assertEquals("foo1", reader.getFieldName());
        Assertions.assertEquals("bar", reader.stringValue());
        reader.stepOut();

        Assertions.assertEquals(IonType.STRUCT, reader.next());
        reader.stepIn();
        Assertions.assertEquals(IonType.SYMBOL, reader.next());
        Assertions.assertEquals("foo2", reader.getFieldName());
        Assertions.assertEquals("baz", reader.stringValue());
        reader.stepOut();

        // now re-seek to the first value
        seekableReader.hoist(span);

        Assertions.assertEquals(IonType.STRUCT, reader.next());
        reader.stepIn();
        Assertions.assertEquals(IonType.SYMBOL, reader.next());
        Assertions.assertEquals("foo1", reader.getFieldName());

        // this assertion fails
        Assertions.assertEquals("bar", reader.stringValue());
        reader.stepOut();
    }