2
70%
master: 70%

Ran 20 Nov 2017 01:10PM UTC

Files 211

Run time 7s

Badge

Embed ▾

Committed 20 Nov 2017 12:41PM UTC coverage: 33.865% (-0.01%) from 33.878%

Job # MC_TEST_CHUNK=2

Build Type

push

travis-ci

Committed by

pypt

Commit Message

Fix bug from original all_url_variants() implementation; test topic-inferred URL variants directly instead of just a helper

In original __get_topic_url_variants() implementation (dating back to
controversy mapper), the code was made to discard all the URL variants
previously found with URL normalization, etc. if at least one URL was
found to be amongst the topic stories.

For example, if before the call to __get_topic_url_variants(), the
function would have generated URL variants:

* http://localhost/?cruft=cruft&cruft2=cruft2
* http://localhost/

...and __get_topic_url_variants() were to find that one of the variants
(e.g. http://localhost) was in topic_links, it would discard all the
rest of the variants (the "?cruft=cruft&cruft2=cruft2" variant) and go
with only the variants found in topic_links.

Changes:

* Variants from topic_links are treated as any other variants and merged
into the variant set.
* Unit test tests all_url_variants() instead of
__get_topic_url_variants() to make sure that we're getting all of the
expected variants.

Run Details

5116 of 15107 relevant lines covered (33.87%)

352.58 hits per line

berkmancenter / mediacloud / 4392 / 2
70%
master: 70%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 4392.2 (MC_TEST_CHUNK=2)

berkmancenter / mediacloud / 4392 / 2 70% master: 70%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 4392.2 (MC_TEST_CHUNK=2)

berkmancenter / mediacloud / 4392 / 2
70%
master: 70%

README BADGES
x