Bazaar

TestUiFactory use sys.stdout for password prompt

Bug #110204 reported by Vincent Ladeuil on 2007-04-26

Affects		Status	Importance	Assigned to	Milestone
	Bazaar	Fix Released	Low	Vincent Ladeuil	Bazaar 0.17

Bug Description

CLIUIFactory use sys.stdout to encode prompt.
TestUIFactory overrides sys.stdout without taking into account this prompt encoding.

Tests writers get trapped, their tests run ok locally but fail on pqm :)

Vincent Ladeuil (vila) on 2007-04-26

Changed in bzr:
assignee:	nobody → v-ladeuil
importance:	Undecided → Low
status:	Unconfirmed → In Progress

Revision history for this message

Vincent Ladeuil (vila) wrote on 2007-04-26:

Encodes prompt with the Right stdout Edit (2.8 KiB, text/plain)

Fixed in the attached patch by letting TestUIFactory encode the prompt with its own stdout.

Changed in bzr:
status:	In Progress → Fix Committed

Revision history for this message

Martin Pool (mbp) wrote on 2007-04-26: Re: [Bug 110204] Re: TestUiFactory use sys.stdout for password prompt

On 4/26/07, vila <email address hidden> wrote:
> Fixed in the attached patch by letting TestUIFactory encode the prompt
> with its own stdout.
>
> ** Attachment added: "Encodes prompt with the Right stdout"
> http://librarian.launchpad.net/7409541/110204.patch

--
Martin

Revision history for this message

John A Meinel (jameinel) wrote on 2007-05-23:

should be available in bzr.dev 2489