Merge pull request #349 from dod-ccpo/default-role-#160832928

do not set default kwarg for atat role when creating users

atst/domain/authnid/__init__.py

@@ -31,7 +31,9 @@ class AuthenticationContext:
 
         except NotFoundError:
             email = self._get_user_email()
-            return Users.create(**{"email": email, **self.parsed_sdn})
+            return Users.create(
+                atat_role_name="default", email=email, **self.parsed_sdn
+            )
 
     def _get_user_email(self):
         try:

atst/domain/users.py

@@ -28,7 +28,7 @@ class Users(object):
         return user
 
     @classmethod
-    def create(cls, atat_role_name="developer", **kwargs):
+    def create(cls, atat_role_name=None, **kwargs):
         atat_role = Roles.get(atat_role_name)
 
         try:

atst/domain/workspaces/workspaces.py

@@ -85,6 +85,7 @@ class Workspaces(object):
             first_name=data["first_name"],
             last_name=data["last_name"],
             email=data["email"],
+            atat_role_name="default",
         )
         return Workspaces.add_member(workspace, new_user, data["workspace_role"])