Skip to content

Allow notification sync test to load poller - #10576

Open
Tomeshwari-02 wants to merge 1 commit into
SandeepVashishtha:masterfrom
Tomeshwari-02:fix/notification-poller-json-load-10566
Open

Allow notification sync test to load poller#10576
Tomeshwari-02 wants to merge 1 commit into
SandeepVashishtha:masterfrom
Tomeshwari-02:fix/notification-poller-json-load-10566

Conversation

@Tomeshwari-02

Copy link
Copy Markdown

Summary

  • removes the static mock notification JSON import from the poller load path
  • lets direct notification sync tests use their lightweight hook and auth harness
  • clears migrated legacy notification storage after merging

Validation

  • node --test tests\notificationSync.test.mjs

Fixes #10566

@vercel

vercel Bot commented Jul 13, 2026

Copy link
Copy Markdown
Contributor

@Tomeshwari-02 is attempting to deploy a commit to the sandeepvashishtha's projects Team on Vercel.

A member of the Team first needs to authorize it.

@codescene-access codescene-access Bot 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.

Our agent can fix these. Install it.

No quality gates enabled for this code.

Quality Gate Profile: Custom Configuration
Install CodeScene MCP: safeguard and uplift AI-generated code. Catch issues early with our IDE extension and CLI tool.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bug: notificationSync test cannot import mockNotifications JSON in Node

1 participant