Merge pull request #15 from mok-liee/master

fix: toString method can now handle 0 as number

src/comparators.ts

@@ -7,14 +7,14 @@ export abstract class Comparator implements Item {
   constructor(protected selector: Function | string, protected comparatorKeyWord: string, protected value?: string | number) {}
 
   toString(): string {
-    if (this.value) {
-      if (typeof this.value === 'number') {
-        return `${this.selector} ${this.comparatorKeyWord} ${this.value}`;
-      } else {
-        return `${this.selector} ${this.comparatorKeyWord} '${this.value}'`;
-      }
+    if (typeof this.value === 'undefined' || this.value === null) {
+      return `${this.selector} ${this.comparatorKeyWord}`;
+    }
+    if (typeof this.value === 'number') {
+      return `${this.selector} ${this.comparatorKeyWord} ${this.value}`;
+    } else {
+      return `${this.selector} ${this.comparatorKeyWord} '${this.value}'`;
     }
-    return `${this.selector} ${this.comparatorKeyWord}`;
   }
 }
 

test/comparators.test.ts

@@ -95,3 +95,7 @@ test('not in string[]', () => {
 test('not in number[]', () => {
   expect(sfNotIn('status', [1, 2]).toString()).toBe('status not in [1, 2]');
 });
+
+test('.toString() can handle 0 as number', () => {
+  expect(sfEqual('id', 0).toString()).toBe('id : 0');
+});