1032729 - Update UserProfileSerializer to include username and privacy_policy_accepted fields

Status: VERIFIED FIXED

(Mozilla QA Graveyard :: One and Done, defect)

Description

[Pankaj Malhotra (:bitgeeky)]

Updated UserProfileSerializer and corresponding Unit tests to include the newly added username and privacy_policy_accepted fields.

Comment 1

[Pankaj Malhotra (:bitgeeky)]

Attachment: Link to pull request.

Comment 2

[Bob Silverberg [:bsilverberg]]

Comment on attachment 8448586 [details] [review]
Link to pull request.

Good work, bitgeeky. Just a few nits to address and it should be good to go.

Comment 3

[Pankaj Malhotra (:bitgeeky)]

Comment on attachment 8448586 [details] [review]
Link to pull request.

Made the suggested changes. Please review.

Comment 4

[Bob Silverberg [:bsilverberg]]

Comment on attachment 8448586 [details] [review]
Link to pull request.

Landed in https://github.com/mozilla/oneanddone/commit/f21d24eccfb7d6c3309580c051fc23bc12a2b3d9

Comment 5

[Bob Silverberg [:bsilverberg]]

Oops, there is a problem here in that the User.username needs to be set too, not just Profile.username, as we need to ensure that User.username is unique too. I'm not going to back out the commit, so please make the change based on the latest version of master and submit a new pull request that fixes this. Also please add a test for this, which verifies that we can create two users in a row, and which does not result in a unique constraint error. Please write that test first so that when you run it you see it fail - that way you'll know you wrote the test correctly. Then fix the code to fix the error, after which the test should pass.

Comment 6

[Pankaj Malhotra (:bitgeeky)]

Attachment: Link to pull request.

https://github.com/mozilla/oneanddone/pull/148

Thanks for pointing this out Bob. It would have created lot of problem later.
I have added User.username field to the User Serializer which would fix this issue.

Comment 7

[Bob Silverberg [:bsilverberg]]

Comment on attachment 8449591 [details] [review]
Link to pull request.

Landed in https://github.com/mozilla/oneanddone/commit/749805df85bba22881e1ffb00be59ee8ff531ec9

Comment 8

[Bob Silverberg [:bsilverberg]]

Thanks bitgeeky. One thing to remember though, please use the bug number and description for the first line of your commit message in the future.

Comment 9

[Rebecca Billings [:rbillings]]

Please add verification steps or add "qa-" to the whiteboard. Thanks

Comment 10

[Rebecca Billings [:rbillings]]

Verifying as [qa-]