nodeSolidServer / nodeSolidServer/node-solid-server

Assertions in async callbacks will not mark tests as failed

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

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

主要言語
JavaScript
スター
1.8k
フォーク
308
PR マージ指標
30日以内にマージされた PR はありません

説明

Looking through the tests for NSS I see some that don't appear to actually achieve anything (unless I'm missing something).

Some of them have been marked to skip, which is fair enough.

For example:

https://github.com/solid/node-solid-server/blob/master/test/integration/ldp-test.js#L178
https://github.com/solid/node-solid-server/blob/master/test/integration/ldp-test.js#L199
https://github.com/solid/node-solid-server/blob/master/test/integration/ldp-test.js#L214
https://github.com/solid/node-solid-server/blob/master/test/integration/ldp-test.js#L326 (This one isn't skipped)

If they're in tests like that, there could be a possibility that it's leaked into NSS itself as well.

コントリビューションガイド

コントリビューションガイドを開く

はじめの一歩

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

調査の方向性

まず、test/integration/ldp-test.js の 178、199、214、326 行目にある非同期テストケースを読みます。それらのコールバック内のアサーションがどのように処理されているかを確認し、続いて同じパターンについて関連する NSS のテストコードを調べます。影響を受けるテストが何も報告せずに通過するのではなく、アサーションの失敗を確実に報告できれば完了です。

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

評価

技術スタック
javascript
領域
testing-qa
issue の種類
バグ
難易度
3/5
見積もり時間
1〜2日
活発さ
停滞
明瞭さ
おおむね明確
初心者へのやさしさ
35/100

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

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