test order of parsing events

Details

  • Type: Task Task
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Won't Fix
  • Affects Version/s: None
  • Fix Version/s: None
  • Component/s: Core
  • Labels:
    None

Description

For some elements (especially tables and figures), it is essential that a parser emits events in a specified order. We could adapt the WellformednessCheckingSink to test that.

Issue Links

relates to

Bug - A problem which impairs or prevents the functions of the product. DOXIA-177 Invalid XHTML because of wrong position of table caption

  • Major - Major loss of function.
  • Closed - The issue is considered finished, the resolution is correct. Issues which are not closed can be reopened.

Activity

Hide
Permalink
Lukas Theussl added a comment -

Actually I think it is not possible to do this generally, at least I don't see how to enforce a particular event order for arbitrary parsers. We could define some standard parsing sequence for certain elements, but these would have to be checked on a per-parser level anyway. So in practice I guess it's just easier to assure that sinks are flexible enough to deal with any (reasonable) order of parsing events, so that the output is valid for the given format. (Btw I have added a SinkEventTestingSink that can be used to test event sequences of particular parsers).

Show
Lukas Theussl added a comment - Actually I think it is not possible to do this generally, at least I don't see how to enforce a particular event order for arbitrary parsers. We could define some standard parsing sequence for certain elements, but these would have to be checked on a per-parser level anyway. So in practice I guess it's just easier to assure that sinks are flexible enough to deal with any (reasonable) order of parsing events, so that the output is valid for the given format. (Btw I have added a SinkEventTestingSink that can be used to test event sequences of particular parsers).

People

  • Assignee:
    Lukas Theussl
    Reporter:
    Lukas Theussl
Vote (0)
Watch (0)

Dates

  • Created:
    Updated:
    Resolved: