Skip to content

🌱 improve the unit Test coverage in client tests#637

Open
adity1raut wants to merge 2 commits intokubestellar:mainfrom
adity1raut:TestingClient
Open

🌱 improve the unit Test coverage in client tests#637
adity1raut wants to merge 2 commits intokubestellar:mainfrom
adity1raut:TestingClient

Conversation

@adity1raut
Copy link
Copy Markdown
Contributor

@adity1raut adity1raut commented Jan 13, 2026

…nt tests

Summary

feature : Refactor kubeconfig test utilities and enhance error handling in client tests

Related issue(s)

…ent tests

Signed-off-by: adity1raut <araut7798@gmail.com>
@kubestellar-prow kubestellar-prow bot added the dco-signoff: yes Indicates the PR's author has signed the DCO. label Jan 13, 2026
@kubestellar-prow
Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign mikespreitzer for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@kubestellar-prow kubestellar-prow bot added the needs-ok-to-test Indicates a PR that requires an org member to verify it is safe to test. label Jan 13, 2026
@kubestellar-prow
Copy link
Copy Markdown
Contributor

Hi @adity1raut. Thanks for your PR.

I'm waiting for a kubestellar member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.

Once the patch is verified, the new status will be reflected by the ok-to-test label.

I understand the commands that are listed here.

Details

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@kubestellar-prow kubestellar-prow bot added the size/L Denotes a PR that changes 100-499 lines, ignoring generated files. label Jan 13, 2026
@adity1raut adity1raut changed the title ✨ feature : Refactor kubeconfig test utilities and enhance error handling in client tests ✨ feature : Add The unit Test coverage in client tests Jan 13, 2026
…ent tests

Signed-off-by: adity1raut <araut7798@gmail.com>
@adity1raut
Copy link
Copy Markdown
Contributor Author

/cc @clubanderson PATL

@kubestellar-prow
Copy link
Copy Markdown
Contributor

@adity1raut: GitHub didn't allow me to request PR reviews from the following users: PATL.

Note that only kubestellar members and repo collaborators can review this PR, and authors cannot review their own PRs.

Details

In response to this:

/cc @clubanderson PATL

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@adity1raut
Copy link
Copy Markdown
Contributor Author

/cc @MikeSpreitzer PTAL

@kubestellar-prow
Copy link
Copy Markdown
Contributor

@adity1raut: GitHub didn't allow me to request PR reviews from the following users: PTAL.

Note that only kubestellar members and repo collaborators can review this PR, and authors cannot review their own PRs.

Details

In response to this:

/cc @MikeSpreitzer PTAL

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository.

@clubanderson clubanderson added the kind/test Categorizes issue or PR as related to testing. label Jan 13, 2026
@clubanderson
Copy link
Copy Markdown
Contributor

❌ PR Title Verification Failed

Your PR title does not follow the required format.

Current title: ✨ feature : Add The unit Test coverage in client tests

Required Format

PR titles must start with one of these emoji prefixes:

Emoji Meaning
⚠️ Breaking change
Non-breaking feature
🐛 Patch fix / Bug fix
📖 Documentation
🚀 Release
🌱 Infra/Tests/Other

How to Fix

Edit your PR title to start with the appropriate emoji. For example:

  • ✨ Add new feature for user authentication
  • 🐛 Fix crash when loading empty config
  • 📖 Update installation guide

You can edit the title by clicking the Edit button next to your PR title.


This comment was posted to help you fix the PR title format.

@adity1raut adity1raut changed the title ✨ feature : Add The unit Test coverage in client tests 🌱 improve the unit Test coverage in client tests Jan 14, 2026
@clubanderson
Copy link
Copy Markdown
Contributor

/ok-to-test

@kubestellar-prow kubestellar-prow bot added ok-to-test Indicates a non-member PR verified by an org member that is safe to test. and removed needs-ok-to-test Indicates a PR that requires an org member to verify it is safe to test. labels Jan 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dco-signoff: yes Indicates the PR's author has signed the DCO. kind/test Categorizes issue or PR as related to testing. ok-to-test Indicates a non-member PR verified by an org member that is safe to test. size/L Denotes a PR that changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feature: Improve Unit Test Coverage Across the Project

2 participants