subscription_list: Exclude archived channels from list

Author: chimnayajith

Diff for: lib/widgets/subscription_list.dart

@@ -85,6 +85,7 @@ class _SubscriptionListPageBodyState extends State<SubscriptionListPageBody> wit
     final List<Subscription> pinned = [];
     final List<Subscription> unpinned = [];
     for (final subscription in store.subscriptions.values) {
+      if (subscription.isArchived) continue;
       if (subscription.pinToTop) {
         pinned.add(subscription);
       } else {
@@ -188,6 +189,8 @@ class _SubscriptionList extends StatelessWidget {
 
   @override
   Widget build(BuildContext context) {
+    assert(subscriptions.every((subscription) => !subscription.isArchived));
+
     return SliverList.builder(
       itemCount: subscriptions.length,
       itemBuilder: (BuildContext context, int index) {

Diff for: test/widgets/subscription_list_test.dart

@@ -95,6 +95,25 @@ void main() {
     check(isUnpinnedHeaderInTree()).isTrue();
   });
 
+  testWidgets('archived subscriptions are filtered out', (tester) async {
+    await setupStreamListPage(tester, subscriptions: [
+      eg.subscription(eg.stream(streamId: 1, isArchived: true), pinToTop: true),
+      eg.subscription(eg.stream(streamId: 2, isArchived: true), pinToTop: false),
+      eg.subscription(eg.stream(streamId: 3), pinToTop: true),
+      eg.subscription(eg.stream(streamId: 4), pinToTop: false),
+    ]);
+
+    check(getItemCount()).equals(2);
+    check(isPinnedHeaderInTree()).isTrue();
+    check(isUnpinnedHeaderInTree()).isTrue();
+
+    check(find.text('stream 1')).findsNothing();
+    check(find.text('stream 2')).findsNothing();
+
+    check(find.text('stream 3')).findsOne();
+    check(find.text('stream 4')).findsOne();
+  });
+
   group('subscription sorting', () {
     Iterable<int> listedStreamIds(WidgetTester tester) => tester
       .widgetList<SubscriptionItem>(find.byType(SubscriptionItem))