Cleanup user creation in tests

Review Request #9458 — Created Dec. 19, 2017 and submitted

Information

Review Board
release-3.0.x
879fdfd...

Reviewers

We now are always careful to provide an e-mail to users in tests. Also,
calls have been updated to use User.objects.create_user over
User.objects.create where appropriate. This was prompted by a change
in Djblets (see https://reviews.reviewboard.org/r/9445/) which started
causing unit test failures due to Users without e-mail addresses
being serialized. So to be extra safe, I've gone in and added e-mail
addresses to all calls that were lacking them.

Ran unit tests.

Description From Last Updated

The description doesn't tell me the "why" of this. What's this solving?

chipx86chipx86

E127 continuation line over-indented for visual indent

reviewbotreviewbot

E127 continuation line over-indented for visual indent

reviewbotreviewbot

E127 continuation line over-indented for visual indent

reviewbotreviewbot

user22? not user1?

daviddavid
Checks run (1 failed, 1 succeeded)
flake8 failed.
JSHint passed.

flake8

brennie
david
  1. 
      
  2. user22? not user1?

  3. 
      
brennie
chipx86
  1. 
      
  2. The description doesn't tell me the "why" of this. What's this solving?

  3. 
      
brennie
brennie
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to release-3.0.x (3b34522)
Loading...