feat(application-generic): add unit test script and CI job

[LetItRock]

Summary

Adds unit test infrastructure for libs/application-generic and a dedicated CI job in the PR workflow.

What changed

libs/application-generic/package.json

• Added "test:unit": "jest --config jest.config.js --forceExit src" script (was missing entirely). Uses test:unit name to avoid conflicting with Nx's test target detection (which would include it in test_unit_libs that requires a lint target)
• Added jest-environment-node@27.5.1 as devDependency to fix version mismatch — the hoisted jest-environment-node@30 from root was incompatible with the package's jest@27

libs/application-generic/jest.config.js

• Added testPathIgnorePatterns to exclude:
  • Integration tests that require running Redis/BullMQ (queues, bull-mq, in-memory-provider, cache-service)
  • Pre-existing broken suites annotated with TODOs for follow-up fixes (ESM compat, vitest imports, OpenTelemetry resolution, assertion bugs)

libs/application-generic/jest.setup.js

• Extended the newrelic mock with startSegment and recordMetric methods that were missing, causing test failures in storage and other services

libs/application-generic/src/.env.test

• Added test environment file with STORE_ENCRYPTION_KEY for encryption unit tests

.github/workflows/on-pr.yml

• Added test-unit-libs output to get-affected job — queries Nx for affected libs with test:unit target
• Added test_unit_application_generic job that:
  • Depends on get-affected (Nx affected detection via test:unit target)
  • Only runs when @novu/application-generic is in the affected test-unit-libs list
  • Uses slim project setup (no Redis/MongoDB/ClickHouse)
  • Builds workspace dependencies via Nx before running tests
  • Runs pnpm test:unit in the library directory

Test results

Running pnpm test:unit in libs/application-generic:

• 25 test suites pass, 430 tests pass, 0 failures
• 20 additional suites are excluded with TODO annotations for follow-up work:
  • ESM compatibility (p-queue, @launchdarkly/js-sdk-common)
  • vitest imports in Jest test files
  • @opentelemetry module resolution
  • Assertion style mismatch (chai .to.equal in Jest context)
  • Failing test assertions

Slack Thread

  

[netlify]

✅ Deploy Preview for dashboard-v2-novu-staging canceled.

Name	Link
🔨 Latest commit	b949ce8
🔍 Latest deploy log	https://app.netlify.com/projects/dashboard-v2-novu-staging/deploys/69dd2fb73b647d00080c2e8c

[coderabbitai]

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 51410592-3b3a-4ba8-a832-b1229c88130e

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

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

[github-actions]

Hey there and thank you for opening this pull request! 👋

We require pull request titles to follow specific formatting rules and it looks like your proposed title needs to be adjusted.

Your PR title is: feat(application-generic): add unit test script and CI job

Requirements:

1. Follow the Conventional Commits specification
2. As a team member, include Linear ticket ID at the end: fixes TICKET-ID or include it in your branch name

Expected format: feat(scope): Add fancy new feature fixes NOV-123

Details:

PR title must end with 'fixes TICKET-ID' (e.g., 'fixes NOV-123') or include ticket ID in branch name