Skip to content

Commit 4a62569

Browse files
committed
fix: fix incorrect pagination verifications
1 parent 65d11f2 commit 4a62569

16 files changed

+9
-83
lines changed

packages/stream_feeds/test/state/activity_comment_list_test.dart

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -96,11 +96,6 @@ void main() {
9696
// Verify state was updated with merged comments
9797
expect(tester.activityCommentListState.comments, hasLength(2));
9898
expect(tester.activityCommentListState.canLoadMore, isFalse);
99-
},
100-
verify: (tester) {
101-
final nextPageQuery = tester.activityCommentList.query.copyWith(
102-
next: tester.activityCommentListState.pagination?.next,
103-
);
10499

105100
tester.verifyApi(
106101
(api) => api.getComments(

packages/stream_feeds/test/state/activity_list_test.dart

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -155,11 +155,6 @@ void main() {
155155
// Verify state was updated with merged activities
156156
expect(tester.activityListState.activities, hasLength(2));
157157
expect(tester.activityListState.canLoadMore, isFalse);
158-
},
159-
verify: (tester) {
160-
final nextPageQuery = tester.activityList.query.copyWith(
161-
next: tester.activityListState.pagination?.next,
162-
);
163158

164159
tester.verifyApi(
165160
(api) => api.queryActivities(

packages/stream_feeds/test/state/activity_reaction_list_test.dart

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -85,11 +85,6 @@ void main() {
8585
// Verify state was updated with merged reactions
8686
expect(tester.activityReactionListState.reactions, hasLength(2));
8787
expect(tester.activityReactionListState.canLoadMore, isFalse);
88-
},
89-
verify: (tester) {
90-
final nextPageQuery = tester.activityReactionList.query.copyWith(
91-
next: tester.activityReactionListState.pagination?.next,
92-
);
9388

9489
tester.verifyApi(
9590
(api) => api.queryActivityReactions(

packages/stream_feeds/test/state/activity_test.dart

Lines changed: 9 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -73,15 +73,16 @@ void main() {
7373
// Verify state was updated
7474
expect(tester.activityState.comments, hasLength(2));
7575
expect(tester.activityState.canLoadMoreComments, isFalse);
76+
77+
tester.verifyApi(
78+
(api) => api.getComments(
79+
next: 'next-cursor',
80+
objectId: activityId,
81+
objectType: 'activity',
82+
depth: 3,
83+
),
84+
);
7685
},
77-
verify: (tester) => tester.verifyApi(
78-
(api) => api.getComments(
79-
next: 'next-cursor',
80-
objectId: activityId,
81-
objectType: 'activity',
82-
depth: 3,
83-
),
84-
),
8586
);
8687

8788
activityTest(

packages/stream_feeds/test/state/bookmark_folder_list_test.dart

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -71,11 +71,6 @@ void main() {
7171
// Verify state was updated with merged folders
7272
expect(tester.bookmarkFolderListState.bookmarkFolders, hasLength(2));
7373
expect(tester.bookmarkFolderListState.canLoadMore, isFalse);
74-
},
75-
verify: (tester) {
76-
final nextPageQuery = tester.bookmarkFolderList.query.copyWith(
77-
next: tester.bookmarkFolderListState.pagination?.next,
78-
);
7974

8075
tester.verifyApi(
8176
(api) => api.queryBookmarkFolders(

packages/stream_feeds/test/state/bookmark_list_test.dart

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -77,11 +77,6 @@ void main() {
7777
// Verify state was updated with merged bookmarks
7878
expect(tester.bookmarkListState.bookmarks, hasLength(2));
7979
expect(tester.bookmarkListState.canLoadMore, isFalse);
80-
},
81-
verify: (tester) {
82-
final nextPageQuery = tester.bookmarkList.query.copyWith(
83-
next: tester.bookmarkListState.pagination?.next,
84-
);
8580

8681
tester.verifyApi(
8782
(api) => api.queryBookmarks(

packages/stream_feeds/test/state/comment_list_test.dart

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -69,11 +69,6 @@ void main() {
6969
// Verify state was updated with merged comments
7070
expect(tester.commentListState.comments, hasLength(4));
7171
expect(tester.commentListState.canLoadMore, isFalse);
72-
},
73-
verify: (tester) {
74-
final nextPageQuery = tester.commentList.query.copyWith(
75-
next: tester.commentListState.pagination?.next,
76-
);
7772

7873
tester.verifyApi(
7974
(api) => api.queryComments(

packages/stream_feeds/test/state/comment_reaction_list_test.dart

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -76,11 +76,6 @@ void main() {
7676
// Verify state was updated with merged reactions
7777
expect(tester.commentReactionListState.reactions, hasLength(2));
7878
expect(tester.commentReactionListState.canLoadMore, isFalse);
79-
},
80-
verify: (tester) {
81-
final nextPageQuery = tester.commentReactionList.query.copyWith(
82-
next: tester.commentReactionListState.pagination?.next,
83-
);
8479

8580
tester.verifyApi(
8681
(api) => api.queryCommentReactions(

packages/stream_feeds/test/state/comment_reply_list_test.dart

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -96,11 +96,6 @@ void main() {
9696
// Verify state was updated with merged replies
9797
expect(tester.commentReplyListState.replies, hasLength(2));
9898
expect(tester.commentReplyListState.canLoadMore, isFalse);
99-
},
100-
verify: (tester) {
101-
final nextPageQuery = tester.commentReplyList.query.copyWith(
102-
next: tester.commentReplyListState.pagination?.next,
103-
);
10499

105100
tester.verifyApi(
106101
(api) => api.getCommentReplies(

packages/stream_feeds/test/state/feed_list_test.dart

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -60,11 +60,6 @@ void main() {
6060
// Verify state was updated with merged feeds
6161
expect(tester.feedListState.feeds, hasLength(2));
6262
expect(tester.feedListState.canLoadMore, isFalse);
63-
},
64-
verify: (tester) {
65-
final nextPageQuery = tester.feedList.query.copyWith(
66-
next: tester.feedListState.pagination?.next,
67-
);
6863

6964
tester.verifyApi(
7065
(api) => api.queryFeeds(

0 commit comments

Comments
 (0)