Open Bug 438726 Opened 18 years ago Updated 1 year ago

Test news subscription code

Categories

(MailNews Core :: Networking: NNTP, task)

task

Tracking

(Not tracked)

People

(Reporter: jcranmer, Unassigned)

Details

Attachments

(1 file)

Attached patch Alpha version. — — Splinter Review
Test the subscription code for news. This is an area where some minor regressions have occurred in the past (most notably becoming unable to subscribe to `+'). Basic tests: * Subscribe works. * Unsubscribe works. * Authentication before subscription works (needs authentication of NNTP first) * The new groups stuff works * Subscription search works Fixed subscription bugs: bug 6031 -- macs don't like newsgroups > 32 characters bug 50059 -- Subscription doesn't reflect in folder pane bug 68303 -- Unsubscription doesn't work with i18n bug 141628 -- Newsgroups starting with '.' (also cf. 69231 and 123560) (Summary: if there's a group after it, ignore the .; if just .'s, ignore) bug 190946 -- Refreshing newsgroups does not download all groups bug 195440 -- Crash when cancel in download group list bug 238688 -- Can't subscribe to newsgroups with '+' (This is actually a problem with the subscribe UI dialog and not back code). In the current state, authenticated subscription can't be tested, nor can bug 238688; bug 195440 requires some hackery to have LIST call a cancel subscribe function (subclass and override). Attached is my starting prototype; it doesn't test everything yet, though.
Status: NEW → ASSIGNED
Product: Core → MailNews Core
Assignee: Pidgeot18 → nobody
Status: ASSIGNED → NEW
Severity: normal → S3
Severity: S3 → N/A
Type: defect → task
Version: Trunk → unspecified
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: