Added parser tests back in

Author: marziply

tests/filter.spec.js

@@ -4,188 +4,196 @@ import yank from '../src/yank.js'
 import data from './data.json'
 
 describe('src/filter', () => {
-  describe('as', () => {
-    it('should set node.key.name to the argument given', () => {
-      const name = 'test-name'
-      const node = {
-        key: {
-          name: null
+  describe('filters', () => {
+    describe('as', () => {
+      it('should set node.key.name to the argument given', () => {
+        const name = 'test-name'
+        const node = {
+          key: {
+            name: null
+          }
         }
-      }
-
-      filters.as({
-        node,
-        data,
-        flag: null,
-        args: [
-          name
-        ]
-      })
 
-      expect(node.key.name).toEqual(name)
-    })
+        filters.as({
+          node,
+          data,
+          flag: null,
+          args: [
+            name
+          ]
+        })
+
+        expect(node.key.name).toEqual(name)
+      })
 
-    it('should rename a property after picking it from a data object', () => {
-      const result = yank(data, `
+      it('should rename a property after picking it from a data object', () => {
+        const result = yank(data, `
         dateOfBirth | as(dob)
       `)
 
-      expect(result).toEqual({
-        dob: data.dateOfBirth
+        expect(result).toEqual({
+          dob: data.dateOfBirth
+        })
       })
     })
-  })
 
-  describe('nullable', () => {
-    it('should set node.options.nullable to true', () => {
-      const node = {
-        options: {
-          nullable: false
+    describe('nullable', () => {
+      it('should set node.options.nullable to true', () => {
+        const node = {
+          options: {
+            nullable: false
+          }
         }
-      }
-
-      filters.nullable({
-        node,
-        data,
-        flag: {
-          on: jest.fn()
-        },
-        args: []
-      })
 
-      expect(node.options.nullable).toBe(true)
-    })
+        filters.nullable({
+          node,
+          data,
+          flag: {
+            on: jest.fn()
+          },
+          args: []
+        })
 
-    it('MACRO flag should set all children to nullable', () => {
-      const node = {
-        options: {
-          nullable: true
-        },
-        children: [
-          {
-            options: {
-              nullable: false
-            }
+        expect(node.options.nullable).toBe(true)
+      })
+
+      it('MACRO flag should set all children to nullable', () => {
+        const node = {
+          options: {
+            nullable: true
           },
-          {
-            options: {
-              nullable: false
+          children: [
+            {
+              options: {
+                nullable: false
+              }
+            },
+            {
+              options: {
+                nullable: false
+              }
             }
-          }
-        ]
-      }
-
-      filters.nullable({
-        node,
-        data,
-        args: [],
-        flag: {
-          on: (_, cb) => cb()
+          ]
         }
-      })
 
-      expect(node.options.nullable).toBe(true)
-      expect(node.children.every(c => c.options.nullable === true)).toBe(true)
-    })
+        filters.nullable({
+          node,
+          data,
+          args: [],
+          flag: {
+            on: (_, cb) => cb()
+          }
+        })
+
+        expect(node.options.nullable).toBe(true)
+        expect(node.children.every(c => c.options.nullable === true)).toBe(true)
+      })
 
-    it('should set undefined values to null', () => {
-      const result = yank(data, `
+      it('should set undefined values to null', () => {
+        const result = yank(data, `
         noValueHere | nullable,
-        alsoNoValueHere
-      `)
+          alsoNoValueHere
+        `)
 
-      expect(result.alsoNoValueHere).toBeUndefined()
-      expect(result).toEqual({
-        noValueHere: null
+        expect(result.alsoNoValueHere).toBeUndefined()
+        expect(result).toEqual({
+          noValueHere: null
+        })
       })
     })
-  })
 
-  describe('extract', () => {
-    it('should set node.options.extract to true', () => {
-      const node = {
-        options: {
-          extract: false
+    describe('extract', () => {
+      it('should set node.options.extract to true', () => {
+        const node = {
+          options: {
+            extract: false
+          }
         }
-      }
 
-      filters.extract({
-        node,
-        data,
-        args: [],
-        flag: null
-      })
+        filters.extract({
+          node,
+          data,
+          args: [],
+          flag: null
+        })
 
-      expect(node.options.extract).toBe(true)
-    })
+        expect(node.options.extract).toBe(true)
+      })
 
-    it('should merge current depth of values into parent depth', () => {
-      const result = yank(data, `
+      it('should merge current depth of values into parent depth', () => {
+        const result = yank(data, `
         addressDetails | extract: {
           address1,
           address2
         }
       `)
 
-      expect(result.addressDetails).toBeUndefined()
-      expect(result).toEqual({
-        address1: data.addressDetails.address1,
-        address2: data.addressDetails.address2
+        expect(result.addressDetails).toBeUndefined()
+        expect(result).toEqual({
+          address1: data.addressDetails.address1,
+          address2: data.addressDetails.address2
+        })
       })
     })
-  })
 
-  describe('exec', () => {
-    it('should set node.options.exec to a method in the current depth', () => {
-      const value = 'test-value'
-      const arg = 'test-arg'
-      const args = [
-        'arg1',
-        'arg2'
-      ]
-      const dataWithFn = {
-        ...data,
-        testFn: jest.fn().mockReturnValue(value)
-      }
-      const node = {
-        options: {
-          exec: null
-        }
-      }
-
-      filters.exec({
-        node,
-        data: dataWithFn,
-        flag: null,
-        args: [
-          'testFn',
-          ...args
+    describe('exec', () => {
+      it('should set node.options.exec to a method in the current depth', () => {
+        const value = 'test-value'
+        const arg = 'test-arg'
+        const args = [
+          'arg1',
+          'arg2'
         ]
-      })
+        const dataWithFn = {
+          ...data,
+          testFn: jest.fn().mockReturnValue(value)
+        }
+        const node = {
+          options: {
+            exec: null
+          }
+        }
 
-      const result = node.options.exec(arg)
+        filters.exec({
+          node,
+          data: dataWithFn,
+          flag: null,
+          args: [
+            'testFn',
+            ...args
+          ]
+        })
 
-      expect(dataWithFn.testFn).toBeCalledWith(arg, ...args)
-      expect(result).toEqual(value)
-    })
+        const result = node.options.exec(arg)
+
+        expect(dataWithFn.testFn).toBeCalledWith(arg, ...args)
+        expect(result).toEqual(value)
+      })
 
-    it('should execute a method in the current depth with the picked value', () => {
-      const value = 'test-value'
-      const arg = 'foo'
-      const dataWithFn = {
-        ...data,
-        testFn: jest
-          .fn()
-          .mockReturnValue(value)
-      }
-      const result = yank(dataWithFn, `
+      it('should execute a method in the current depth with the picked value', () => {
+        const value = 'test-value'
+        const arg = 'foo'
+        const dataWithFn = {
+          ...data,
+          testFn: jest
+            .fn()
+            .mockReturnValue(value)
+        }
+        const result = yank(dataWithFn, `
         firstName | exec(testFn, ${arg})
       `)
 
-      expect(dataWithFn.testFn).toBeCalledWith(data.firstName, arg)
-      expect(result).toEqual({
-        firstName: value
+        expect(dataWithFn.testFn).toBeCalledWith(data.firstName, arg)
+        expect(result).toEqual({
+          firstName: value
+        })
       })
     })
   })
+
+  describe('Filter', () => {
+    describe('filter', () => {
+      // @TODO
+    })
+  })
 })

tests/parser.spec.js

@@ -57,25 +57,6 @@ describe('src/parser', () => {
     })
   })
 
-  describe('filterfy', () => {
-    it('should create an array of filters from a given node in a schema', () => {
-      const raws = ['nullable', 'as(first_name)']
-      const parser = new Parser([])
-      const [nullable, as] = parser.filterfy(raws)
-
-      expect(nullable).toEqual(expect.objectContaining({
-        name: 'nullable',
-        args: []
-      }))
-      expect(as).toEqual(expect.objectContaining({
-        name: 'as',
-        args: [
-          'first_name'
-        ]
-      }))
-    })
-  })
-
   describe('tokenise', () => {
     const tokenise = tok => {
       const parser = new Parser([])
@@ -158,7 +139,7 @@ describe('src/parser', () => {
   })
 
   describe('measure', () => {
-    it.only('should measure the size of scope given', () => {
+    it('should measure the size of scope given', () => {
       const parser = new Parser([])
 
       expect(parser.measure(':{')).toEqual(1)