Skip to content

[BUGFIX] Fix SQLite functional tests with Docker on WSL2#1932

Closed
oliverklee wants to merge 1 commit intomainfrom
bugfix/sqlite
Closed

[BUGFIX] Fix SQLite functional tests with Docker on WSL2#1932
oliverklee wants to merge 1 commit intomainfrom
bugfix/sqlite

Conversation

@oliverklee
Copy link
Copy Markdown
Contributor

SQLite functional tests failed with "unable to open database file" because the container user had no permission to write to the tmpfs mount directory. Adding uid/gid options to the tmpfs mount fixes this.

This is a copy of the corresponding change from the TYPO3 Core runTests.sh:

https://review.typo3.org/c/Packages/TYPO3.CMS/+/92388

Fixes #1925

@oliverklee oliverklee requested a review from a team January 23, 2026 08:41
@oliverklee oliverklee self-assigned this Jan 23, 2026
@oliverklee oliverklee added bug Something isn't working runTests.sh labels Jan 23, 2026
@oliverklee oliverklee marked this pull request as draft January 23, 2026 08:42
@oliverklee oliverklee removed the request for review from a team January 23, 2026 08:42
@coveralls
Copy link
Copy Markdown

coveralls commented Jan 23, 2026

Pull Request Test Coverage Report for Build 21289227642

Details

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 100.0%

Totals Coverage Status
Change from base Build 21288067095: 0.0%
Covered Lines: 111
Relevant Lines: 111

💛 - Coveralls

SQLite functional tests failed with "unable to open database file"
because the container user had no permission to write to the tmpfs
mount directory. Adding uid/gid options to the tmpfs mount fixes this.

This is a copy of the corresponding change from the TYPO3 Core
`runTests.sh`:

https://review.typo3.org/c/Packages/TYPO3.CMS/+/92388

Fixes #1925
@sbuerk
Copy link
Copy Markdown
Contributor

sbuerk commented Jan 24, 2026

@oliverklee This does not make sense. The core change was related to have that for the functional sqlite test and the "tmpfs" mount (ramdisk).

This does not exists for this extension anymore. It has been removed with #1394 to solve #1222.

My (unpopular) opinion on that:

  • The issue itself may make sense.
  • The solution to drop the tmpfs instead of moving it is questionable, unless the intention was to remove the ramdisk usage and leav it to disk io (performance decrease).

@cumuru
Copy link
Copy Markdown
Contributor

cumuru commented Mar 26, 2026

Closing after talking to @oliverklee

@cumuru cumuru closed this Mar 26, 2026
@cumuru cumuru deleted the bugfix/sqlite branch March 26, 2026 14:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working runTests.sh

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Fix SQLite functional tests failing on Linux/WSL2

4 participants