Skip to content

Add test commands for running and coverage#26

Merged
Mujhtech merged 2 commits into
mainfrom
chore/update-dagryn-workflow-stuck-test
Apr 22, 2026
Merged

Add test commands for running and coverage#26
Mujhtech merged 2 commits into
mainfrom
chore/update-dagryn-workflow-stuck-test

Conversation

@Mujhtech

Copy link
Copy Markdown
Owner

Description

[Provide a brief description of the changes in this pull request]

Related Issue

[If applicable, link to the issue this PR addresses]

Type of Change

  • Bug fix
  • New feature
  • Documentation update
  • Code refactor
  • Other (please specify)

Checklist

  • I have read the contribution guidelines
  • My code follows the project's coding style
  • I have tested my changes
  • I have updated the documentation accordingly

Additional Notes

[Add any additional information or context about the PR here]

Copilot AI review requested due to automatic review settings April 22, 2026 10:36
@dagryn-dev

dagryn-dev Bot commented Apr 22, 2026

Copy link
Copy Markdown

🟡 Dagryn workflow RUNNING

  • PR: Add test commands for running and coverage
  • Commit: Add test commands for running and coverage
  • Branch: chore/update-dagryn-workflow-stuck-test
  • SHA: 11c7276

View run in Dagryn

@dagryn-dev

dagryn-dev Bot commented Apr 22, 2026

Copy link
Copy Markdown

🟡 Dagryn workflow RUNNING

  • PR: Add test commands for running and coverage
  • Commit: Add test commands for running and coverage
  • Branch: chore/update-dagryn-workflow-stuck-test
  • SHA: 11c7276

View run in Dagryn

@dagryn-dev

dagryn-dev Bot commented Apr 22, 2026

Copy link
Copy Markdown

🟡 Dagryn workflow RUNNING

  • PR: Add test commands for running and coverage
  • Commit: Add test commands for running and coverage
  • Branch: chore/update-dagryn-workflow-stuck-test
  • SHA: 11c7276

View run in Dagryn

@dagryn-dev

dagryn-dev Bot commented Apr 22, 2026

Copy link
Copy Markdown

🟡 Dagryn workflow RUNNING

  • PR: Add test commands for running and coverage
  • Commit: Add test commands for running and coverage
  • Branch: chore/update-dagryn-workflow-stuck-test
  • SHA: 11c7276

View run in Dagryn

@dagryn-dev

dagryn-dev Bot commented Apr 22, 2026

Copy link
Copy Markdown

Dagryn workflow FAILED

  • PR: Add test commands for running and coverage
  • Commit: Change frontend test command to 'pnpm run test:run'
  • Branch: chore/update-dagryn-workflow-stuck-test
  • SHA: effc382
  • Duration: 5m 46s

View run in Dagryn

@dagryn-dev

dagryn-dev Bot commented Apr 22, 2026

Copy link
Copy Markdown

Dagryn workflow FAILED

  • PR: Add test commands for running and coverage
  • Commit: Change frontend test command to 'pnpm run test:run'
  • Branch: chore/update-dagryn-workflow-stuck-test
  • SHA: effc382
  • Duration: 5m 46s

View run in Dagryn

@Mujhtech
Mujhtech merged commit 46c4fe9 into main Apr 22, 2026
3 of 6 checks passed
@dagryn-dev

dagryn-dev Bot commented Apr 22, 2026

Copy link
Copy Markdown

Dagryn workflow FAILED

  • PR: Add test commands for running and coverage
  • Commit: Change frontend test command to 'pnpm run test:run'
  • Branch: chore/update-dagryn-workflow-stuck-test
  • SHA: effc382
  • Duration: 5m 33s

View run in Dagryn

@dagryn-dev

dagryn-dev Bot commented Apr 22, 2026

Copy link
Copy Markdown

Dagryn workflow FAILED

  • PR: Add test commands for running and coverage
  • Commit: Change frontend test command to 'pnpm run test:run'
  • Branch: chore/update-dagryn-workflow-stuck-test
  • SHA: effc382
  • Duration: 5m 27s

View run in Dagryn

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds explicit frontend test scripts for non-watch execution and coverage collection to standardize local/CI usage.

Changes:

  • Add test:run script (vitest run) for one-shot test execution
  • Add test:coverage script (vitest run --coverage) for coverage runs
  • Remove the previous coverage script name

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread frontend/package.json
Comment on lines 12 to +14
"test": "vitest",
"coverage": "vitest run --coverage"
"test:run": "vitest run",
"test:coverage": "vitest run --coverage"

Copilot AI Apr 22, 2026

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

test:coverage runs vitest run --coverage, but this repo doesn’t include a Vitest coverage provider (e.g. @vitest/coverage-v8 or @vitest/coverage-istanbul) in dependencies/lockfile. As-is, this script will fail when executed; add an appropriate @vitest/coverage-* devDependency (and configure Vitest if needed) so the command works.

Copilot uses AI. Check for mistakes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants