fix(test): fix integration tests and add to CI (#221)

Author: wesleytodd

.github/workflows/test.yml

@@ -24,6 +24,18 @@ jobs:
     - run: npm install
     - run: npm test
 
+  testint:
+    runs-on: ubuntu-latest
+    steps:
+    - uses: actions/checkout@v4
+    - name: Use Node.js 24
+      uses: actions/setup-node@v4
+      with:
+        node-version: 24
+    - run: npm install
+    - run: npm run test:integration
+      env:
+          GITHUB_TOKEN: ${{ secrets.INT_TEST_TOKEN }}
   action-in-action:
     runs-on: ubuntu-latest
     steps:

lib/meetings.js

@@ -32,7 +32,7 @@ function getNextIssue (opts) {
     title,
     date,
     agendaLabel: opts.agendaLabel,
-    agendaIssues: opts.agendaIssues,
+    agendaIssues: opts.agendaIssues || [],
     meetingLink: opts.meetingLink,
     labels: opts.meetingLabels,
     meetingNotes: opts.meetingNotes || '',

package.json

@@ -13,7 +13,8 @@
   "scripts": {
     "build": "npx @vercel/ncc build",
     "test": "standard && mocha test/index.js",
-    "test:integration": "standard && mocha",
+    "test:integration": "standard && mocha test/integration.js",
+    "test:integration:debug": "GITHUB_TOKEN=$(gh auth token) mocha --inspect --inspect-brk test/integration.js",
     "lint:fix": "standard --fix",
     "preversion": "npm t",
     "postpublish": "git push origin && git push origin --tags",

test/integration.js

@@ -1,8 +1,8 @@
 'use strict'
-/* global Temporal */
 require('dotenv').config()
 const { suite, test, before } = require('mocha')
 const assert = require('assert')
+/* global Temporal */
 if (!global.Temporal) {
   const polyfill = require('@js-temporal/polyfill')
   global.Temporal = polyfill.Temporal
@@ -20,18 +20,21 @@ suite(`${pkg.name} integration`, () => {
     const issue = await meetings.shouldCreateNextMeetingIssue(client, {
       owner: 'wesleytodd',
       repo: 'meeting-maker',
-      issueTitle: (date) => `Test Meeting ${date.toZonedDateTimeISO('UTC').toPlainDate().toString()}`,
+      issueTitle: ({ date }) => `Test Meeting ${date.toZonedDateTimeISO('UTC').toPlainDate().toString()}`,
+      createWithin: 'P7D',
+      agendaLabel: 'meeting-agenda',
       schedules: [
         // 1pm GMT April 16 repeating every 28 days
         '2020-04-16T13:00:00.0Z/P28D'
       ],
-      now: Temporal.Instant.from('2020-04-13T13:00:00.0Z')
+      now: Temporal.Instant.from('2020-04-13T13:00:00.0Z'),
+      meetingLabels: ['testMeeting', 'test']
     })
     assert.deepStrictEqual(issue.owner, 'wesleytodd')
     assert.deepStrictEqual(issue.repo, 'meeting-maker')
     assert.deepStrictEqual(issue.title, `Test Meeting ${Temporal.Instant.from('2020-04-16T13:00:00.0Z').toZonedDateTimeISO('UTC').toPlainDate().toString()}`)
     assert.deepStrictEqual(issue.agendaLabel, 'meeting-agenda')
-    assert.deepStrictEqual(issue.meetingLabels, ['testMeeting, test'])
+    assert.deepStrictEqual(issue.labels, ['testMeeting', 'test'])
     assert(typeof issue.body === 'string')
     assert(Array.isArray(issue.agendaIssues))
   })
@@ -40,6 +43,7 @@ suite(`${pkg.name} integration`, () => {
     const issue = await meetings.createNextMeeting(client, {
       owner: 'wesleytodd',
       repo: 'meeting-maker',
+      createWithin: 'P7D',
       schedules: [
         // 5pm GMT April 2 repeating every 28 days
         '2020-04-02T17:00:00.0Z/P28D',
@@ -48,8 +52,8 @@ suite(`${pkg.name} integration`, () => {
         '2020-04-16T13:00:00.0Z/P28D'
       ],
       now: Temporal.Instant.from('2020-04-13T13:00:00.0Z'),
-      issueTitle: (date) => `Test Meeting ${date.toZonedDateTimeISO('UTC').toPlainDate().toString()}`,
-      labels: ['testMeeting', 'test']
+      issueTitle: ({ date }) => `Test Meeting ${date.toZonedDateTimeISO('UTC').toPlainDate().toString()}`,
+      meetingLabels: ['testMeeting', 'test']
     })
 
     assert.deepStrictEqual(issue.data.title, `Test Meeting ${Temporal.Instant.from('2020-04-16T13:00:00.0Z').toZonedDateTimeISO('UTC').toPlainDate().toString()}`)