improve unit tests

Author: GrantBirki

__tests__/functions/params.test.js

@@ -13,14 +13,14 @@ test('with empty param object', async () => {
 })
 
 test('it parses positional parameters', async () => {
-  expect(parseParams('foo')).toHaveProperty('_', ['foo'])
+  expect(parseParams('foo bar baz')).toHaveProperty('_', ['foo', 'bar', 'baz'])
 })
 
 test('it parses arguments using the default settings of library', async () => {
-  const parsed = parseParams('--foo bar --env.foo=bar')
+  const parsed = parseParams('--foo bar --env.foo=bar baz')
   expect(parsed).toHaveProperty('foo', 'bar')
   expect(parsed).toHaveProperty('env', {foo: 'bar'})
-  expect(parsed).toHaveProperty('_', [])
+  expect(parsed).toHaveProperty('_', ['baz'])
 })
 
 test('it works with empty string', async () => {

__tests__/main.test.js

@@ -1069,9 +1069,51 @@ test('stores params and parsed params into context', async () => {
   }
   const data = expect.objectContaining({
     auto_merge: true,
+    ref: 'test-ref',
+    environment: 'production',
+    owner: 'corp',
+    repo: 'test',
+    production_environment: true,
+    required_contexts: [],
     payload: expect.objectContaining({
       params,
-      parsed_params
+      parsed_params,
+      sha: null,
+      type: 'branch-deploy'
+    })
+  })
+  expect(await run()).toBe('success')
+  expect(createDeploymentMock).toHaveBeenCalledWith(data)
+  expect(setOutputMock).toHaveBeenCalledWith('params', params)
+  expect(setOutputMock).toHaveBeenCalledWith('parsed_params', parsed_params)
+})
+
+test('stores params and parsed params into context with complex params', async () => {
+  github.context.payload.comment.body =
+    '.deploy | something1 --foo=bar --env.development=false --env.production=true'
+  const params =
+    'something1 --foo=bar --env.development=false --env.production=true'
+  const parsed_params = {
+    _: ['something1'],
+    foo: 'bar',
+    env: {
+      development: 'false',
+      production: 'true'
+    }
+  }
+  const data = expect.objectContaining({
+    auto_merge: true,
+    ref: 'test-ref',
+    environment: 'production',
+    owner: 'corp',
+    repo: 'test',
+    production_environment: true,
+    required_contexts: [],
+    payload: expect.objectContaining({
+      params,
+      parsed_params,
+      sha: null,
+      type: 'branch-deploy'
     })
   })
   expect(await run()).toBe('success')