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

chor: Added test for attribute syntax #72

Merged
merged 2 commits into from
Mar 8, 2021
Merged

chor: Added test for attribute syntax #72

merged 2 commits into from
Mar 8, 2021

Conversation

akabekobeko
Copy link
Member

And added test: debug to package.json to run a single test file. Change the xxx.test.ts part to the one you want to test.

And added `test: debug` to package.json to run a single test file. Change the `xxx.test.ts` part to the one you want to test.
@akabekobeko akabekobeko self-assigned this Mar 8, 2021
@akabekobeko
Copy link
Member Author

#46 対応する予定だったが remark-attr 修正 を待つことになるので、

  • 属性構文のテストを追加
    • index.test.ts ではなく構文単位でテストされるほうがよいのでそうした
    • index.test.ts 側のテストはあわせて削除
  • VFM v2.0 で自前実装することになってもテストは有効

という観点からテスト追加を PR することにした。

ついでに構文 (ファイル) 単位のテストをするため package.jsontest:debug を追加。ファイル名部分をその時点でデバッグしたいものに変更することで局所的なテストをおこなえる。コマンド側でログ出力も許可してある。ある構文の問題を調べる場合はこれを npm run test:debug として使い、テストが通ったら npm test で連携も確認するというのを想定。

@akabekobeko akabekobeko merged commit 4e844f8 into master Mar 8, 2021
@akabekobeko akabekobeko deleted the add-attr-test branch March 8, 2021 05:58
@akabekobeko akabekobeko added this to the v1.0.0 milestone Apr 9, 2021
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.

1 participant