xml reader over, but need test.