Launchpad itself

test_domination.py tests disabled

Bug #39880 reported by Stuart Bishop
6
Affects Status Importance Assigned to Milestone
Launchpad itself
Fix Released
High
Celso Providelo

Bug Description

archivepublisher/tests/test_domination.py was disabled, with no comment on
why or by whom. We need to fix the tests and reenable, or ensure that the
relevant code is being tested elsewhere.

See original description
Revision history for this message
Celso Providelo (cprov) wrote :

I've seen it, I suspect it was dsilvers and will sort this with him.

Changed in soyuz:
assignee: nobody → cprov
status: Unconfirmed → In Progress
Revision history for this message
Celso Providelo (cprov) wrote :

test_config and test_publisher were fixed to cope with the current code and re-enabled, test_domination still disable since it uses deprecated code (current API requires DB access).

Revision history for this message
Celso Providelo (cprov) wrote :

Malcc, can you sort this issue acording your last p-d-o branch ?

Changed in soyuz:
assignee: cprov → malcolmcleaton
status: In Progress → Needs Info
Diogo Matsubara (matsubara)
Changed in soyuz:
assignee: malcolmcleaton → nobody
description: updated
Changed in soyuz:
status: Needs Info → Confirmed
Revision history for this message
Stuart Bishop (stub) wrote :

Increasing priority as keeping our test suite coverage is high priority.

Changed in soyuz:
importance: Medium → High
Jonathan Lange (jml)
tags: added: spurious-test-failure
removed: test-system
Revision history for this message
Celso Providelo (cprov) wrote :

This test was rewritten and it is enabled in production (lib/lp/archivepublisher/tests/test_dominator.py)

Changed in soyuz:
assignee: nobody → Celso Providelo (cprov)
status: Triaged → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.