最近<a href="https://sinonjs.org/">Sinon.JS</a>のバージョンをv14.0.0に上げたときに、エラーが出てテスト実行が失敗するようになりました。
この記事では対処方法を書いていきます。
<h2>事象</h2>
たとえば次のようなテストがあったとします。
<code>sinon.useFakeTimers</code> は<a href="https://sinonjs.org/releases/latest/fake-timers/">ドキュメント</a>にある通り、<code>setTimeout</code> や <code>clearTimeout</code>などを置き換える関数です。
<pre class="language-javascript"><code class="language-javascript">import test from 'ava';
import sinon from 'sinon';

test('test', (t) => {
 const fakeTimer = sinon.useFakeTimers();
 t.pass();
 fakeTimer.restore();
});

test('test2', (t) => {
 const fakeTimer = sinon.useFakeTimers();
 t.pass();
 fakeTimer.restore();
});
</code></pre>
2回目の <code>sinon.useFakeTimers()</code> を実行するとき、先ほどのテストファイルで言うと <code>test2</code> を実行するときに次のようなメッセージが表示されます。
<pre><code>TypeError {
 message: 'Can\'t install fake timers twice on the same global object.',
}
</code></pre>
<h2>原因</h2>
書き換えられたグローバルオブジェクトを戻すために <code>sinon.useFakeTimers</code> と <code>fakeTimer.restore()</code> を対になる形で実行しないといけません。
うっかり <code>fakeTimer.restore()</code> を実行しないまま <code>sinon.useFakeTimers</code> を実行すると、元の日時に復元することが難しくなります。
<pre class="language-javascript"><code class="language-javascript">// サンプル
const sinon = require("sinon@12.0.0")

console.log("Original time: " + new Date().getTime()); // "Original time: 1653007080412"
let fakeTimer = sinon.useFakeTimers(Date.parse("2014-06-05T12:07:07.662Z"));
fakeTimer = sinon.useFakeTimers(Date.parse("2018-04-11T14:08:00Z"));
fakeTimer.restore();
console.log("Restored time: " + new Date().getTime()); // "Restored time: 1401970027662"
</code></pre>
今回のサンプルコードの場合はまだ復元できると思いますが、これがより回数を重ねて <code>sinon.useFakeTimers</code> を実行してしまうとより復元が難しくなります。
この問題が、<a href="https://github.com/sinonjs/sinon/issues/2449">Impossible to restore fake timers in certain situations. · Issue #2449 · sinonjs/sinon</a>で報告されて、対応として <code>@sinonjs/fake-timers</code> 側で<a href="https://github.com/sinonjs/fake-timers/pull/426">Prohibit faking of faked timers by cjbarth · Pull Request #426 · sinonjs/fake-timers</a>というPull Requestがマージされました。
Pull RequestのSolutionに「If an attempt is make to fake a timer that is already faked, an exception will be thrown.」と書いてある通り、timerがすでにfakeだった場合に再度 <code>sinon.useFakeTimers</code> を実行した場合に例外が投げられるという変更がされました。
この変更により、複数のテストで <code>sinon.useFakeTimers</code> と <code>fakeTimer.restore</code> を実行していた場合に、AVA上でテストが並列で実行されることもあって <code>fakeTimer.restore</code> が実行される前に <code>sinon.useFakeTimers</code> が実行される場合が出てきました。
その結果として「Can't install fake timers twice on the same global object.」というエラーが出力されるようになりました。
<h2>解決策</h2>
解決方法は2つあります。
まず1つは、テストコード側で並列実行をやめて直列実行にすることです。具体的には次の通り書くとテストが成功します。
<pre class="language-javascript"><code class="language-javascript">import test from 'ava';
import sinon from 'sinon';

test.serial('test', (t) => {
 const fakeTimer = sinon.useFakeTimers();
 t.pass();
 fakeTimer.restore();
});

test.serial('test2', (t) => {
 const fakeTimer = sinon.useFakeTimers();
 t.pass();
 fakeTimer.restore();
});
</code></pre>
または <code>test.before</code> や <code>test.after</code> といったテストファイル内の最初と最後のテスト前後で実行されるフックを使って、fakeTimerを使うのも良いです。
<pre class="language-javascript"><code class="language-javascript">import test from 'ava';
import sinon from 'sinon';

let fakeTimer = null;

test.before(() => {
 fakeTimer = sinon.useFakeTimers();
});

test.after(() => {
	if (!fakeTimer) {
		return;
	}
 fakeTimer.restore();
});

test('test', (t) => {
 t.pass();
});

test('test2', (t) => {
 t.pass();
});
</code></pre>
AVAのように並列実行がデフォルトのテストフレームワークだと同じ問題が起きそうですが、他のJestやVitestなどはどうしているのか気になります。

学ぶ、考える、書き出す。

タグ「テスト」の記事一覧