[StellarKnights] success/failure を明示

[ViVi-shark]

#423

---

対象

　成功／失敗の概念があるのは、【アタック判定】に防御力との比較が指定された場合のみであるはず。

方針

　当該の【アタック判定】において、成功数が１以上ならば成功、成功数が０ならば失敗とみなす。

　基本ルールブック『銀剣のステラナイツ』の時点では「失敗」という表現はもちいられていないが（同書p150）、『銀剣のステラナイツ　星屑のリヴラガーデン』p119では同じ意味合いの処理に対して「失敗」という表現がされているため、この方針で妥当だと考える。

備考

　成功数が 0 になるテストケースがもともとなかったので、それも追加してある。

[codecov]

Codecov Report

Merging #425 (5887761) into master (2df3312) will increase coverage by 0.03%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #425      +/-   ##
==========================================
+ Coverage   95.14%   95.18%   +0.03%     
==========================================
  Files         286      287       +1     
  Lines       18471    18594     +123     
==========================================
+ Hits        17575    17699     +124     
+ Misses        896      895       -1     

Impacted Files	Coverage Δ
lib/bcdice/game_system/StellarKnights.rb	98.73% <100.00%> (+0.10%)	⬆️
lib/bcdice/game_system.rb	100.00% <0.00%> (ø)
...cdice/game_system/sword_world/transcendent_test.rb	100.00% <0.00%> (ø)
lib/bcdice/game_system/SkynautsBouken.rb	100.00% <0.00%> (ø)
lib/bcdice/game_system/SwordWorld.rb	97.35% <0.00%> (+0.09%)	⬆️
lib/bcdice/game_system/MonotoneMuseum.rb	98.30% <0.00%> (+1.75%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2df3312...5887761. Read the comment docs.

[ysakasin]

ありがとうございます！ テストデータの部分にコメントしましたので修正お願いします。

[ViVi-shark]

@ysakasin

　ご確認ありがとうございます。そのまま採用しました。

[ysakasin]

ありがとうございます！