search

esnya / bcdice-js

[DEPRECATED] BCDice for JavaScript (Browser/Node.js)
MIT License
12 stars 3 forks source link

Fix BeginningIdol, etc #8

Closed ysakasin closed 5 years ago

ysakasin commented 5 years ago

内容

いくつかのダイスボットがテストケースを通過するようにする。 ref #7

したこと

  1. コマンドが失敗するようなテストケースではダイス残りという項目があるため、これを無視
  2. 行において、 /REGEX/ =~ varのような評価のみがされていた時、この式そのものがスキップされたいたのを修正

新たにテスト通過したもの

coveralls commented 5 years ago

Coverage Status

Coverage remained the same at 85.437% when pulling 3b92b98d746da53a492cc54fab91db4e099a8750 on ysakasin:fix_beginning_idol into efef6fcbf868a62b44ad56ff466f4064628eb70f on ukatama:master.

ysakasin commented 5 years ago

私の手元ではCIで落ちたコミットも通過していたのですが、GemfileのOpalとバージョンが違かったようです。

Opalのバージョンを上げれば、またいくつか通過するテストが増えそうです。

esnya commented 5 years ago

修正ありがとうございます！