Opened 12 months ago

Closed 12 months ago

Last modified 12 months ago

#13489 closed Patch - Bug Fix (Fixed)

Fix TestIPTVRecorder after change to IPTVChannelInfo

Reported by: ijc Owned by: paul-h
Priority: blocker Milestone: 31.0
Component: MythTV - HTTP Streaming Version: Master Head
Severity: high Keywords:
Cc: paul-h Ticket locked: no


My patches in broke the related unit tests. My apologies.

Attached is the fix (also pushed to

Commit message for reference:

    Fix TestIPTVRecorder after change to IPTVChannelInfo
    In a60e7429a09a ("IPTVChannelInfo: explictly set IPTVTuningData protocol to
    invalid") I removed `IPTVChannelInfo::IsValid()` thinking it was unused but
    missed the uses in these test cases. Reintroduce the method, but make it
    `protected` and a friend of the `TestIPTVRecorder` class.
    The above change also changed the behaviour of `IPTVChannelInfo`'s constructors
    such that the protocol is not always probed, and therefore `IsValid()` is not
    necessarily true. This is the case when constructed via
    `IPTVChannelFetcher::ParsePlayList`. Update `TestIPTVRecorder::ParseChanInfo` to
    reflect this by now asserting that the tuning data is not valid and that the
    protocol is `inValid` as expected.

Attachments (1)

0001-Fix-TestIPTVRecorder-after-change-to-IPTVChannelInfo.patch (6.8 KB) - added by ijc 12 months ago.

Download all attachments as: .zip

Change History (3)

Changed 12 months ago by ijc

comment:1 Changed 12 months ago by paul-h

Milestone: needs_triage31.0
Owner: changed from cpinkham to paul-h
Status: newaccepted

I'll commit this for now but I do want to store the protocol in the DB so we only have to do the test download once.

There are also some URL's we don't handle properly at all like this made up one since the URL doesn't end in .m3u8 :-

comment:2 Changed 12 months ago by paul-h

Resolution: Fixed
Status: acceptedclosed

In [changeset:"36d96c956969a85feb8578f0cbe2cc42fee01016/mythtv"


(The changeset message doesn't reference this ticket)

Last edited 12 months ago by paul-h (previous) (diff)
Note: See TracTickets for help on using tickets.