test: add unit tests for the module

[jrutila]

🔗 Linked issue

#546

❓ Type of change

• 📖 Documentation (updates to the documentation or readme)
• 🐞 Bug fix (a non-breaking change that fixes an issue)
• 👌 Enhancement (improving an existing functionality)
• ✨ New feature (a non-breaking change that adds functionality)
• 🧹 Chore (updates to the build process or auxiliary tools and libraries)
• ⚠️ Breaking change (fix or feature that would cause existing functionality to change)
• Tests

📚 Description

The codebase is missing unit tests. This PR will add the test bench and some basic cases from where to start expanding the test base. The tests now test that the module setup works correctly. Next step would be to add tests for components.

📝 Checklist

• I have linked an issue or discussion.
• I have updated the documentation accordingly.

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Updated (UTC)
ui	✅ Ready (Inspect)	Visit Preview	Nov 2, 2023 9:59am

[benjamincanac]

Thanks for taking the initiative of starting this! 😊 Do you think it would make sense to use https://github.com/danielroe/nuxt-vitest?

[jrutila]

Do you think it would make sense to use https://github.com/danielroe/nuxt-vitest?

Definitely that could be used when testing the components. The package seems to be targeted more on testing nuxt applications instead of modules. Can look into it.

To prevent feature creep for this PR the component testing should probably be in different PR. Should I still update the contribution guidelines in doc for this PR? How about build process?

[benjamincanac]

Not sure about the contribution guidelines as there is a draft PR that changes it already: #860

For the build process, you can just add a test step in .github/workflows/ci.yml and .github/workflows/ci-dev.yml.

[jrutila]

@benjamincanac I can't see what is the problem with Vercel. I added the test steps to the ci files.

[benjamincanac]

Should be good now! 😊

[jrutila]

The CI seems to be ok and build passes. Is this PR ok to complete or should I add something to it, sill?

[benjamincanac]

No I think it's a really good init! We can merge this and continue in other PRs. Thanks a lot! 😊