Project

General

Profile

Bug #11413

Bug #10288: Fix newly identified issues to make our test suite more robust and faster

Test suite: newly added XMPP account is not persisted, and "Pidgin has the expected persistent accounts configured" doesn't notice

Added by intrigeri over 3 years ago. Updated over 3 years ago.

Status:
Resolved
Priority:
Elevated
Assignee:
-
Category:
Test suite
Target version:
Start date:
05/14/2016
Due date:
% Done:

100%

Feature Branch:
test/11413-pidgin-persistence
Type of work:
Code
Blueprint:
Starter:
Affected tool:
Instant Messaging

Description

I see the second "Then Pidgin automatically enables my XMPP account" fail in "Scenario: Using a persistent Pidgin configuration": I see the account manager, and the XMPP account we created before rebooting is not listed (what?!). OTOH, "Pidgin has the expected persistent accounts configured" has succeeded, so clearly something is wrong; and FWIW, I've verified that the random IRC nickname listed in the account manager is the same as during the previous boot. But the persistent accounts.xml does not contain the newly created account, so we have two bugs:

  • the new account is not persisted
  • "Pidgin has the expected persistent accounts configured" missed that, which hid the former bug

Related issues

Related to Tails - Feature #11306: Move #tails to XMPP Resolved 10/28/2015

Associated revisions

Revision 3b6ae359 (diff)
Added by intrigeri over 3 years ago

Test suite: close Pidgin before we inspect or persist its accounts.xml.

I've seen a case when that file is not saved (and thus, not persisted)
if we shut down the system while Pidgin is still running.

Will-fix: #11413

Revision 8e5079bc
Added by anonym over 3 years ago

Merge remote-tracking branch 'origin/test/11413-pidgin-persistence' into testing

Fix-committed: #11413

History

#1 Updated by intrigeri over 3 years ago

#2 Updated by intrigeri over 3 years ago

  • Feature Branch set to test/11413-pidgin-persistence

#3 Updated by intrigeri over 3 years ago

  • Assignee changed from intrigeri to anonym
  • % Done changed from 0 to 50
  • QA Check set to Ready for QA

#4 Updated by intrigeri over 3 years ago

  • Status changed from Confirmed to In Progress

#5 Updated by anonym over 3 years ago

  • Status changed from In Progress to Fix committed
  • Assignee deleted (anonym)
  • % Done changed from 50 to 100
  • QA Check changed from Ready for QA to Pass

It definitely fixes an issue here. Merged!

#6 Updated by anonym over 3 years ago

  • Status changed from Fix committed to Resolved

Also available in: Atom PDF