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

plugin_testの「なる」が一般的な動詞すぎる #568

Closed
kujirahand opened this issue Aug 16, 2020 · 6 comments
Closed

plugin_testの「なる」が一般的な動詞すぎる #568

kujirahand opened this issue Aug 16, 2020 · 6 comments

Comments

@kujirahand
Copy link
Owner

kujirahand commented Aug 16, 2020

plugin_testに「なる」という動詞がありますが、一般的な動詞すぎるので、別名にできたらと思います。
「ASSERT等」が、タイプしにくいのがそもそもの問題でしょうか。

以下の命令はどうでしょうか?

  • AとBでテスト実行。🌟
  • AとBをテスト確認。
  • AがBとテスト等しい。
  • AとBをイコールテスト。
  • AとBがテスト正しい
  • AとBがテスト異なる

「AとBでテスト実行」が一番しっくり来るでしょうか。

@massongit
Copy link
Collaborator

AとBでテスト実行 だと等しいというニュアンスが伝わらないので、 AがBとテスト等しい の方が良いかもしれないです。

@weyk
Copy link
Contributor

weyk commented Sep 12, 2020

条件(等しいとか異なるとか)は、命令につなげず、引数にする方が自然になるとも思います。
Objectが無く、また、引数の型による多態が無いためか、
命令語がかなり日本語として怪しくなってます。
AとBを「等値」でテストする
とか。enumのように、
AとBを等値でテストする
のように書けるとよいのですが、うまい方法が思いつきません。
(なでしこ1の変数宣言を必須にせず、同値の文字列として扱われる仕様も混乱の元なきがしますし)

@kujirahand
Copy link
Owner Author

kujirahand commented Sep 14, 2020

ちなみに、思い出してv1を見るとテスト実行を採用していましたね。

v1のtestフォルダを見ると、かなりたくさんのテスト実行の例があります。
英語でもよくあるassertコマンドなども、等しいというニュアンスはそこにないので、その辺りは、こだわらなくても良いかと思います。テスト書くときは、等しいか等しくないかのチェックが大半と思いますので。

日本語の自然さで選ぶのが、ユーザーから見て使いやすい命令となるでしょうか。

@weyk
Copy link
Contributor

weyk commented Sep 14, 2020

mochaとassertでやってますが、chaiとかのテスト見るとassertよりも解り易いなぁと感じています。
決め打ちのルールを覚えてしまえば、書く側は、assertのが楽かもしれません。
(いまだに、「assertって、真と偽とどっちで止まるんだっけ」という覚えぐわいですが)
とりあえず、命令いちいち調べずとも、ソースをみて何やっているか読み取れるというのが、
人のソースを読むにしろ、過去の自分のソースをいじるにしろ、誰かのソースをもとに
改造したものを作るにしろ、使いやすいと思っています。初見で知らない単語(命令)で
動きの予測ができないものが並ぶと、実際以上に複雑に見えるというのもあると思っています。

が、単に私の考えなので、忘れてください。v1にあったのなら、
v1と同じに、なでしこ流の命令語が良いのではないでしょうか。

@kujirahand
Copy link
Owner Author

kujirahand commented Oct 1, 2020

とりあえず、v1互換で、「テスト実行」を導入してみます。
これに加えて「AがBとテスト等しい」もエイリアスで追加することで、より分かりやすくなるでしょうか。

@kujirahand
Copy link
Owner Author

コミットするとき、入れ忘れました。

72a0208

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

No branches or pull requests

3 participants