[PR #6298] [MERGED] test: add grid integration tests #7299

New issue

Closed

opened 2026-03-23 23:18:59 +00:00 by mirror · 0 comments

mirror commented

2026-03-23 23:18:59 +00:00

Owner

📋 Pull Request Information

Original PR: https://github.com/AppFlowy-IO/AppFlowy/pull/6298
Author: @richardshiue
Created: 9/13/2024
Status: ✅ Merged
Merged: 9/16/2024
Merged by: @richardshiue

Base: main ← Head: test/integration-test-reorganization

📝 Commits (5)

2e5df4c test: run board integration tests
fdb8874 test: add grid integration tests
c5b97fb test: fix delete group test
7d70e35 chore: fix rust-lib tests
b4746e0 chore: comment out failing test fix tests and remove tOdOS

📊 Changes

23 files changed (+910 additions, -173 deletions)

View changed files

➕ frontend/appflowy_flutter/assets/test/workspaces/database/v069.afdb (+14 -0)
📝 frontend/appflowy_flutter/integration_test/desktop/board/board_field_test.dart (+1 -0)
📝 frontend/appflowy_flutter/integration_test/desktop/board/board_hide_groups_test.dart (+40 -39)
📝 frontend/appflowy_flutter/integration_test/desktop/board/board_test_runner.dart (+4 -0)
📝 frontend/appflowy_flutter/integration_test/desktop/database/database_filter_test.dart (+22 -22)
📝 frontend/appflowy_flutter/integration_test/desktop/database/database_row_page_test.dart (+2 -2)
➖ frontend/appflowy_flutter/integration_test/desktop/database/database_row_test.dart (+0 -66)
📝 frontend/appflowy_flutter/integration_test/desktop/database/database_share_test.dart (+1 -1)
📝 frontend/appflowy_flutter/integration_test/desktop/database/database_sort_test.dart (+6 -6)
➕ frontend/appflowy_flutter/integration_test/desktop/grid/grid_create_row_test.dart (+203 -0)
➕ frontend/appflowy_flutter/integration_test/desktop/grid/grid_filter_and_sort_test.dart (+120 -0)
➕ frontend/appflowy_flutter/integration_test/desktop/grid/grid_reopen_test.dart (+183 -0)
➕ frontend/appflowy_flutter/integration_test/desktop/grid/grid_reorder_row_test.dart (+214 -0)
➕ frontend/appflowy_flutter/integration_test/desktop/grid/grid_test_extensions.dart (+13 -0)
📝 frontend/appflowy_flutter/integration_test/desktop_runner_2.dart (+0 -2)
📝 frontend/appflowy_flutter/integration_test/desktop_runner_3.dart (+10 -0)
📝 frontend/appflowy_flutter/integration_test/shared/database_test_op.dart (+53 -25)
📝 frontend/appflowy_flutter/lib/plugins/database/application/row/row_cache.dart (+5 -1)
📝 frontend/appflowy_flutter/lib/plugins/database/grid/presentation/grid_page.dart (+9 -6)
📝 frontend/appflowy_flutter/lib/plugins/database/grid/presentation/widgets/row/action.dart (+1 -0)

...and 3 more files

📄 Description

NOT READY 4 MERGE!!!!!
NOT READY 4 MERGE!!!!!
NOT READY 4 MERGE!!!!!

it is now

Feature Preview

PR Checklist

My code adheres to AppFlowy's Conventions
I've listed at least one issue that this PR fixes in the description above.
I've added a test(s) to validate changes in this PR, or this PR only contains semantic changes.
All existing tests are passing.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/AppFlowy-IO/AppFlowy/pull/6298 **Author:** [@richardshiue](https://github.com/richardshiue) **Created:** 9/13/2024 **Status:** ✅ Merged **Merged:** 9/16/2024 **Merged by:** [@richardshiue](https://github.com/richardshiue) **Base:** `main` ← **Head:** `test/integration-test-reorganization` --- ### 📝 Commits (5) - [`2e5df4c`](https://github.com/AppFlowy-IO/AppFlowy/commit/2e5df4c2c80b446bfe1927c7a18b395a8ff8bf58) test: run board integration tests - [`fdb8874`](https://github.com/AppFlowy-IO/AppFlowy/commit/fdb8874c75a05ab6fc857069f2dc5af362fc7d73) test: add grid integration tests - [`c5b97fb`](https://github.com/AppFlowy-IO/AppFlowy/commit/c5b97fbbef2e37e46a162f1a4f3c57abbbbe817a) test: fix delete group test - [`7d70e35`](https://github.com/AppFlowy-IO/AppFlowy/commit/7d70e35f396cf226dc96b8c1e2e926edb28475f4) chore: fix rust-lib tests - [`b4746e0`](https://github.com/AppFlowy-IO/AppFlowy/commit/b4746e08ef1d2a15d14552c854f56bf219993c8b) chore: comment out failing test fix tests and remove tOdOS ### 📊 Changes **23 files changed** (+910 additions, -173 deletions) <details> <summary>View changed files</summary> ➕ `frontend/appflowy_flutter/assets/test/workspaces/database/v069.afdb` (+14 -0) 📝 `frontend/appflowy_flutter/integration_test/desktop/board/board_field_test.dart` (+1 -0) 📝 `frontend/appflowy_flutter/integration_test/desktop/board/board_hide_groups_test.dart` (+40 -39) 📝 `frontend/appflowy_flutter/integration_test/desktop/board/board_test_runner.dart` (+4 -0) 📝 `frontend/appflowy_flutter/integration_test/desktop/database/database_filter_test.dart` (+22 -22) 📝 `frontend/appflowy_flutter/integration_test/desktop/database/database_row_page_test.dart` (+2 -2) ➖ `frontend/appflowy_flutter/integration_test/desktop/database/database_row_test.dart` (+0 -66) 📝 `frontend/appflowy_flutter/integration_test/desktop/database/database_share_test.dart` (+1 -1) 📝 `frontend/appflowy_flutter/integration_test/desktop/database/database_sort_test.dart` (+6 -6) ➕ `frontend/appflowy_flutter/integration_test/desktop/grid/grid_create_row_test.dart` (+203 -0) ➕ `frontend/appflowy_flutter/integration_test/desktop/grid/grid_filter_and_sort_test.dart` (+120 -0) ➕ `frontend/appflowy_flutter/integration_test/desktop/grid/grid_reopen_test.dart` (+183 -0) ➕ `frontend/appflowy_flutter/integration_test/desktop/grid/grid_reorder_row_test.dart` (+214 -0) ➕ `frontend/appflowy_flutter/integration_test/desktop/grid/grid_test_extensions.dart` (+13 -0) 📝 `frontend/appflowy_flutter/integration_test/desktop_runner_2.dart` (+0 -2) 📝 `frontend/appflowy_flutter/integration_test/desktop_runner_3.dart` (+10 -0) 📝 `frontend/appflowy_flutter/integration_test/shared/database_test_op.dart` (+53 -25) 📝 `frontend/appflowy_flutter/lib/plugins/database/application/row/row_cache.dart` (+5 -1) 📝 `frontend/appflowy_flutter/lib/plugins/database/grid/presentation/grid_page.dart` (+9 -6) 📝 `frontend/appflowy_flutter/lib/plugins/database/grid/presentation/widgets/row/action.dart` (+1 -0) _...and 3 more files_ </details> ### 📄 Description ~~NOT READY 4 MERGE!!!!! NOT READY 4 MERGE!!!!! NOT READY 4 MERGE!!!!!~~ it is now ### Feature Preview  ---  #### PR Checklist - [x] My code adheres to [AppFlowy's Conventions](https://docs.appflowy.io/docs/documentation/software-contributions/conventions) - [ ] I've listed at least one issue that this PR fixes in the description above. - [x] I've added a test(s) to validate changes in this PR, or this PR only contains semantic changes. - [ ] All existing tests are passing. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>