Skip to content

Conversation

@FineArchs
Copy link
Member

test/index.tsから文法要素、予約語、リテラルのテストを分離します。

@syuilo
Copy link
Collaborator

syuilo commented May 21, 2024

std関連のテストも分離できそう

@syuilo
Copy link
Collaborator

syuilo commented May 21, 2024

削除されたテストある?

@FineArchs
Copy link
Member Author

FineArchs commented May 21, 2024

std関連のテストも分離できそう

stdと組み込みプロパティのテストは今あるPRとのコンフリクトが面倒だな…と思って後回しにしてしまいましたが、希望があればこれも分離しようと思います。

削除されたテストある?

予約語のバリデーションのテストを書き換えましたが、以前にあった機能は満たせるようにしているはずです。
あとはテスト用関数exeを拡張した程度で、ほとんどコピペです。

@syuilo
Copy link
Collaborator

syuilo commented May 25, 2024

一部消えてるテストがあるのが気になるわね

@FineArchs
Copy link
Member Author

@syuilo 移動したものではなくですか?でしたら多分ミスなのでどれのことか教えて下さい

@FineArchs
Copy link
Member Author

FineArchs commented May 26, 2024

@syuilo
もしテストの増減をテスト数で判断されたのでしたら、その数が減った原因は先にも言った予約語バリデーションのテストの書き換えと思われます。(test/index.tsのVar name starts with reserved wordname validation of reserved word → test/keywords.ts)

この書き換えではもともと20個ほどあった似たようなテストをjestのeach機能を使ってまとめていますが、eachを二重に使用しているため組み合わせ爆発を警戒し、毎回全通りテストするのではなく乱数生成した10パターンを行うようにしています。

@syuilo
Copy link
Collaborator

syuilo commented May 27, 2024

ぱっと見て分かるのはSKIコンビネータのやつとか(まあこれは無くても良いけど)

@syuilo syuilo merged commit f5779a5 into aiscript-dev:aiscript-next May 27, 2024
@syuilo
Copy link
Collaborator

syuilo commented May 27, 2024

👍🏻 👍🏻

@FineArchs
Copy link
Member Author

ぱっと見て分かるのはSKIコンビネータのやつとか

https://github.com/FineArchs/aiscript/blob/ecd76255403a35023a5ed910305588ed2d7381fc/test/index.ts#L2241-L2273

これのことでしたらtest/index.tsの一番下に残ってますね

@FineArchs FineArchs deleted the test-syn branch May 27, 2024 03:29
@syuilo
Copy link
Collaborator

syuilo commented May 27, 2024

あーGitHubのDiffでは畳まれてて見えなかった
失礼いたしました

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