Closed
Bug 1330931
Opened 7 years ago
Closed 7 years ago
Tests for nytimes.com
Categories
(Toolkit :: Reader Mode, defect)
Toolkit
Reader Mode
Tracking
()
RESOLVED
FIXED
People
(Reporter: evanxd, Assigned: evanxd)
References
(Blocks 1 open bug)
Details
Write test for the nytime article: https://www.nytimes.com/2017/01/13/world/africa/sudan-sanctions.html?hp&action=click&pgtype=Homepage&clickSource=story-heading&module=second-column-region®ion=top-news&WT.nav=top-news. The article only have one `<div class="story-body-supplemental'">` node. Generally, articles on nytimes.com seems to have two `<div class="story-body-supplemental'">` nodes. We can have at least two tests to test nytime articles. We will add second test on Bug 1300697.
Assignee | ||
Updated•7 years ago
|
Summary: Bug 1317930 - Tests for nytimes.com → Tests for nytimes.com
Assignee | ||
Comment 1•7 years ago
|
||
Sent pull request: https://github.com/mozilla/readability/pull/338/commits/2cb6a80fa5d9dfce0f4fbdc9514f2e2306758a7a
Assignee | ||
Updated•7 years ago
|
Assignee: nobody → evan
Assignee | ||
Comment 2•7 years ago
|
||
Fixed the test failures: https://github.com/mozilla/readability/pull/338/commits/00d562a4e5ba59c82b7a52be38ea03490a3ac53e
Assignee | ||
Comment 3•7 years ago
|
||
Merged in GitHub: https://github.com/mozilla/readability/commit/ccc8e9bf4c5400814d9b7a3ea83c21540da4c76f
Assignee | ||
Updated•7 years ago
|
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•