Page MenuHomePhabricator

tests.site_tests.TestPagePreloading.test_preload_langlinks_count is marked as allowed_failure and should be fixed
Closed, ResolvedPublic

Description

The current returned error (if the allowed_failure decorator is removed):

$ python -m unittest tests.site_tests.TestPagePreloading.test_preload_langlinks_count
tests: max_retries reduced from 15 to 1
Retrieving 5 pages from wikipedia:en.
WARNING: ...\pywikibot-core\pywikibot\page.py:5878: UserWarning: Site wikipedia:be-tarask instantiated using
different code "be-x-old"
  link._site = pywikibot.Site(lang, source.family.name)

Retrieving 5 pages from wikipedia:en.
Retrieving 5 pages from wikipedia:en.
Retrieving 5 pages from wikipedia:en.
 28.752s F
======================================================================
FAIL: test_preload_langlinks_count (tests.site_tests.TestPagePreloading)
Test preloading continuation works.
----------------------------------------------------------------------
Traceback (most recent call last):
  File "...\pywikibot-core\tests\site_tests.py", line 3081, in test_preload_langlinks_count
    self.assertEqual(len(list(links)), count)
AssertionError: 20 != 25

----------------------------------------------------------------------
Ran 1 test in 36.351s

FAILED (failures=1)

Also see https://travis-ci.org/wikimedia/pywikibot/jobs/390455476#L3168.

Details

Event Timeline

Change 439643 had a related patch set uploaded (by Dalba; owner: dalba):
[pywikibot/core@master] Fix handling of API continuation in PropertyGenerator

https://gerrit.wikimedia.org/r/439643

Dalba triaged this task as High priority.Jun 11 2018, 6:30 PM

Change 439643 merged by jenkins-bot:
[pywikibot/core@master] Fix handling of API continuation in PropertyGenerator

https://gerrit.wikimedia.org/r/439643

• Vvjjkkii renamed this task from tests.site_tests.TestPagePreloading.test_preload_langlinks_count is marked as allowed_failure and should be fixed to jabaaaaaaa.Jul 1 2018, 1:05 AM
• Vvjjkkii reopened this task as Open.
• Vvjjkkii removed Dalba as the assignee of this task.
• Vvjjkkii updated the task description. (Show Details)
• Vvjjkkii removed subscribers: gerritbot, Aklapper.
CommunityTechBot renamed this task from jabaaaaaaa to tests.site_tests.TestPagePreloading.test_preload_langlinks_count is marked as allowed_failure and should be fixed.Jul 2 2018, 8:27 AM
CommunityTechBot closed this task as Resolved.
CommunityTechBot assigned this task to Dalba.
CommunityTechBot updated the task description. (Show Details)
CommunityTechBot added subscribers: gerritbot, Aklapper.