Remove User.id

Now that all users are created by the create_user factory, it was easy
to do.

airlock/users.py

@@ -9,7 +9,6 @@ class User:
     is a convenience for passing that information around.
     """
 
-    id: int
     username: str
     workspaces: dict = dataclasses.field(default_factory=dict)
     output_checker: bool = dataclasses.field(default=False)
@@ -24,11 +23,10 @@ def from_session(cls, session_data):
             return
         workspaces = user.get("workspaces", dict())
         output_checker = user.get("output_checker", False)
-        return cls(user["id"], user["username"], workspaces, output_checker)
+        return cls(user["username"], workspaces, output_checker)
 
     def to_dict(self):
         return {
-            "id": self.id,
             "username": self.username,
             "workspaces": self.workspaces,
             "output_checker": self.output_checker,

tests/factories.py

@@ -18,7 +18,7 @@ def create_user(username="testuser", workspaces=None, output_checker=False):
         workspaces_dict = {
             workspace: {"project": "project"} for workspace in workspaces
         }
-    return User(1, username, workspaces_dict, output_checker)
+    return User(username, workspaces_dict, output_checker)
 
 
 def ensure_workspace(workspace_or_name):