Add directly as mentor in Group (#234)

* feat(api): add support to directly add as a mentor in the group

Signed-off-by: Aman <aman2@me.iitr.ac.in>

* bfix: fix ci

Signed-off-by: Aman <aman2@me.iitr.ac.in>

.github/workflows/ci.yml

@@ -23,7 +23,9 @@ jobs:
       # Setup the flutter environment.
       - uses: subosito/flutter-action@v1
         with:
-          channel: "stable"
+          # Temporary fix to failing CI
+          flutter-version: '3.0.4'
+          # channel: "stable"
 
       # Get flutter dependencies.
       - run: flutter pub get

lib/services/API/group_members_api.dart

@@ -10,7 +10,7 @@ abstract class GroupMembersApi {
   Future<GroupMembers>? fetchGroupMembers(String groupId);
 
   Future<AddGroupMembersResponse>? addGroupMembers(
-      String groupId, String listOfMails);
+      String groupId, String listOfMails, bool isMentor);
 
   Future updateMemberRole(String memberId, bool isMentor);
 
@@ -43,10 +43,10 @@ class HttpGroupMembersApi implements GroupMembersApi {
 
   @override
   Future<AddGroupMembersResponse>? addGroupMembers(
-      String groupId, String listOfMails) async {
+      String groupId, String listOfMails, bool isMentor) async {
     var endpoint = '/groups/$groupId/members';
     var uri = EnvironmentConfig.CV_API_BASE_URL + endpoint;
-    var json = {'emails': listOfMails};
+    var json = {'emails': listOfMails, 'mentor': isMentor};
 
     try {
       ApiUtils.addTokenToHeaders(headers);

lib/ui/views/groups/group_details_view.dart

@@ -114,14 +114,14 @@ class _GroupDetailsViewState extends State<GroupDetailsView> {
     );
   }
 
-  Future<void> onAddMemberPressed(BuildContext context) async {
+  Future<void> onAddMemberPressed(BuildContext context, bool isMentor) async {
     if (Validators.validateAndSaveForm(_formKey)) {
       FocusScope.of(context).requestFocus(FocusNode());
       Navigator.pop(context);
 
       _dialogService.showCustomProgressDialog(title: 'Adding');
 
-      await _model.addMembers(_recievedGroup.id, _emails!);
+      await _model.addMembers(_recievedGroup.id, _emails!, isMentor);
 
       _dialogService.popDialog();
 
@@ -189,7 +189,8 @@ class _GroupDetailsViewState extends State<GroupDetailsView> {
             ),
             CVFlatButton(
               key: addButtonGlobalKey,
-              triggerFunction: onAddMemberPressed,
+              triggerFunction: (context) =>
+                  onAddMemberPressed(context, !member),
               context: context,
               buttonText: 'ADD',
             ),

lib/viewmodels/groups/group_details_viewmodel.dart

@@ -109,11 +109,11 @@ class GroupDetailsViewModel extends BaseModel {
     }
   }
 
-  Future addMembers(String groupId, String emails) async {
+  Future addMembers(String groupId, String emails, bool isMentor) async {
     setStateFor(ADD_GROUP_MEMBERS, ViewState.Busy);
     try {
       var addGroupMembers =
-          await _groupMembersApi.addGroupMembers(groupId, emails);
+          await _groupMembersApi.addGroupMembers(groupId, emails, isMentor);
 
       var _addedMembers = addGroupMembers!.added.join(', ');
       var _pendingMembers = addGroupMembers.pending.join(', ');

test/service_tests/group_members_api_test.dart

@@ -56,23 +56,24 @@ void main() {
             Response(jsonEncode(mockAddGroupMembersResponse), 201)));
         var _groupMembersApi = HttpGroupMembersApi();
 
-        expect((await _groupMembersApi.addGroupMembers('1', '')).toString(),
+        expect(
+            (await _groupMembersApi.addGroupMembers('1', '', false)).toString(),
             _addGroupMembersResponse.toString());
       });
 
       test('When called & http client throws Exceptions', () async {
         var _groupMembersApi = HttpGroupMembersApi();
 
         ApiUtils.client = MockClient((_) => throw UnauthorizedException(''));
-        expect(_groupMembersApi.addGroupMembers('1', ''),
+        expect(_groupMembersApi.addGroupMembers('1', '', false),
             throwsA(isInstanceOf<Failure>()));
 
         ApiUtils.client = MockClient((_) => throw NotFoundException(''));
-        expect(_groupMembersApi.addGroupMembers('1', ''),
+        expect(_groupMembersApi.addGroupMembers('1', '', false),
             throwsA(isInstanceOf<Failure>()));
 
         ApiUtils.client = MockClient((_) => throw Exception(''));
-        expect(_groupMembersApi.addGroupMembers('1', ''),
+        expect(_groupMembersApi.addGroupMembers('1', '', false),
             throwsA(isInstanceOf<Failure>()));
       });
     });

test/viewmodel_tests/groups/group_details_viewmodel_test.dart

@@ -64,19 +64,19 @@ void main() {
       test('When called & service returns success response', () async {
         var _mockGroupMembersApi = getAndRegisterGroupMembersApiMock();
         when(_mockGroupMembersApi.addGroupMembers(
-                '1', 'test@test.com,pending@test.com,invalid@test.com'))
+                '1', 'test@test.com,pending@test.com,invalid@test.com', false))
             .thenAnswer((_) => Future.value(_addedMembers));
 
         when(_mockGroupMembersApi.fetchGroupMembers('1'))
             .thenAnswer((_) => Future.value(_groupMembers));
 
         var _model = GroupDetailsViewModel();
         await _model.addMembers(
-            '1', 'test@test.com,pending@test.com,invalid@test.com');
+            '1', 'test@test.com,pending@test.com,invalid@test.com', false);
 
         // verify API call is made..
         verify(_mockGroupMembersApi.addGroupMembers(
-            '1', 'test@test.com,pending@test.com,invalid@test.com'));
+            '1', 'test@test.com,pending@test.com,invalid@test.com', false));
         verify(_mockGroupMembersApi.fetchGroupMembers('1'));
         expect(_model.stateFor(_model.ADD_GROUP_MEMBERS), ViewState.Success);
 
@@ -88,12 +88,12 @@ void main() {
       test('When called & service returns error', () async {
         var _mockGroupMembersApi = getAndRegisterGroupMembersApiMock();
         when(_mockGroupMembersApi.addGroupMembers(
-                '1', 'test@test.com,pending@test.com,invalid@test.com'))
+                '1', 'test@test.com,pending@test.com,invalid@test.com', false))
             .thenThrow(Failure('Some Error Occurred!'));
 
         var _model = GroupDetailsViewModel();
         await _model.addMembers(
-            '1', 'test@test.com,pending@test.com,invalid@test.com');
+            '1', 'test@test.com,pending@test.com,invalid@test.com', false);
 
         // verify Error ViewState with proper error message..
         expect(_model.stateFor(_model.ADD_GROUP_MEMBERS), ViewState.Error);