IonReader.Builder(ionValue) #51
Conversation
…to ionReader-builder
|
Continues #46 |
| } | ||
|
|
||
| [TestMethod] | ||
| public void SymbolTableTest() |
There was a problem hiding this comment.
This should test that UserTreeReader.GetSymbolTable returns the correct symbol table. To test this, create a binary Ion stream that contains two different symbol tables, each with one user symbol value in scope. Make sure that the values have the correct text and that calling UserTreeReader.GetSymbolTable when positioned at each value returns the expected symbol table.
However, looking at the source code, I don't think this will work as-is because the UserTreeReader changes its current symbol table based on the IIonValues it retrieves from SystemTreeReader. As we know, SystemTreeReader does not surface IIonValue representations of symbol tables (see #50 ). As a result, UserTreeReader will never change its current symbol table.
Therefore, let's drop this test and make UserTreeReader.GetSymbolTable throw an exception until #50 is resolved.
*Issue #28 *
Adding an IonReader Builder with IonValue as input parameter.