843458 - Extra space character after city name, before the comma, e.g. "Mountain View ,"

Status: VERIFIED FIXED

(Participation Infrastructure :: Phonebook, defect)

Description

[Stephen Donner [:stephend] Not actively reading bugmail]

Attachment: Screenshot on dev

test_profile_creation is failing now on dev, likely due to https://github.com/mozilla/mozillians/commit/ff1344b800a91c91d4245adf6f4e6b29ef9c6b68, from bug 763792 (though I have no proof, as of yet).

See http://qa-selenium.mv.mozilla.com:8080/view/Mozillians/job/mozillians.dev/2935/HTML_Report/?

STR:

1. Sign in and view https://mozillians-dev.allizom.org/en-US/u/masso32098

Here's why it's failing:

We set:

location.set_state('California')
location.set_city('Mountain View')

Yet, when we do our asserts, there's an extra space in here: "Mountain View ,"

That should be "Mountain View,"

Comment 1

[Andrei Hajdukewycz [:sancus]]

This'll be fixed by https://github.com/mozilla/mozillians/pull/404

Comment 2

[[github robot]]

Commits pushed to master at https://github.com/mozilla/mozillians

https://github.com/mozilla/mozillians/commit/d93a6ee33fca43bc61c36dcd6d384fa55aa6486f
[Fix Bug 843458] Remove significant whitespace before some </a> tags

https://github.com/mozilla/mozillians/commit/f8037f134882efc3a86d136aac9b71aa1769f2c0
Merge pull request #404 from Sancus/bug843458

[Fix Bug 843458] Remove significant whitespace before some </a> tags

Comment 4

[Matt Brandt [:mbrandt]]

QA verified on dev and stage - thank you for the wicked quick patch!
mbrandt$ py.test --baseurl=http://mozillians-dev.allizom.org --credentials=/Users/mbrandt/Dropbox/workspaces/webqa-credentials/mozillians.yaml --destructive -ktest_profile_creation
== test session starts ==
platform darwin -- Python 2.7.2 -- pytest-2.2.4
collected 20 items 

tests/test_profile.py X
== 19 tests deselected by '-ktest_profile_creation' ==
== 19 deselected, 1 xpassed in 1.20 seconds ==