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

538: Add dispatch test. #549

Merged
merged 6 commits into from
Oct 17, 2019

Conversation

fukatani
Copy link
Contributor

No description provided.

@fukatani fukatani changed the title #438: Add dispatch test. #538: Add dispatch test. Oct 14, 2019
@fukatani
Copy link
Contributor Author

おっしゃるとおりで、submission/problemや他のサービスにも拡大するつもりですが、一旦この状態でいい感じか見せようとしてました。
AtCoderだけ入れて、Hacktoberfest向けに残りの難易度低めのイシューとして空けてもいいのかなあと思っています。

サービスのカバレッジも上がるので、数値だけで言えば全サービスのテストをした方がいいです。
若干サービスのテストなのかdispatchのテストなのかわからなくなりますが。

@kmyk
Copy link
Member

kmyk commented Oct 14, 2019

そういうことならよさそうです。
しかし 1 種類だけなのと 2 種類以上なのは差があるので、やはりなにかもうひとつは追加しておいてほしいです。

@fukatani fukatani changed the title #538: Add dispatch test. [WIP] #538: Add dispatch test. Oct 14, 2019
@@ -1256,3 +1256,4 @@ def _from_table_row(cls, tr: bs4.Tag) -> 'AtCoderSubmissionTestCaseResult':
onlinejudge.dispatch.services += [AtCoderService]
onlinejudge.dispatch.problems += [AtCoderProblem]
onlinejudge.dispatch.submissions += [AtCoderSubmission]
onlinejudge.dispatch.contests += [AtCoderContest]
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

AtCoderContestがdispatchできてなかったことに気づきました。

@fukatani
Copy link
Contributor Author

fukatani commented Oct 16, 2019

dispatchはうまく使うと、unittestの粒度上げるのに使えそうですね。内部の可視性が上がったと言いますか。
また、自作IDEにも使いたい機能が見つかりました。

まだやれることはありますが、バグ修正も含んでしまいましたし、リリースが近い気もするので、ひとまずこの辺でレビューを依頼します。

→7.3.0のリリースの後にマージしましょうか。

@fukatani fukatani changed the title [WIP] #538: Add dispatch test. 538: Add dispatch test. Oct 16, 2019
Copy link
Member

@kmyk kmyk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@kmyk
Copy link
Member

kmyk commented Oct 16, 2019

(7.3.0 はバグ修正がメインですし、これもついでに入れてしまいたい気がする? どちらでもよいですが)

@fukatani
Copy link
Contributor Author

ですよね!gitter見てぶれましたが、初志貫徹で行きます。

@fukatani fukatani merged commit 7087b92 into online-judge-tools:master Oct 17, 2019
@fukatani fukatani deleted the issue/438/dispatch branch October 17, 2019 12:49
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.

2 participants