Merge pull request #11 from JasonEtco/sting-delimited

Comma delimited lists

Author: JasonEtco

README.md

@@ -23,10 +23,8 @@ This reads from the `.github/ISSUE_TEMPLATE.md` file. This file should have fron
 ```markdown
 ---
 title: Someone just pushed
-assignees:
-  - JasonEtco
-labels:
-  - bug
+assignees: JasonEtco, matchai
+labels: bug, enhancement
 ---
 Someone just pushed, oh no! Here's who did it: {{ payload.sender.login }}
 ```

index.js

@@ -3,6 +3,11 @@ const fm = require('front-matter')
 const nunjucks = require('nunjucks')
 const dateFilter = require('nunjucks-date-filter')
 
+function listToArray (list) {
+  if (!list) return []
+  return Array.isArray(list) ? list : list.split(', ')
+}
+
 Toolkit.run(async tools => {
   const template = tools.arguments._[0] || '.github/ISSUE_TEMPLATE.md'
   const env = nunjucks.configure({ autoescape: false })
@@ -33,8 +38,8 @@ Toolkit.run(async tools => {
   const issue = await tools.github.issues.create({
     ...tools.context.repo,
     ...templated,
-    assignees: attributes.assignees || [],
-    labels: attributes.labels || [],
+    assignees: listToArray(attributes.assignees),
+    labels: listToArray(attributes.labels),
     milestone: attributes.milestone
   })
 

tests/__snapshots__/index.test.js.snap

@@ -34,6 +34,29 @@ Array [
 ]
 `;
 
+exports[`create-an-issue creates a new issue with assignees and labels as comma-delimited strings 1`] = `
+Object {
+  "assignees": Array [
+    "JasonEtco",
+    "matchai",
+  ],
+  "body": "The action create-an-issue is the best action.",
+  "labels": Array [
+    "bug",
+    "enhancement",
+  ],
+  "title": "DO EVERYTHING",
+}
+`;
+
+exports[`create-an-issue creates a new issue with assignees and labels as comma-delimited strings 2`] = `
+Array [
+  Array [
+    "Created issue DO EVERYTHING#1: www",
+  ],
+]
+`;
+
 exports[`create-an-issue creates a new issue with assignees, labels and a milestone 1`] = `
 Object {
   "assignees": Array [

tests/fixtures/.github/split-strings.md

@@ -0,0 +1,6 @@
+---
+title: DO EVERYTHING
+assignees: JasonEtco, matchai
+labels: bug, enhancement
+---
+The action {{ action }} is the best action.

tests/index.test.js

@@ -62,4 +62,12 @@ describe('create-an-issue', () => {
     expect(tools.log.success).toHaveBeenCalled()
     expect(tools.log.success.mock.calls).toMatchSnapshot()
   })
+
+  it('creates a new issue with assignees and labels as comma-delimited strings', async () => {
+    tools.arguments._[0] = '.github/split-strings.md'
+    await actionFn(tools)
+    expect(params).toMatchSnapshot()
+    expect(tools.log.success).toHaveBeenCalled()
+    expect(tools.log.success.mock.calls).toMatchSnapshot()
+  })
 })