[schlessera]

Follow-up to #408 and #413.

wp db export and wp db import (and a few other db-command scenarios) call the sqlite3 command-line tool for SQLite databases. The GitHub-hosted runners have it, the Docker image did not, so every SQLite job of db-command failed when running in Docker containers, with errors like:

Error: The sqlite3 binary could not be found. Please install sqlite3 to use the export command.

This installs the sqlite3 package. Images of PHP 8.0 and older already get a matching sqlite3 from the SQLite built in #413, which comes first in the PATH.

Testing

• Built the images of PHP 8.3 (Debian sqlite3 3.46.1) and 7.4 (/usr/local/bin/sqlite3 3.54.1).
• db-command with wp-cli-tests v5.4.0 and this Dockerfile, SQLite, 4 workers in Docker: 62 scenarios passed in 71 s.

Summary by CodeRabbit

• Chores
  • Added SQLite command-line tools to the Docker image.

[coderabbitai]

No actionable comments were generated in the recent review.

Recent review info

Run configuration

• Source mode: Auto
• Repository: Unavailable
• Branch: Unavailable
• File: No configuration file from this repository applied
• Commit: Unavailable
• Layers: defaults
• Review profile: CHILL
• Plan: Advanced
• Run ID: 2ab2233e-0842-4b71-a757-2823da988181

Commits

Reviewing files that changed from the base of the PR and between 36e43f3 and 20e5e8d.

Files selected for processing: 1

• utils/docker/Dockerfile

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.

Walkthrough

The Docker image’s apt package list now installs sqlite3.

Changes: Docker image package

Layer / File(s)	Summary
Install SQLite CLI utils/docker/Dockerfile	The apt package list now includes sqlite3.

Estimated code review effort: 1 (Trivial) | ~3 minutes

Change: Bug fix

Possibly related issues

• Run Behat in throwaway containers when Docker is available #391 — The issue concerns declared test requirements in containers; this change installs the SQLite executable.
• Declare and check the external tools that scenarios need #393 — The issue concerns missing external tools; this change adds sqlite3 to the Docker image.
• Report scenarios that are filtered out by unmet requirements #392 — The issue concerns unmet CI and container requirements; this change supplies the SQLite executable.
• Epic: Explicit test requirements #395 — The issue concerns missing external tools for tests; this change installs sqlite3.

Possibly related PRs

• wp-cli/wp-cli-tests#413 — This PR also changes SQLite support in the Dockerfile, including a conditional SQLite build.

Suggested labels: bug, scope:testing, scope:artifacts

Suggested reviewers: brianhenryie

Merge Risk: Minimal · up to 20e5e

No actionable merge-blocking risk was identified in this Docker package change.

Pre-merge checks: 5

Check name	Status	Explanation
Description Check		Check skipped - CodeRabbit’s high-level summary is enabled.
Title check		The title clearly and concisely describes the main change: installing the SQLite command-line tool in Docker workers.
Docstring Coverage		No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentat…
Linked Issues check		Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check		Check skipped because no linked issues were found for this pull request.

Full details: Docstring Coverage

Explanation

No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)

Finishing Touches

Generate unit tests (beta)

• Commit to this branch
• Create a new PR

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

Share

• X
• Mastodon
• Reddit
• LinkedIn

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!