api: Cut ZulipStream.canRemoveSubscribersGroup as it changes format

This field is becoming a group-setting value.  We haven't yet
implemented those (that's #814), so we can no longer handle this field
until we do that work.  Conveniently we weren't actually making any
use of this field, though -- so for now, just ignore it.

Fixes: #1082

Author: gnprice

lib/api/model/model.dart

@@ -342,19 +342,11 @@ class ZulipStream {
   ChannelPostPolicy channelPostPolicy;
   // final bool isAnnouncementOnly; // deprecated for `channelPostPolicy`; ignore
 
-  // TODO(server-6): `canRemoveSubscribersGroupId` added in FL 142
-  // TODO(server-8): in FL 197 renamed to `canRemoveSubscribersGroup`
-  @JsonKey(readValue: _readCanRemoveSubscribersGroup)
-  int? canRemoveSubscribersGroup;
+  // GroupSettingsValue canRemoveSubscribersGroup; // TODO(#814)
 
   // TODO(server-8): added in FL 199, was previously only on [Subscription] objects
   int? streamWeeklyTraffic;
 
-  static int? _readCanRemoveSubscribersGroup(Map<dynamic, dynamic> json, String key) {
-    return (json[key] as int?)
-      ?? (json['can_remove_subscribers_group_id'] as int?);
-  }
-
   ZulipStream({
     required this.streamId,
     required this.name,
@@ -367,7 +359,6 @@ class ZulipStream {
     required this.historyPublicToSubscribers,
     required this.messageRetentionDays,
     required this.channelPostPolicy,
-    required this.canRemoveSubscribersGroup,
     required this.streamWeeklyTraffic,
   });
 
@@ -478,7 +469,6 @@ class Subscription extends ZulipStream {
     required super.historyPublicToSubscribers,
     required super.messageRetentionDays,
     required super.channelPostPolicy,
-    required super.canRemoveSubscribersGroup,
     required super.streamWeeklyTraffic,
     required this.desktopNotifications,
     required this.emailNotifications,

lib/api/model/model.g.dart

@@ -281,7 +281,7 @@ class ChannelStoreImpl with ChannelStore {
             stream.channelPostPolicy = event.value as ChannelPostPolicy;
           case ChannelPropertyName.canRemoveSubscribersGroup:
           case ChannelPropertyName.canRemoveSubscribersGroupId:
-            stream.canRemoveSubscribersGroup = event.value as int?;
+            break; // not tracking this property
           case ChannelPropertyName.streamWeeklyTraffic:
             stream.streamWeeklyTraffic = event.value as int?;
         }

lib/model/channel.dart

@@ -22,7 +22,6 @@ extension UserChecks on Subject<User> {
 }
 
 extension ZulipStreamChecks on Subject<ZulipStream> {
-  Subject<int?> get canRemoveSubscribersGroup => has((e) => e.canRemoveSubscribersGroup, 'canRemoveSubscribersGroup');
 }
 
 extension MessageChecks on Subject<Message> {

test/api/model/model_checks.dart

@@ -67,43 +67,6 @@ void main() {
     });
   });
 
-  group('ZulipStream.canRemoveSubscribersGroup', () {
-    final Map<String, dynamic> baseJson = Map.unmodifiable({
-      'stream_id': 123,
-      'name': 'A stream',
-      'description': 'A description',
-      'rendered_description': '<p>A description</p>',
-      'date_created': 1686774898,
-      'first_message_id': null,
-      'invite_only': false,
-      'is_web_public': false,
-      'history_public_to_subscribers': true,
-      'message_retention_days': null,
-      'stream_post_policy': ChannelPostPolicy.any.apiValue,
-      // 'can_remove_subscribers_group': null,
-      'stream_weekly_traffic': null,
-    });
-
-    test('smoke', () {
-      check(ZulipStream.fromJson({ ...baseJson,
-        'can_remove_subscribers_group': 123,
-      })).canRemoveSubscribersGroup.equals(123);
-    });
-
-    // TODO(server-8): field renamed in FL 197
-    test('support old can_remove_subscribers_group_id', () {
-      check(ZulipStream.fromJson({ ...baseJson,
-        'can_remove_subscribers_group_id': 456,
-      })).canRemoveSubscribersGroup.equals(456);
-    });
-
-    // TODO(server-6): field added in FL 142
-    test('support field missing', () {
-      check(ZulipStream.fromJson({ ...baseJson,
-      })).canRemoveSubscribersGroup.isNull();
-    });
-  });
-
   group('Subscription', () {
     test('converts color to int', () {
       Subscription subWithColor(String color) {

test/api/model/model_test.dart

@@ -212,7 +212,6 @@ ZulipStream stream({
   bool? historyPublicToSubscribers,
   int? messageRetentionDays,
   ChannelPostPolicy? channelPostPolicy,
-  int? canRemoveSubscribersGroup,
   int? streamWeeklyTraffic,
 }) {
   _checkPositive(streamId, 'stream ID');
@@ -232,7 +231,6 @@ ZulipStream stream({
     historyPublicToSubscribers: historyPublicToSubscribers ?? true,
     messageRetentionDays: messageRetentionDays,
     channelPostPolicy: channelPostPolicy ?? ChannelPostPolicy.any,
-    canRemoveSubscribersGroup: canRemoveSubscribersGroup ?? 123,
     streamWeeklyTraffic: streamWeeklyTraffic,
   );
 }
@@ -270,7 +268,6 @@ Subscription subscription(
     historyPublicToSubscribers: stream.historyPublicToSubscribers,
     messageRetentionDays: stream.messageRetentionDays,
     channelPostPolicy: stream.channelPostPolicy,
-    canRemoveSubscribersGroup: stream.canRemoveSubscribersGroup,
     streamWeeklyTraffic: stream.streamWeeklyTraffic,
     desktopNotifications: desktopNotifications ?? false,
     emailNotifications: emailNotifications ?? false,