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

チュートリアル Javaコーディング問題(JUnit版) #41

Open
wants to merge 11 commits into
base: master
Choose a base branch
from

Conversation

Syogo-Suganoya
Copy link
Contributor

目的

Javaコーディング問題(JUnit版)のサンプルコードの作成

このサンプルを使用するnotion

https://www.notion.so/givery/Java-JUnit-8a1c1718e3404bd1b80b8f918fa532d3

@Syogo-Suganoya Syogo-Suganoya self-assigned this Jan 24, 2023
@Syogo-Suganoya Syogo-Suganoya marked this pull request as ready for review February 19, 2023 04:44
@Syogo-Suganoya
Copy link
Contributor Author

@snakazawa review me

@givery givery requested a review from snakazawa February 19, 2023 04:46
@givery givery assigned snakazawa and unassigned Syogo-Suganoya Feb 19, 2023
Copy link
Contributor Author

@Syogo-Suganoya Syogo-Suganoya left a comment

Choose a reason for hiding this comment

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

@snakazawa review me

Copy link
Contributor Author

@Syogo-Suganoya Syogo-Suganoya left a comment

Choose a reason for hiding this comment

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

@snakazawa review me

@snakazawa
Copy link
Contributor

gradle 使ったほうが良い感じになりそうなのでPR出します。

@Syogo-Suganoya
Copy link
Contributor Author

memo

非公開テストの動作確認

自動で公開→非公開の順で採番されました。

> Task :test
ok 1 [基本実装] 入力が 15 と 8 のとき、期待された出力結果が得られる
ok 2 [基本実装] 入力が 2 と 3 のとき、期待された出力結果が得られる
ok 3 [基本実装] 入力が 65 と 94 のとき、期待された出力結果が得られる
ok 4 [非公開テスト] 非公開テスト2
ok 5 [非公開テスト] 非公開テスト1
ok 6 [非公開テスト] 非公開テスト3

@Syogo-Suganoya
Copy link
Contributor Author

@Syogo-Suganoya
notionにgradleを反映する

@Syogo-Suganoya
Copy link
Contributor Author

@Syogo-Suganoya notionにgradleを反映する

しました。

@Syogo-Suganoya Syogo-Suganoya mentioned this pull request Oct 1, 2023
5 tasks
@shunjikonishi
Copy link
Contributor

shunjikonishi commented Apr 8, 2024

Marineで動作確認しました

  • solution.mdがないせいでValidateがエラーになる
  • build定義が javac src/main/java/App.java && gradle compileTest -qとなっているがjavacは不要そう
     - 実際、javacを削っても動く
  • 上に非公開テストうんぬんというコメントがあるがsecret testは定義されていない
  • notionとの関連が不明

@Syogo-Suganoya
Copy link
Contributor Author

Syogo-Suganoya commented Apr 8, 2024

javacは不要そう

javacをつけたのは、mavenで実装しようとしていた時にコンパイルエラーを即座にユーザーに伝えるためにつけていました。
その後、gradleに変更しましたが、mavenの時の設定をそのまま持ち越したのでjavacが残りました。

#41 (comment)

javac src/main/java/App.java は App.java 内でのコンパイルエラーを即座にユーザーに伝えるためです(mavenの実行は遅い)。


secret testは定義されていない

mavenからgradleに変更した際に、テストの順番がどのように影響するかを確認しました。
その結果をメモしています。


solution.mdがないせいでValidateがエラーになる

TODO: @Syogo-Suganoya notionの作り込み編から簡易編にsolution.mdの説明を移動する。

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.

4 participants