Skip to content

feat: added unit tests - #2539

Merged
bajrangCoder merged 3 commits into
Acode-Foundation:mainfrom
RohitKushvaha01:vitests
Jul 31, 2026
Merged

feat: added unit tests#2539
bajrangCoder merged 3 commits into
Acode-Foundation:mainfrom
RohitKushvaha01:vitests

Conversation

@RohitKushvaha01

Copy link
Copy Markdown
Member

No description provided.

@greptile-apps

greptile-apps Bot commented Jul 27, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

Adds a Vitest unit-testing setup and corrects quick-tools feedback cleanup.

  • Adds unit tests for utility modules and quick-tools state helpers.
  • Adds Vitest scripts, configuration, and development dependencies.
  • Separates container and button deduplication while clearing quick-tools feedback.

Confidence Score: 4/5

The PR should not merge until the unit-test suite is connected to pull-request CI.

The previously reported automation gap remains: pull-request workflows can pass without executing the newly added Vitest suite.

Files Needing Attention: package.json and .github/workflows/ci.yml

Important Files Changed

Filename Overview
package.json Adds Vitest scripts and dependency, but the test command remains disconnected from pull-request CI.
src/handlers/quickToolsState.js Separates visited containers from visited buttons so matching containers are correctly cleaned.
tests/unit/quickToolsState.test.js Updates regression coverage to require cleanup of containers that themselves carry feedback state.
vitest.config.js Configures source aliases and limits collection to the new unit-test directory.

Reviews (2): Last reviewed commit: "fix" | Re-trigger Greptile

Comment thread package.json
Comment thread tests/unit/quickToolsState.test.js Outdated
@bajrangCoder

This comment was marked as outdated.

Resolve package-lock.json conflict by regenerating lockfile from main with vitest.
@bajrangCoder
bajrangCoder added this pull request to the merge queue Jul 31, 2026
Merged via the queue into Acode-Foundation:main with commit 36b44d9 Jul 31, 2026
6 checks passed
@github-project-automation github-project-automation Bot moved this from Backlog to Done in The Code Board - Acode Jul 31, 2026
@RohitKushvaha01
RohitKushvaha01 deleted the vitests branch July 31, 2026 02:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants