Skip to content

Fix frontend test command to pass when no tests are present - #17

Merged
namann5 merged 1 commit into
mainfrom
codex/debug-code
Apr 15, 2026
Merged

Fix frontend test command to pass when no tests are present#17
namann5 merged 1 commit into
mainfrom
codex/debug-code

Conversation

@namann5

@namann5 namann5 commented Apr 15, 2026

Copy link
Copy Markdown
Owner

Motivation

  • CI and local test runs failed when the frontend has no test files, so the npm test command should succeed in that case to avoid false failures.

Description

  • Updated deepscan-frontend/package.json to change the test script to react-scripts test --passWithNoTests so the command exits successfully when no tests are found.

Testing

  • Ran backend tests with cd deepscan-backend && npm test -- --runInBand (passed) and verified the frontend test command reproduced the failure before the change and now succeeds with CI=true npm test -- --watchAll=false (exits 0 with no tests).

Codex Task

Summary by CodeRabbit

  • Chores
    • Updated test script configuration to improve test execution handling.

@vercel

vercel Bot commented Apr 15, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
ai-deepfake-frontend Ready Ready Preview, Comment Apr 15, 2026 3:50pm

@coderabbitai

coderabbitai Bot commented Apr 15, 2026

Copy link
Copy Markdown

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 0b641bbb-443b-4bd4-aebb-4a2489f1fbea

📥 Commits

Reviewing files that changed from the base of the PR and between c71f2a2 and 783d243.

📒 Files selected for processing (1)
  • deepscan-frontend/package.json

📝 Walkthrough

Walkthrough

The test npm script in the frontend package.json was modified to add the --passWithNoTests flag to the react-scripts test command, allowing test runs to pass successfully when no test files are present.

Changes

Cohort / File(s) Summary
Test Script Configuration
deepscan-frontend/package.json
Added --passWithNoTests flag to the test script, changing react-scripts test to react-scripts test --passWithNoTests to handle scenarios where no tests exist.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

🐰 A flag so small, yet wise and true,
When no tests exist, it passes through,
The scripts now graceful, never shall fail,
Success prevails, a rabbit's tail! 🐇✨

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch codex/debug-code

Comment @coderabbitai help to get the list of available commands and usage tips.

@namann5
namann5 merged commit 4413bba into main Apr 15, 2026
3 of 5 checks passed
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.

1 participant