[GH-ISSUE #2901] [Bug] Integration tests are broken on Windows #1272

New issue

Closed

opened 2026-03-23 20:47:13 +00:00 by mirror · 2 comments

mirror commented

2026-03-23 20:47:13 +00:00

Owner

Originally created by @a-wallen on GitHub (Jun 25, 2023).
Original GitHub issue: https://github.com/AppFlowy-IO/AppFlowy/issues/2901

Bug Description

Some integration tests on Windows have been broken since I started work here, but I needed help figuring out why. While I was working on adding custom fonts to AppFlowy, I realized that there was a problem with the following that I think causes the issue

  static Future<Directory> testLocation(String? name) async {
    final dir = await getApplicationDocumentsDirectory();
    var path = '${dir.path}/flowy_test';
    if (name != null) {
      path += '/$name';
    }
    return Directory(path).create(recursive: true);
  }

How to Reproduce

Any integration test that depends on TestFolder seems to be affected.

Expected Behavior

Tests pass on macOS and linux because the platform file separator is / and not \

Operating System

Windows

AppFlowy Version(s)

main

Screenshots

No response

Additional Context

No response

Originally created by @a-wallen on GitHub (Jun 25, 2023). Original GitHub issue: https://github.com/AppFlowy-IO/AppFlowy/issues/2901 ### Bug Description Some integration tests on Windows have been broken since I started work here, but I needed help figuring out why. While I was working on adding custom fonts to AppFlowy, I realized that there was a problem with the following that I think causes the issue ```dart static Future<Directory> testLocation(String? name) async { final dir = await getApplicationDocumentsDirectory(); var path = '${dir.path}/flowy_test'; if (name != null) { path += '/$name'; } return Directory(path).create(recursive: true); } ``` ### How to Reproduce Any integration test that depends on `TestFolder` seems to be affected. ### Expected Behavior Tests pass on macOS and linux because the platform file separator is `/` and not `\` ### Operating System Windows ### AppFlowy Version(s) main ### Screenshots _No response_ ### Additional Context _No response_

mirror closed this issue

2026-03-23 20:47:13 +00:00

mirror commented

2026-03-23 20:47:14 +00:00

Author

Owner

@a-wallen commented on GitHub (Jun 25, 2023):

I just checked whether the platform path separator was the issue, and it turns out that the tests fail because the process cannot delete the test location because the application is still running during the teardown phase and has the test location open.

@a-wallen commented on GitHub (Jun 25, 2023): I just checked whether the platform path separator was the issue, and it turns out that the tests fail because the process cannot delete the test location because the application is still running during the teardown phase and has the test location open.

mirror commented

2026-03-23 20:47:14 +00:00

Author

Owner

@a-wallen commented on GitHub (Jun 25, 2023):

AppFlowy.exe can't be terminated to release the file handles because the way that some tests are written require the application to be open.

@a-wallen commented on GitHub (Jun 25, 2023): AppFlowy.exe can't be terminated to release the file handles because the way that some tests are written require the application to be open.

mirror referenced this issue

2026-03-23 21:38:41 +00:00

[PR #1272] [MERGED] chore: update locale #4638