Add unit test for integrity - Githubissues

next-theme / hexo-theme-next

🎉 Elegant and powerful theme for Hexo.

https://theme-next.js.org

Other

2.31k stars 411 forks source link

Add unit test for integrity #729

Closed njzjz closed 4 months ago

njzjz commented 5 months ago

PR Checklist

[x] The commit message follows guidelines for NexT.
[x] The changes have been tested (for bug fixes / features).
[ ] Docs in NexT website have been added / updated (for features).

PR Type

[ ] Bugfix.
[ ] Feature.
[ ] Improvement.
[ ] Code style update (formatting, linting).
[ ] Refactoring (no functional changes).
[ ] Documentation.
[ ] Translation.
[x] Other... Please describe: unit test

What is the current behavior?

The test is expected to fail due to https://github.com/next-theme/hexo-theme-next/issues/728.

What is the new behavior?

Link to demo site with this changes:
Screenshots with this changes:

How to use?

In NexT _config.yml:

njzjz commented 5 months ago

It fails as expected due to #728, and should be fixed by #727.

stevenjoezhang commented 4 months ago

Thank you, I have implemented a more strict integrity check in the @next-theme/plugins repository (https://github.com/next-theme/plugins/commit/6683aaee4f550881ced31b0549572797ad6f7b6c), and I believe it will resolve this issue. I will close this pull request.