Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

入力コードがどのテストにどう落ちるか出力する処理を追加 #746

Merged
merged 9 commits into from Jun 17, 2020

Conversation

hnymA
Copy link
Contributor

@hnymA hnymA commented Jun 10, 2020

resolve #741

@RyokoIzuta
Copy link
Contributor

QuickSortだけexpectedもactualも null になっているのが気になりますが,これはテストに assertArrayEqualsを使用している場合は回避不可能なんですかね?

2020-06-16 15:35:47 [Time-limited test] [INFO]  KGenProgMain - initial failed tests (4/7)
example.QuickSortTest.test04: expected null, actual null
example.QuickSortTest.test02: expected null, actual null
example.QuickSortTest.test01: expected null, actual null
example.QuickSortTest.test07: expected null, actual null

それ以外はいいと思います.お疲れ様です.

@hnymA
Copy link
Contributor Author

hnymA commented Jun 17, 2020

@RyokoIzuta
pic

failedResonが ".*expected:\\<(.*)\\>.*but was:\\<(.*)\\>.*" の形をしていないときにパースに失敗してnullを吐くので
その際はfailedResonをそのまま出力するように改良しました

その点だけ再度見てもらってもいいですか?

@shinsuke-mat
Copy link
Member

slackでも伝えたけど,Failure.getException().getMessage() の中身を手前でパースする必要はなさそう.assertionフレームワークは様々なので全対応のパースは不可能で,なおかつ現状パース処理が不毛なので.

@hnymA
Copy link
Contributor Author

hnymA commented Jun 17, 2020

↑のコメントを受けてfailedReasonを出力するだけのシンプルな処理にしました
また,failedReason をprivate fieldにしてgetterを足しました

@shinsuke-mat shinsuke-mat merged commit 312f340 into master Jun 17, 2020
@shinsuke-mat shinsuke-mat deleted the initial-test-failed-info branch June 17, 2020 02:45
@shinsuke-mat
Copy link
Member

よさそう.お疲れさん.マージします.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

入力コードがどのテストに失敗しているかを出力する.
3 participants