Skip to content

Removed debug() calls from Jest tests#2484

Merged
anth-volk merged 2 commits into
PolicyEngine:masterfrom
v-lerie:valerie/issue2210
Apr 22, 2025
Merged

Removed debug() calls from Jest tests#2484
anth-volk merged 2 commits into
PolicyEngine:masterfrom
v-lerie:valerie/issue2210

Conversation

@v-lerie
Copy link
Copy Markdown
Collaborator

@v-lerie v-lerie commented Apr 10, 2025

Description

Fixes #2210.

Changes

Removed the 'screen.debug()' statement from 'Research.test.js' test file.

@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 10, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
policyengine-app ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 20, 2025 3:52am

Comment thread src/__tests__/pages/Research.test.js Outdated
);

screen.debug();
// screen.debug();
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

issue, blocking: Please fully remove this line, don't just comment it out

@github-project-automation github-project-automation Bot moved this from Todo to PR: Review Requested in policyengine-app Apr 16, 2025
@anth-volk
Copy link
Copy Markdown
Collaborator

Please do the following:

  1. Give the PR a name that represents what you're trying to do (e.g., Remove debug() calls from Jest tests)
  2. In the description on the PR, adds Fixes Remove debug() calls from Jest tests #2210
  3. Check why the tests here fail

@v-lerie v-lerie changed the title Fixes {2210} Removed debug() calls from Jest tests Apr 20, 2025
@anth-volk anth-volk self-requested a review April 22, 2025 20:48
Copy link
Copy Markdown
Collaborator

@anth-volk anth-volk left a comment

Choose a reason for hiding this comment

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

@v-lerie thanks for your modifications to this!

@anth-volk anth-volk merged commit e36ecc9 into PolicyEngine:master Apr 22, 2025
4 checks passed
@github-project-automation github-project-automation Bot moved this from PR: Review Requested to Done in policyengine-app Apr 22, 2025
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.

Remove debug() calls from Jest tests

2 participants