fix: get regex to actually work

Author: theBenForce

src/StreamCleaner.test.ts

@@ -2,20 +2,44 @@ import { Readable } from "stream";
 import StreamCleaner, { MATCH_NON_PRINTABLE } from "./StreamCleaner";
 
 describe("StreamCleaner", () => {
-  test("remove non-printable characters", () => {
+  test("remove non-printable characters", async () => {
     const source = new Readable();
 
     const transform = new StreamCleaner(MATCH_NON_PRINTABLE);
 
-    source
-      .pipe(transform)
-      .on("data", chunk =>
-        expect(chunk.toString()).not.toMatch(MATCH_NON_PRINTABLE)
-      );
+    let result = new Promise(resolve => {
+      source
+        .pipe(transform)
+        .on("data", (chunk: Buffer) => expect(chunk).toHaveLength(1))
+        .on("end", resolve);
+    });
 
     for (let i = 0; i <= 31; i++) {
+      const chr = String.fromCharCode(i);
+      source.push(chr + "A" + chr);
+    }
+    source.push(null);
+
+    await result;
+  });
+
+  test("keep printable characters", async () => {
+    const source = new Readable();
+
+    const transform = new StreamCleaner(MATCH_NON_PRINTABLE);
+
+    let result = new Promise(resolve => {
+      source
+        .pipe(transform)
+        .on("data", (chunk: Buffer) => expect(chunk).toHaveLength(1))
+        .on("end", resolve);
+    });
+
+    for (let i = 32; i <= 126; i++) {
       source.push(String.fromCharCode(i));
     }
     source.push(null);
+
+    await result;
   });
 });

src/StreamCleaner.ts

@@ -1,6 +1,6 @@
 import { Transform, TransformCallback } from "stream";
 
-export const MATCH_NON_PRINTABLE = /[^\000-\031]+/gi;
+export const MATCH_NON_PRINTABLE = /[\u0000-\u0031]+/g;
 
 export default class StreamCleaner extends Transform {
   constructor(private readonly pattern: RegExp) {