[PR #7901] [MERGED] test: simple ai chat tests #8196

New issue

Closed

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

mirror commented

2026-03-23 23:23:03 +00:00

Owner

📋 Pull Request Information

Original PR: https://github.com/AppFlowy-IO/AppFlowy/pull/7901
Author: @LucasXu0
Created: 5/9/2025
Status: ✅ Merged
Merged: 5/9/2025
Merged by: @LucasXu0

Base: main ← Head: simple_chat_test

📝 Commits (5)

54d0fc7 fix: repeated setState in animation list
955358a test: add simple chat integration test
527cfa2 chore: update chat list
0f3b249 test: send messages without default messages
9d3e9fb fix: refresh space after moving page

📊 Changes

12 files changed (+360 additions, -31 deletions)

View changed files

➕ frontend/appflowy_flutter/integration_test/desktop/chat/chat_page_test.dart (+264 -0)
📝 frontend/appflowy_flutter/integration_test/desktop_runner_9.dart (+2 -0)
📝 frontend/appflowy_flutter/integration_test/shared/ai_test_op.dart (+28 -0)
📝 frontend/appflowy_flutter/integration_test/shared/common_operations.dart (+2 -0)
📝 frontend/appflowy_flutter/lib/plugins/ai_chat/application/chat_bloc.dart (+6 -0)
📝 frontend/appflowy_flutter/lib/plugins/ai_chat/presentation/animated_chat_list.dart (+38 -26)
📝 frontend/appflowy_flutter/lib/plugins/ai_chat/presentation/chat_page/chat_animation_list_widget.dart (+10 -1)
📝 frontend/appflowy_flutter/lib/plugins/ai_chat/presentation/chat_page/load_chat_message_status_ready.dart (+3 -0)
📝 frontend/appflowy_flutter/lib/workspace/presentation/home/desktop_home_screen.dart (+1 -1)
📝 frontend/appflowy_flutter/lib/workspace/presentation/home/hotkeys.dart (+1 -1)
📝 frontend/appflowy_flutter/lib/workspace/presentation/home/menu/sidebar/space/sidebar_space.dart (+2 -2)
📝 frontend/appflowy_flutter/lib/workspace/presentation/home/menu/view/view_item.dart (+3 -0)

📄 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

Improve the chat page functionality and add an integration test for chat messaging

New Features:

Add integration test for chat messaging functionality

Enhancements:

Refactor chat animation list to improve message handling
Add state management improvements for chat messages

Tests:

Create a new integration test for sending and receiving messages in the chat page

Chores:

Add logging for chat message operations

_{🔄 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/7901 **Author:** [@LucasXu0](https://github.com/LucasXu0) **Created:** 5/9/2025 **Status:** ✅ Merged **Merged:** 5/9/2025 **Merged by:** [@LucasXu0](https://github.com/LucasXu0) **Base:** `main` ← **Head:** `simple_chat_test` --- ### 📝 Commits (5) - [`54d0fc7`](https://github.com/AppFlowy-IO/AppFlowy/commit/54d0fc7b04d246857cf90d343a5623727730e641) fix: repeated setState in animation list - [`955358a`](https://github.com/AppFlowy-IO/AppFlowy/commit/955358a7ac05f7a1437a52cfe19fdcd0432219c8) test: add simple chat integration test - [`527cfa2`](https://github.com/AppFlowy-IO/AppFlowy/commit/527cfa2c5f220a10a9829a8ce22fdd0c06d3eb65) chore: update chat list - [`0f3b249`](https://github.com/AppFlowy-IO/AppFlowy/commit/0f3b24946356e678ad65145efe88ac12645b4f7f) test: send messages without default messages - [`9d3e9fb`](https://github.com/AppFlowy-IO/AppFlowy/commit/9d3e9fbe5e9a6ad7b68e857c1e2f9698b66ea2f4) fix: refresh space after moving page ### 📊 Changes **12 files changed** (+360 additions, -31 deletions) <details> <summary>View changed files</summary> ➕ `frontend/appflowy_flutter/integration_test/desktop/chat/chat_page_test.dart` (+264 -0) 📝 `frontend/appflowy_flutter/integration_test/desktop_runner_9.dart` (+2 -0) 📝 `frontend/appflowy_flutter/integration_test/shared/ai_test_op.dart` (+28 -0) 📝 `frontend/appflowy_flutter/integration_test/shared/common_operations.dart` (+2 -0) 📝 `frontend/appflowy_flutter/lib/plugins/ai_chat/application/chat_bloc.dart` (+6 -0) 📝 `frontend/appflowy_flutter/lib/plugins/ai_chat/presentation/animated_chat_list.dart` (+38 -26) 📝 `frontend/appflowy_flutter/lib/plugins/ai_chat/presentation/chat_page/chat_animation_list_widget.dart` (+10 -1) 📝 `frontend/appflowy_flutter/lib/plugins/ai_chat/presentation/chat_page/load_chat_message_status_ready.dart` (+3 -0) 📝 `frontend/appflowy_flutter/lib/workspace/presentation/home/desktop_home_screen.dart` (+1 -1) 📝 `frontend/appflowy_flutter/lib/workspace/presentation/home/hotkeys.dart` (+1 -1) 📝 `frontend/appflowy_flutter/lib/workspace/presentation/home/menu/sidebar/space/sidebar_space.dart` (+2 -2) 📝 `frontend/appflowy_flutter/lib/workspace/presentation/home/menu/view/view_item.dart` (+3 -0) </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 Improve the chat page functionality and add an integration test for chat messaging New Features: - Add integration test for chat messaging functionality Enhancements: - Refactor chat animation list to improve message handling - Add state management improvements for chat messages Tests: - Create a new integration test for sending and receiving messages in the chat page Chores: - Add logging for chat message operations --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>