Some concerns about this method:

1) It creates instance variables such as `viewtest_email` that subclasses need to know, but which are not explicitly declared or documented anywhere else.

2) Nothing in this method is specific to `viewtest` (as opposed to any other kind of test).  So it's strange to include `viewtest` in the name of the method and instance variables.

A better approach would be to have a generic `create_logged_in_user()` helper method that accepts an email, password, and username.  Then the test case can specify what email/password/username it wants for the user.