Add fix and test

Signed-off-by: Nicolas Chamo <nicolas@chamo.com.ar>

spec/unit/filter-component.spec.js

@@ -0,0 +1,39 @@
+import {FilterComponent} from "../../src/filter-component"
+import {mkEvent} from '../test-utils'
+
+describe("Filter Component", function() {
+
+    describe("types", function() {
+        it("should filter out events with other types", function() {
+            const filter = new FilterComponent({ types: [ 'm.room.message' ] })
+
+            const event = mkEvent({
+                type: 'm.room.member',
+                content: {  },
+                room: 'roomId',
+                event: true,
+            })
+
+            const checkResult = filter.check(event)
+
+            expect(checkResult).toBe(false);
+        });
+
+        it("should validate events with the same type", function() {
+            const filter = new FilterComponent({ types: [ 'm.room.message' ] })
+
+            const event = mkEvent({
+                type: 'm.room.message',
+                content: {  },
+                room: 'roomId',
+                event: true,
+            })
+
+            const checkResult = filter.check(event)
+
+            expect(checkResult).toBe(true);
+        });
+
+    });
+
+});

src/filter-component.js

@@ -108,8 +108,9 @@ FilterComponent.prototype._checkFields =
         }
 
         const allowed_values = self[name];
-        if (allowed_values) {
-            if (!allowed_values.map(match_func)) {
+        if (allowed_values && allowed_values.length > 0) {
+            const anyMatch = allowed_values.map(match_func).reduce((a, b) => a || b);
+            if (!anyMatch) {
                 return false;
             }
         }