Skip to content

test: add unit tests for remove - #607

Closed
Shreya2005-2005 wants to merge 1 commit into
urunc-dev:mainfrom
Shreya2005-2005:test/add-remove-tests
Closed

test: add unit tests for remove#607
Shreya2005-2005 wants to merge 1 commit into
urunc-dev:mainfrom
Shreya2005-2005:test/add-remove-tests

Conversation

@Shreya2005-2005

Copy link
Copy Markdown

Description

Adds unit tests for the remove() helper function in pkg/unikontainers/utils.go.
This function had no test coverage. The tests cover:

  • removing the first element
  • removing a middle element
  • removing the only element

Related issues

How was this tested?

Ran unit tests locally:
go test ./pkg/unikontainers/... -v -run TestRemove
All 3 test cases passed. Linter also passes locally (make lint).

LLM usage

Claude Sonnet 4.6 was used to assist in understanding the codebase and test structure. All code was reviewed and understood before submission.

Checklist

  • I have read the contribution guide.
  • The linter passes locally (make lint).
  • The e2e tests of at least one tool pass locally.
  • If LLMs were used: I have read the llm policy.

Signed-off-by: Shreya2005-2005 <bhakatmistu2005@gmail.com>
@netlify

netlify Bot commented May 2, 2026

Copy link
Copy Markdown

Deploy Preview for urunc canceled.

Name Link
🔨 Latest commit 4ad9ffa
🔍 Latest deploy log https://app.netlify.com/projects/urunc/deploys/69f59b77be87a50007655839

@cmainas

cmainas commented May 5, 2026

Copy link
Copy Markdown
Contributor

Hello @Shreya2005-2005 ,

there is no reason to test this function. It simple uses Go semantics. Also looking at remove we should replace it, having as reference https://go.dev/wiki/SliceTricks

@cmainas cmainas closed this May 5, 2026
@cmainas cmainas reopened this May 5, 2026
@github-actions

github-actions Bot commented Jul 5, 2026

Copy link
Copy Markdown

This PR is stale because it has been open 60 days with no activity. Remove stale label or comment or this will be closed in 15 days.

@github-actions github-actions Bot added the Stale label Jul 5, 2026
@github-actions

Copy link
Copy Markdown

This PR was closed because it has been stalled for 75 days with no activity.

@github-actions github-actions Bot closed this Jul 21, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add more unit tests

2 participants