kusumotolab / kGenProg

A High-performance, High-extensibility and High-portability APR System
MIT License
48 stars 13 forks source link

CTZ03がMainTestで修正できない #744

Closed YoshikiHigo closed 4 years ago

YoshikiHigo commented 4 years ago

RouletteStatementSelectionだと修正できる. RouletteStatementSelectionは複文も挿入候補にするが,RouletteStatementAndConditionSelectionは複文が挿入候補ではないことによる影響.

YoshikiHigo commented 4 years ago

個人的には複文は挿入候補にしなくてよいと考えている.

shinsuke-mat commented 4 years ago

[mtg結論] しばらく結合テストのみCTZ03を @Ignore しておく. その内,@Ignore テストの是非について議論する.

shinsuke-mat commented 4 years ago

ふと思いついた.

当該テストは結合テストなので,いわゆるsmoke testにしておくのが良い. とりあえず例外が出ずに最後まで通過すればOK.修正できなくてもOK.という立ち位置.

「バグが修正できるはず」という期待は機能テストという観点では強すぎる制約. 「理論上修正できる」という前提ならともかく,探索的手法ではやりすぎ.

YoshikiHigo commented 4 years ago

smoke testにするのはCTZ03だけではなく,kGenProgMainTestにある結合テスト全部ってことですよね?

shinsuke-mat commented 4 years ago

どっちでも良いとは思います. 理屈の観点:全部smokeが合理的. 実践の観点:一つくらい修正成功を期待するテストがあった方がregression testとして役立ちそう.

YoshikiHigo commented 4 years ago

これは閉じてよいですね.

772 で解決済み.