releaser: Add Vitest coverage for utility functions#801
Open
skools-here wants to merge 15 commits into
Open
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
This PR adds unit test coverage (via Vitest) for the tools/releaser utility modules that support the release pipeline (version handling, git tagging/changelog, GitHub releases/assets, plugin discovery, and ArtifactHub metadata updates), addressing #798.
Changes:
- Adds new Vitest configuration for the
tools/releaserpackage (including coverage settings). - Introduces new unit tests for releaser utility modules:
artifacthub,git,github,plugin, andversion.
Reviewed changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| tools/releaser/vitest.config.ts | Adds Vitest + coverage configuration for releaser utility tests. |
| tools/releaser/src/utils/version.test.ts | Adds tests for version sanitization/validation helpers. |
| tools/releaser/src/utils/plugin.test.ts | Adds tests for plugin path discovery, plugin listing, metadata extraction, and tarball selection. |
| tools/releaser/src/utils/github.test.ts | Adds tests for GitHub release creation, lookup, asset upload, and publish behavior. |
| tools/releaser/src/utils/git.test.ts | Adds tests for git helper functions (repo root, tags, status, changelog, etc.). |
| tools/releaser/src/utils/artifacthub.test.ts | Adds tests for ArtifactHub config/template read/update helpers. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
skools-here
force-pushed
the
releaser-utils-vitest-coverage
branch
from
8d65215 to
72f4cbf
Compare
skools-here
force-pushed
the
releaser-utils-vitest-coverage
branch
from
8e2d6a8 to
850c896
Compare
skools-here
force-pushed
the
releaser-utils-vitest-coverage
branch
from
ea54389 to
2693121
Compare
skools-here
force-pushed
the
releaser-utils-vitest-coverage
branch
from
84f5b1f to
7f0a3b9
Compare
skools-here
force-pushed
the
releaser-utils-vitest-coverage
branch
from
8c253c7 to
409c528
Compare
skools-here
force-pushed
the
releaser-utils-vitest-coverage
branch
from
c651d15 to
c7dcbc7
Compare
Author
|
Hi! @illume I've fixed the Copilot review comments . Could you please take another look when you have a chance? Thanks! |
skools-here
force-pushed
the
releaser-utils-vitest-coverage
branch
from
8171ea6 to
69e57c3
Compare
skools-here
force-pushed
the
releaser-utils-vitest-coverage
branch
from
a6be3c7 to
5d9e7e2
Compare
skools-here
force-pushed
the
releaser-utils-vitest-coverage
branch
from
0ef2cb3 to
667a65a
Compare
Signed-off-by: skools-here <sajalkulshreshtha9@gmail.com>
Signed-off-by: skools-here <sajalkulshreshtha9@gmail.com>
Signed-off-by: skools-here <sajalkulshreshtha9@gmail.com>
Signed-off-by: skools-here <sajalkulshreshtha9@gmail.com>
Signed-off-by: skools-here <sajalkulshreshtha9@gmail.com>
Signed-off-by: skools-here <sajalkulshreshtha9@gmail.com>
Signed-off-by: skools-here <sajalkulshreshtha9@gmail.com>
Signed-off-by: skools-here <sajalkulshreshtha9@gmail.com>
Signed-off-by: skools-here <sajalkulshreshtha9@gmail.com>
Signed-off-by: skools-here <sajalkulshreshtha9@gmail.com>
Signed-off-by: skools-here <sajalkulshreshtha9@gmail.com>
Signed-off-by: skools-here <sajalkulshreshtha9@gmail.com>
Signed-off-by: skools-here <sajalkulshreshtha9@gmail.com>
Signed-off-by: skools-here <sajalkulshreshtha9@gmail.com>
Signed-off-by: skools-here <sajalkulshreshtha9@gmail.com>
skools-here
force-pushed
the
releaser-utils-vitest-coverage
branch
from
89d0754 to
5d8e7a7
Compare
Author
|
ptal @illume |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Adds Vitest test coverage for the releaser utility modules.
fixes : #798
Changes
artifacthub.ts)git.ts)github.ts)plugin.ts)version.ts)Results