refactor(code_push_protocol): create collaborators via email (#506)

packages/shorebird_code_push_client/lib/src/code_push_client.dart

@@ -46,11 +46,11 @@ class CodePushClient {
   /// Collaborators can manage the app including its releases and patches.
   Future<void> createAppCollaborator({
     required String appId,
-    required int userId,
+    required String email,
   }) async {
     final response = await _httpClient.post(
       Uri.parse('$_v1/apps/$appId/collaborators'),
-      body: json.encode(CreateAppCollaboratorRequest(userId: userId).toJson()),
+      body: json.encode(CreateAppCollaboratorRequest(email: email).toJson()),
     );
 
     if (response.statusCode != HttpStatus.created) {

packages/shorebird_code_push_client/test/src/code_push_client_test.dart

@@ -55,7 +55,7 @@ void main() {
 
     group('createAppCollaborator', () {
       const appId = 'test-app-id';
-      const userId = 42;
+      const email = '[email protected]';
 
       test('throws an exception if the http request fails (unknown)', () async {
         when(
@@ -69,7 +69,7 @@ void main() {
         );
 
         expect(
-          codePushClient.createAppCollaborator(appId: appId, userId: userId),
+          codePushClient.createAppCollaborator(appId: appId, email: email),
           throwsA(
             isA<CodePushException>().having(
               (e) => e.message,
@@ -95,7 +95,7 @@ void main() {
         );
 
         expect(
-          codePushClient.createAppCollaborator(appId: appId, userId: userId),
+          codePushClient.createAppCollaborator(appId: appId, email: email),
           throwsA(
             isA<CodePushException>().having(
               (e) => e.message,
@@ -115,10 +115,7 @@ void main() {
           ),
         ).thenAnswer((_) async => http.Response('', HttpStatus.created));
 
-        await codePushClient.createAppCollaborator(
-          appId: appId,
-          userId: userId,
-        );
+        await codePushClient.createAppCollaborator(appId: appId, email: email);
 
         final uri = verify(
           () => httpClient.post(

packages/shorebird_code_push_protocol/lib/src/messages/create_app_collaborator/create_app_collaborator_request.dart

@@ -8,7 +8,7 @@ part 'create_app_collaborator_request.g.dart';
 @JsonSerializable()
 class CreateAppCollaboratorRequest {
   /// {@macro create_app_collaborator_request}
-  const CreateAppCollaboratorRequest({required this.userId});
+  const CreateAppCollaboratorRequest({required this.email});
 
   /// Converts a Map<String, dynamic> to a [CreateAppCollaboratorRequest]
   factory CreateAppCollaboratorRequest.fromJson(Map<String, dynamic> json) =>
@@ -17,6 +17,6 @@ class CreateAppCollaboratorRequest {
   /// Converts a [CreateAppCollaboratorRequest] to a Map<String, dynamic>
   Map<String, dynamic> toJson() => _$CreateAppCollaboratorRequestToJson(this);
 
-  /// The unique identifier of the collaborator to add.
-  final int userId;
+  /// The email of the collaborator to add.
+  final String email;
 }

packages/shorebird_code_push_protocol/test/src/messages/create_app_collaborator/create_app_collaborator_request_test.dart

@@ -4,7 +4,9 @@ import 'package:test/test.dart';
 void main() {
   group(CreateAppCollaboratorRequest, () {
     test('can be (de)serialized', () {
-      const request = CreateAppCollaboratorRequest(userId: 42);
+      const request = CreateAppCollaboratorRequest(
+        email: '[email protected]',
+      );
       expect(
         CreateAppCollaboratorRequest.fromJson(request.toJson()).toJson(),
         equals(request.toJson()),