🐛 Bug Report: Digests with filter behave improperly

[MasonD]

📜 Description

When a digest has a filter attached and that filter succeeds, messages are sent immediately, resulting in one message per trigger with each message having a digest of the triggers leading up to it.

👟 Reproduction steps

This branch https://github.com/MasonD/novu/tree/digest-filter-failing-test has a failing test "should digest events within time interval with filter".

👍 Expected behavior

Digests behave identically to when they have no filter attached when they have a filter that passes. The test I wrote passes.

👎 Actual Behavior with Screenshots

The test I wrote fails. Digests send one message per trigger when they have a filter even when the filter passes.

Novu version

next (commit cf7ec7d)

npm version

8.11.0

node version

v16.15.1

📃 Provide any additional context for the Bug.

No response

👀 Have you spent some time to check if this bug has been raised before?

• I checked and didn't find a similar issue

🏢 Have you read the Contributing Guidelines?

• I have read the Contributing Guidelines

Are you willing to submit PR?

Yes I am willing to submit a PR!

[jainpawan21]

@davidsoderberg

[ainouzgali]

Hi @MasonD , thanks for reporting!! I pushed a fix into next, let me know if you still encounter problems :)