BCDice original test failures

[esnya]

テスト通らないもの一覧

[
      '_InsaneScp',
      'ArsMagica',
      'AceKillerGene',
      'BloodMoon',
      'Cthulhu7th',
      'Cthulhu7th_Korean',
      'DetatokoSaga',
      'DetatokoSaga_Korean',
      'Dracurouge',
      'Dracurouge_Korean',
      'EarthDawn3',
      'EarthDawn4',
      'EclipsePhase',
      'Elysion',
      'FilledWith',
      'GardenOrder',
      'GurpsFW',
      'IthaWenUa',
      'Kamigakari',
      'KillDeathBusiness',
      'KillDeathBusiness_Korean',
      'MagicaLogia',
      'MetalHeadExtream',
      'OneWayHeroics',
      'Oukahoushin3rd',
      'Peekaboo',
      'PlotTest',
      'Ryutama',
      'ShinMegamiTenseiKakuseihen',
      'Skynauts',
      'SwordWorld2_0',
      'SwordWorld2_5',
      'Warhammer',
      'None',
      'SevenFortressMobius',
      'NightWizard',
      'NightWizard3rd',
      'Postman',
      'StellarKnights',
]

[esnya]

CthulhuTech.txt

.stepがOpal未実装
forに書き直すとbreakで落ち

[ysakasin]

SwordWorld.txt, DoubleCross.txt

本来のテスト実行時は $SEND_STR_MAX がデフォルトよりも大きい値になるため、結果は省略されないが、bcdice-jsでのテスト実行時にはデフォルト値のため結果が一部省略されてしまう。その省略が原因でテストが落ちている。

ライブラリを使う側としては、省略される利点はあまりないと思っているので、 $SEND_STR_MAX を大きい値決めうちにしても良いのではと感じる。

SwordWorld 該当箇所

https://github.com/ukatama/BCDice/blob/192f7bcf7b85d903c7bb9eb5374a792922c3a85d/src/diceBot/SwordWorld.rb#L458

デフォルト値

https://github.com/ukatama/BCDice/blob/192f7bcf7b85d903c7bb9eb5374a792922c3a85d/src/configBcDice.rb#L11

テスト実行時

https://github.com/ukatama/BCDice/blob/192f7bcf7b85d903c7bb9eb5374a792922c3a85d/src/cgiDiceBot.rb#L19

[ysakasin]

analyzeDiceCommandResultMethodの問題

https://github.com/ukatama/BCDice/blob/192f7bcf7b85d903c7bb9eb5374a792922c3a85d/src/diceBot/DiceBot.rb#L442

Opalのバグ。

analyzeDiceCommandResultMethod では public_methods(false) を呼び出しているが、これがMRIとOpalでは挙動が異なっている。Opalにおいて public_methods は単に methods のaliasになっている。

opal/opal#909

opal/opal#1848 にて修正のPRを投げてマージされたが、この差分はリリースされてない。

対象範囲

• ColossalHunter
• Dracurouge
• LogHorizon
• LogHorizon_Korean
• OneWayHeroics

[esnya]

$SEND_STR_MAXの問題を修正しました。情報ありがとうございます！ @ysakasin