[PR #7658] chore: enable integration test on windows #8026

New issue

Open

opened 2026-03-23 23:22:16 +00:00 by mirror · 0 comments

mirror commented

2026-03-23 23:22:16 +00:00

Owner

📋 Pull Request Information

Original PR: https://github.com/AppFlowy-IO/AppFlowy/pull/7658
Author: @LucasXu0
Created: 3/31/2025
Status: 🔄 Open

Base: main ← Head: enable_ci_on_windows_linux_2

📝 Commits (10+)

ce91b0c chore: enable integration test on windows and linux
eb9037a Merge branch 'main' into enable_ci_on_windows_linux_2
f308c30 fix: file name too long
6664fc9 Merge branch 'main' into enable_ci_on_windows_linux_2
3d3b2b3 fix: link error
39354ae Merge branch 'main' into enable_ci_on_windows_linux_2
befb8d1 feat: add new windows tests
74fd436 fix: windows assertions
9f9f7bb fix: unable to accept the response from 'mark longer' (#7725)
ba57b9a fix: otp launch review issues (#7730)

📊 Changes

80 files changed (+1513 additions, -825 deletions)

View changed files

📝 .github/actions/flutter_build/action.yml (+1 -1)
📝 .github/actions/flutter_integration_test/action.yml (+38 -10)
📝 .github/workflows/flutter_ci.yaml (+5 -3)
➕ .github/workflows/windows_flutter_ci.yml (+116 -0)
📝 frontend/appflowy_flutter/integration_test/desktop/document/document_copy_and_paste_test.dart (+3 -13)
📝 frontend/appflowy_flutter/integration_test/desktop/document/document_create_and_delete_test.dart (+2 -0)
📝 frontend/appflowy_flutter/integration_test/desktop/document/document_link_preview_test.dart (+5 -4)
📝 frontend/appflowy_flutter/integration_test/shared/common_operations.dart (+2 -4)
📝 frontend/appflowy_flutter/lib/core/helpers/url_launcher.dart (+12 -10)
📝 frontend/appflowy_flutter/lib/mobile/presentation/base/view_page/more_bottom_sheet.dart (+1 -1)
📝 frontend/appflowy_flutter/lib/mobile/presentation/bottom_sheet/bottom_sheet_view_page.dart (+3 -1)
📝 frontend/appflowy_flutter/lib/mobile/presentation/home/mobile_home_setting_page.dart (+1 -7)
📝 frontend/appflowy_flutter/lib/mobile/presentation/setting/personal_info/personal_info_setting_group.dart (+2 -4)
📝 frontend/appflowy_flutter/lib/mobile/presentation/setting/user_session_setting_group.dart (+9 -2)
📝 frontend/appflowy_flutter/lib/plugins/database/board/presentation/board_page.dart (+15 -8)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/editor_configuration.dart (+0 -1)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/editor_page.dart (+5 -11)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/ai/ai_writer_toolbar_item.dart (+6 -5)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/base/markdown_text_robot.dart (+37 -4)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/copy_and_paste/custom_paste_command.dart (+2 -2)

...and 60 more files

📄 Description

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.

Summary by Sourcery

Extend integration testing to support Windows and macOS platforms in addition to Linux

CI:

Update GitHub Actions workflow to add integration test jobs for Windows and macOS
Modify flutter integration test action to support multi-platform testing with platform-specific configuration

_{🔄 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/7658 **Author:** [@LucasXu0](https://github.com/LucasXu0) **Created:** 3/31/2025 **Status:** 🔄 Open **Base:** `main` ← **Head:** `enable_ci_on_windows_linux_2` --- ### 📝 Commits (10+) - [`ce91b0c`](https://github.com/AppFlowy-IO/AppFlowy/commit/ce91b0caa26001ba1a1c64ccb854e871ac6664bc) chore: enable integration test on windows and linux - [`eb9037a`](https://github.com/AppFlowy-IO/AppFlowy/commit/eb9037a1821e9e992f13b9ed198165d2ca91dd45) Merge branch 'main' into enable_ci_on_windows_linux_2 - [`f308c30`](https://github.com/AppFlowy-IO/AppFlowy/commit/f308c309aa99b3bf7e8ef973dfe7f96fe1eba79c) fix: file name too long - [`6664fc9`](https://github.com/AppFlowy-IO/AppFlowy/commit/6664fc912bcb2eb57cd32ad2c15461b649918696) Merge branch 'main' into enable_ci_on_windows_linux_2 - [`3d3b2b3`](https://github.com/AppFlowy-IO/AppFlowy/commit/3d3b2b3fdaf370234e58a6446a5d4cdaa639da40) fix: link error - [`39354ae`](https://github.com/AppFlowy-IO/AppFlowy/commit/39354ae41011ee14d77b8c06b736fe8c5256b21e) Merge branch 'main' into enable_ci_on_windows_linux_2 - [`befb8d1`](https://github.com/AppFlowy-IO/AppFlowy/commit/befb8d1b4222079e4f7064f291143da9b3c564ae) feat: add new windows tests - [`74fd436`](https://github.com/AppFlowy-IO/AppFlowy/commit/74fd4366658351043e19abea7923f56c35382a4a) fix: windows assertions - [`9f9f7bb`](https://github.com/AppFlowy-IO/AppFlowy/commit/9f9f7bbb6284dbbd5806fdca221de53c1b1803cd) fix: unable to accept the response from 'mark longer' (#7725) - [`ba57b9a`](https://github.com/AppFlowy-IO/AppFlowy/commit/ba57b9adb459e8921d79706fa1b4457df001daaf) fix: otp launch review issues (#7730) ### 📊 Changes **80 files changed** (+1513 additions, -825 deletions) <details> <summary>View changed files</summary> 📝 `.github/actions/flutter_build/action.yml` (+1 -1) 📝 `.github/actions/flutter_integration_test/action.yml` (+38 -10) 📝 `.github/workflows/flutter_ci.yaml` (+5 -3) ➕ `.github/workflows/windows_flutter_ci.yml` (+116 -0) 📝 `frontend/appflowy_flutter/integration_test/desktop/document/document_copy_and_paste_test.dart` (+3 -13) 📝 `frontend/appflowy_flutter/integration_test/desktop/document/document_create_and_delete_test.dart` (+2 -0) 📝 `frontend/appflowy_flutter/integration_test/desktop/document/document_link_preview_test.dart` (+5 -4) 📝 `frontend/appflowy_flutter/integration_test/shared/common_operations.dart` (+2 -4) 📝 `frontend/appflowy_flutter/lib/core/helpers/url_launcher.dart` (+12 -10) 📝 `frontend/appflowy_flutter/lib/mobile/presentation/base/view_page/more_bottom_sheet.dart` (+1 -1) 📝 `frontend/appflowy_flutter/lib/mobile/presentation/bottom_sheet/bottom_sheet_view_page.dart` (+3 -1) 📝 `frontend/appflowy_flutter/lib/mobile/presentation/home/mobile_home_setting_page.dart` (+1 -7) 📝 `frontend/appflowy_flutter/lib/mobile/presentation/setting/personal_info/personal_info_setting_group.dart` (+2 -4) 📝 `frontend/appflowy_flutter/lib/mobile/presentation/setting/user_session_setting_group.dart` (+9 -2) 📝 `frontend/appflowy_flutter/lib/plugins/database/board/presentation/board_page.dart` (+15 -8) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_configuration.dart` (+0 -1) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_page.dart` (+5 -11) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/ai/ai_writer_toolbar_item.dart` (+6 -5) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/base/markdown_text_robot.dart` (+37 -4) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/copy_and_paste/custom_paste_command.dart` (+2 -2) _...and 60 more files_ </details> ### 📄 Description   ### Feature Preview  ---  #### PR Checklist - [ ] 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. - [ ] I've added a test(s) to validate changes in this PR, or this PR only contains semantic changes. - [ ] All existing tests are passing. ## Summary by Sourcery Extend integration testing to support Windows and macOS platforms in addition to Linux CI: - Update GitHub Actions workflow to add integration test jobs for Windows and macOS - Modify flutter integration test action to support multi-platform testing with platform-specific configuration --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>