ordering issues in test_isolation

Bug #551751 reported by Aaron Bentley
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Launchpad itself
Fix Released
High
Gavin Panella

Bug Description

On one of my machines, test_check_no_transaction, test_gen_store_statuses, test_is_transaction_in_progress and test_ensure_no_transaction fail.

This appears to be an ordering issue in createTransaction, because it always uses store[0] to create the transaction, but apparently sometimes doing a select in store[0] doesn't cause a transaction to be detected-- instead store[1] must be used.

I'm not sure whether this is a broken test or a latent bug. Since its failure is inconsistent, I'm disabling it in my disable-isolation-test branch.

Related branches

Deryck Hodge (deryck)
Changed in malone:
status: New → Triaged
importance: Undecided → High
milestone: none → 10.04
tags: added: testfix
Revision history for this message
Ursula Junque (ursinha) wrote : Bug fixed by a commit
Changed in malone:
assignee: nobody → Aaron Bentley (abentley)
status: Triaged → Fix Committed
tags: added: qa-needstesting
Aaron Bentley (abentley)
tags: added: qa-untestable
removed: qa-needstesting
Revision history for this message
Deryck Hodge (deryck) wrote :

Should we leave this bug open to track fixing the tests properly or open a new bug to that effect? I leave it to Aaron to decide, if he wants this one for tracking his disabling-test work. :-)

Aaron Bentley (abentley)
Changed in malone:
status: Fix Committed → Triaged
assignee: Aaron Bentley (abentley) → nobody
Deryck Hodge (deryck)
Changed in malone:
milestone: 10.04 → none
Deryck Hodge (deryck)
tags: removed: qa-untestable
Deryck Hodge (deryck)
tags: added: dhrb
Revision history for this message
Gavin Panella (allenap) wrote :

I think this problem is because one of the stores is the session database (LaunchpadSessionDatabase) and this is set to auto-commit. I'll amend createTransaction() test helper method to seek out a non-auto-commit store and complain if one is not found.

Changed in launchpad:
assignee: nobody → Gavin Panella (allenap)
Revision history for this message
Launchpad QA Bot (lpqabot) wrote :
Changed in launchpad:
milestone: none → 11.02
tags: added: qa-needstesting
Changed in launchpad:
status: Triaged → Fix Committed
Gavin Panella (allenap)
tags: added: qa-untestable
removed: qa-needstesting
William Grant (wgrant)
Changed in launchpad:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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