taskotron / libtaskotron

#62 Remove the TAP/Bayeux workaround from check.py
Closed: Fixed Opened by jskladan.

Closed: Fixed
Reopen Issue

Just as a reminder, once the Bayeux issues are resolved, remove these lines (or a similar one) from check.py.
#TODO: remove this ugly hack - it is here because of the bug in bayeux
tap = "TAP version 13\n1..1\n%s" % check.export_TAP(self.cd)

D68 code contains this:

 tapgen = tap.TAPGenerator()
 tapout = ['TAP version 13', '1..%d' % len(check_details)]

And I think we don't even need to put a FIXME comment there. Because we create the TAP segments in a loop, it actually makes sense for //our code// to create that header, not the TAP generator.

So maybe we can simply close this ticket?

Yeah, we probably can. This was more of a "do not forget about it" ticket, and I guess that if we (ever) decide to merge bayeux and pytap13, the api will change quite a lot (for the exporting purposes), so this ticket won't be necessary even then.
Closing.

Metadata Update from @jskladan:
- Issue marked as depending on: #61
Metadata
None
None
None
None
Normal
Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.