fix(rum-oversight): unknown filter should return empty set

[karlpauls]

This applies adobe/helix-website#667 to the distiller.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 90.57%. Comparing base (55d4b3b) to head (e36ea0f).
Report is 1 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main       #9      +/-   ##
==========================================
+ Coverage   90.44%   90.57%   +0.12%     
==========================================
  Files          13       13              
  Lines        3421     3468      +47     
==========================================
+ Hits         3094     3141      +47     
  Misses        327      327              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[karlpauls]

@kptdobe, I'm trying to apply your change to oversight to the distiller - does this look correct (let me know if you rather want to create a PR yourself)?

Notice that I added the following:

--- a/tools/oversight/test/cruncher.test.js
+++ b/tools/oversight/test/cruncher.test.js
@@ -776,6 +776,7 @@ describe('DataChunks', () => {
     d.load(chunks1);
 
     // define facet functions
+    d.addFacet('host', (bundle) => bundle.host);
     d.addFacet(
       // as a convention, we use ! to indicate negation. When used in URL parameters, this
       // looks like a nice filter:  ?userAgent!=desktop

To make the tests pass (/cc @trieloff )

[kptdobe]

Yes, the host was missing in the test. I added it in my original PR: https://github.com/adobe/helix-website/pull/663/files#diff-52e8567f03066b4a3b5679116feddeffb369c606a16601299b7f3cbecbd5ae9aR700
I may have forgotten the line during "the backport"

[adobe-bot]

🎉 This PR is included in version 1.4.1 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀