Skip to content

@ngrx/signals: Reduce console.warn noise in signals tests #4998

@rainerhahnekamp

Description

@rainerhahnekamp

Which @ngrx/* package(s) are relevant/related to the feature request?

signals

Information

In the signals package there are four test files that cover many edge cases:

  • modules/signals/rxjs-interop/spec/rx-method.spec.ts
  • modules/signals/spec/signal-method.spec.ts
  • modules/signals/spec/signal-store.spec.ts
  • modules/signals/spec/state-source.spec.ts

These tests produce a large number of console.warn messages, which creates unnecessary noise in the CI logs.

Each of these files already sets up spies for console.warn, but the current spies still call through to the real console. We should update the spies so that they do not call the real console.warn, while still allowing the tests to assert on the warnings via the existing spies.

Describe any alternatives/workarounds you're currently using

No response

I would be willing to submit a PR to fix this issue

  • Yes
  • No

Metadata

Metadata

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions