Reset and restore the test database without starting client processes

オープン
#386 コメント 1 件 リアクション 0 件 担当者 0 名 GitHub で見る

メンテナーはふだん 1 日以内に返信

まだ誰も着手していません。

評価

難易度
3/5
見積もり時間
1〜2日
初心者へのやさしさ
55/100
issue の種類
リファクタリング
明瞭さ
おおむね明確
活発さ
活発
技術スタック
mysql, php
領域
database, testing

調査の方向性

Inspect the framework's database setup code that invokes the mysql client per scenario, likely in the Behat test bootstrap. Read #384 to capture baseline CI timings before changing anything. Done means a single persistent connection handles drop/create/restore per suite and the drafted PR shows reduced times recorded in this issue.

索引モデルが issue の本文から書いたものです。

説明

scope:testing

The framework runs its own database bookkeeping through a new mysql client process each time. Per scenario that means a connection test and a DROP DATABASE before it starts. Scenarios that install WordPress add a CREATE DATABASE and an import of the cached install dump. Each client process costs about 25 ms before it does any work, and the import costs about 260 ms.

Steps:

  1. Test the database connection once per suite instead of once per scenario.
  2. Run the drop and create statements over a single persistent connection from the Behat process.
  3. Restore the cached install over that same connection.

Locally this brought a trivial query from 28 ms to 0.2 ms, and the import from 258 to 191 ms on a regular database (79 to 15 ms on an in-memory one). The database ends up in the same state as before.

Measuring: take the baseline from #384, then open a draft PR, run the same CI jobs against it and record both results here before we decide.

主要言語
PHP
スター
42
フォーク
29
平均マージ
21時間 6分
マージ済み PR(30日)
24

環境構築

はじめの一歩

  1. issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
  2. 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
  3. リポジトリをフォークし、ブランチを切って変更します。
  4. issue 番号を参照したプルリクエストを送ります。

wp-cli/wp-cli-tests のほかの issue

wp-cli/wp-cli-tests の issue をすべて見る

似ている issue

PHP の issue をもっと見る

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。