Test and Coverage ✨

Author: abhijeetps

index.js

@@ -2,8 +2,8 @@ const createScheduler = require('probot-scheduler')
 const getConfig = require('probot-config')
 const moment = require('moment')
 
-const createWeeklyDigestLabel = require('./src/markdown/createWeeklyDigestLabel')
-const createConfigYML = require('./src/markdown/createConfigYML')
+// const createWeeklyDigestLabel = require('./src/markdown/createWeeklyDigestLabel')
+const postCreateLabel = require('./src/bin/postCreateLabel')
 const weeklyDigest = require('./src/weeklyDigest')
 const getDate = require('./src/markdown/getDate')
 const defaultConfig = require('./src/markdown/defaultConfig')
@@ -13,40 +13,52 @@ const checkDuplicates = require('./src/markdown/checkDuplicates')
 
 // 12 hours
 const interval = 12 * 60 * 60 * 1000
+const events = ['installation.created', 'installation_repositories']
 
 module.exports = (app) => {
   app.log('Weekly Digest app is running successfully.')
-  app.on('installation.created', async (context) => {
+
+  app.on(events, async (context) => {
     app.log('App has been successfully installed.')
+    app.log('App is running as per ' + context.event)
     app.log('Local Time: ' + moment().format())
-    const [owner, repo] = await context.payload.repositories[0].full_name.split('/')
-    console.log(`Repository: ${owner}/${repo}`)
-    await createWeeklyDigestLabel(context, {owner, repo})
-    await createConfigYML(context, {owner, repo})
     const headDate = getDate.headDate()
     const tailDate = getDate.tailDate()
     const config = defaultConfig
-    weeklyDigest(context, {owner, repo, headDate, tailDate}, config)
+    let repoList = []
+    if (context.event === 'installation' && context.payload.action === 'created') {
+      repoList = context.payload.repositories
+      console.log(repoList)
+    } else if (context.event === 'installation_repositories') {
+      repoList = context.payload.repositories_added
+    }
+    repoList.forEach(async (item) => {
+      const [owner, repo] = item.full_name.split('/')
+      app.log(`Repository: ${owner}/${repo}`)
+      await postCreateLabel(context, {owner, repo, name: 'weekly-digest', color: '9C27B0', description: ''})
+      await weeklyDigest(context, {owner, repo, headDate, tailDate}, config)
+    })
   })
+
   createScheduler(app, {interval: interval})
   app.on('schedule.repository', async (context) => {
-    console.log(`App is running as per schedule.repository`)
+    app.log(`App is running as per schedule.repository`)
     app.log('Local Time: ' + moment().format())
     const { owner, repo } = context.repo()
-    console.log(`Repository: ${owner}/${repo}`)
+    app.log(`Repository: ${owner}/${repo}`)
     const headDate = getDate.headDate()
     const tailDate = getDate.tailDate()
     let { hasDuplicates, url } = await checkDuplicates(context, {owner, repo, headDate})
     if (hasDuplicates) {
       console.log(`Weekly Digest for this week has already been published for ${owner}/${repo}`)
       console.log(`URL: ` + url)
-    } else {
-      let config = await getConfig(context, 'weekly-digest.yml')
-      config = fixConfig(config)
-      console.log(`Publish Day: ${getNumDayFromLongDay(config.publishDay)}, Today: ${moment().day()}`)
-      if (moment().day() === getNumDayFromLongDay(config.publishDay)) {
-        weeklyDigest(context, {owner, repo, headDate, tailDate}, config)
-      }
+      return
+    }
+    let config = await getConfig(context, 'weekly-digest.yml')
+    config = fixConfig(config)
+    console.log(`Publish Day: ${getNumDayFromLongDay(config.publishDay)}, Today: ${moment.utc().day()}`)
+    if (moment.utc().day() === getNumDayFromLongDay(config.publishDay)) {
+      await weeklyDigest(context, {owner, repo, headDate, tailDate}, config)
     }
   })
 }

index.test.js

@@ -0,0 +1,37 @@
+const { Application } = require('probot')
+// Requiring our app implementation
+const myProbotApp = require('./')
+
+const installationCreatedPayload = require('./test/payload/issues.opened.json')
+
+describe('My Probot app', () => {
+  let app, github
+
+  beforeEach(() => {
+    app = new Application()
+    // Initialize the app based on the code from index.js
+    app.load(myProbotApp)
+    // This is an easy way to mock out the GitHub API
+    github = {
+      issues: {
+        createLabel: jest.fn().mockReturnValue(Promise.resolve({}))
+      }
+    }
+    // Passes the mocked out GitHub API into out app instance
+    app.auth = () => Promise.resolve(github)
+  })
+
+  test.skip('creates a comment when an issue is opened', async () => {
+    // Simulates delivery of an issues.opened webhook
+    await app.receive({
+      event: 'installation.created',
+      payload: installationCreatedPayload
+    })
+
+    // This test passes if the code in your index.js file calls `context.github.issues.createComment`
+    expect(github.issues.createLabel).toHaveBeenCalled()
+  })
+})
+
+// For more information about testing with Jest see:
+// https://facebook.github.io/jest/

package.json

@@ -1,40 +1,49 @@
-{
-  "name": "weekly-digest",
-  "version": "0.0.0-development",
-  "description": "",
-  "author": "abhijeetps <[email protected]>",
-  "license": "ISC",
-  "repository": "https://github.com/probot/weekly-digest.git",
-  "scripts": {
-    "dev": "nodemon --exec \"npm start\"",
-    "start": "probot run ./index.js",
-    "lint": "standard --fix",
-    "test": "standard && jest",
-    "test:watch": "jest --watch --notify --notifyMode=change --coverage",
-    "travis-deploy-once": "travis-deploy-once",
-    "semantic-release": "semantic-release"
-  },
-  "dependencies": {
-    "mockdate": "^2.0.2",
-    "moment": "^2.22.2",
-    "nock": "^9.4.1",
-    "probot": "^7.0.1",
-    "probot-config": "^0.1.0",
-    "probot-scheduler": "^1.2.0"
-  },
-  "devDependencies": {
-    "jest": "^23.2.0",
-    "nodemon": "^1.17.5",
-    "smee-client": "^1.0.2",
-    "standard": "^11.0.1",
-    "semantic-release": "^15.6.0"
-  },
-  "engines": {
-    "node": ">= 8.3.0"
-  },
-  "standard": {
-    "env": [
-      "jest"
-    ]
-  }
-}
+{
+  "name": "weekly-digest",
+  "version": "0.0.0-development",
+  "description": "",
+  "author": "abhijeetps <[email protected]>",
+  "license": "ISC",
+  "repository": "https://github.com/probot/weekly-digest.git",
+  "scripts": {
+    "dev": "nodemon --exec \"npm start\"",
+    "start": "probot run ./index.js",
+    "lint": "standard --fix",
+    "test": "standard && jest",
+    "test:watch": "jest --watch --notify --notifyMode=change --coverage",
+    "travis-deploy-once": "travis-deploy-once",
+    "semantic-release": "semantic-release"
+  },
+  "dependencies": {
+    "mockdate": "^2.0.2",
+    "moment": "^2.22.2",
+    "nock": "^9.4.4",
+    "probot": "^7.0.1",
+    "probot-config": "^0.2.0",
+    "probot-scheduler": "^1.2.0"
+  },
+  "devDependencies": {
+    "jest": "^23.4.2",
+    "nodemon": "^1.18.3",
+    "smee-client": "^1.0.2",
+    "standard": "^11.0.1",
+    "semantic-release": "^15.9.3",
+    "travis-deploy-once": "^5.0.1"
+  },
+  "engines": {
+    "node": ">= 8.3.0"
+  },
+  "standard": {
+    "env": [
+      "jest"
+    ]
+  },
+  "jest": {
+    "testEnvironment": "node",
+    "verbose": true,
+    "testURL": "http://localhost/",
+    "collectCoverageFrom": [
+      "src/**/*.js"
+    ]
+  }
+}

src/bin/getSearchIssues.js

@@ -1,12 +1,9 @@
 
 module.exports = async (context, {owner, repo, date, author, type}) => {
   console.log('In getSearchIssues.js...')
-  let searchIssues = await context.github.paginate(
-    context.github.search.issues({
-      q: `repo:${owner}/${repo} type:${type} author:${author} created:>=${date}`,
-      per_page: 100
-    }),
-    res => res.data
-  )
+  let searchIssues = await context.github.search.issues({
+    q: `repo:${owner}/${repo} type:${type} author:${author} created:>=${date}`,
+    per_page: 100
+  })
   return searchIssues
 }

src/bin/putCreateFile.js

@@ -7,7 +7,8 @@ module.exports = async (context, {owner, repo, headDate}) => {
   let type = 'issues'
   let date = getDate.getDayBeforeDate(headDate).substr(0, 19)
   let issues = await getSearchIssues(context, {owner, repo, date, author, type})
-  if (issues.total_count >= 1) {
+  let totalCount = issues.data.total_count
+  if (totalCount >= 1) {
     return {hasDuplicates: true, url: issues.data.items[0].html_url}
   } else {
     return {hasDuplicates: false, url: undefined}

src/markdown/checkDuplicates.js

@@ -0,0 +1,78 @@
+
+const markdownIssues = require('./markdownIssues')
+const markdownPullRequests = require('./markdownPullRequests')
+const markdownContributors = require('./markdownContributors')
+const markdownStargazers = require('./markdownStargazers')
+const markdownCommits = require('./markdownCommits')
+const markdownReleases = require('./markdownReleases')
+
+const getAllIssues = require('./../bin/getAllIssues')
+const getAllPullRequests = require('./.././bin/getAllPullRequests')
+const getStargazers = require('./../bin/getStargazers')
+const getCommits = require('./../bin/getCommits')
+const getReleases = require('./../bin/getReleases')
+
+module.exports = async (context, {owner, repo, headDate, tailDate}, config) => {
+  let body = `Here's the **Weekly Digest** for [*${owner}/${repo}*](https://github.com/${owner}/${repo}):\n`
+  let issuesString
+  let contributorsString
+  let pullRequestsString
+  let stargazersString
+  let commitsString
+  let releasesString
+  if (config.canPublishIssues) {
+    const issues = await getAllIssues(context, {owner, repo, tailDate})
+    issuesString = markdownIssues(issues, headDate, tailDate)
+  }
+  if (config.canPublishPullRequests) {
+    const pullRequests = await getAllPullRequests(context, {owner, repo})
+    pullRequestsString = markdownPullRequests(pullRequests, headDate, tailDate)
+  }
+  if (config.canPublishCommits || config.canPublishContributors) {
+    const commits = await getCommits(context, {owner, repo, tailDate})
+    if (config.canPublishCommits) {
+      commitsString = markdownCommits(commits, headDate, tailDate)
+    }
+    if (config.canPublishContributors) {
+      contributorsString = markdownContributors(commits, headDate, tailDate)
+    }
+  }
+  if (config.canPublishStargazers) {
+    const stargazers = await getStargazers(context, {owner, repo})
+    stargazersString = markdownStargazers(stargazers, headDate, tailDate)
+  }
+  if (config.canPublishReleases) {
+    const releases = await getReleases(context, {owner, repo})
+    releasesString = markdownReleases(releases, headDate, tailDate)
+  }
+  if (issuesString) {
+    body += `\n - - - \n`
+    body += issuesString
+  }
+  if (pullRequestsString) {
+    body += `\n - - - \n`
+    body += pullRequestsString
+  }
+  if (commitsString) {
+    body += `\n - - - \n`
+    body += commitsString
+  }
+  if (contributorsString) {
+    body += `\n - - - \n`
+    body += contributorsString
+  }
+  if (stargazersString) {
+    body += `\n - - - \n`
+    body += stargazersString
+  }
+  if (releasesString) {
+    body += `\n - - - \n`
+    body += releasesString
+  }
+  body += `\n - - - \n`
+  body += '\n'
+  body += `That's all for last week, please <kbd>:eyes: **Watch**</kbd> and <kbd>:star: **Star**</kbd> the repository [*${owner}/${repo}*](https://github.com/${owner}/${repo}) to receive next weekly updates. :smiley:\n\n`
+  body += `*You can also [view all Weekly Digests by clicking here](https://github.com/${owner}/${repo}/issues?q=is:open+is:issue+label:weekly-digest).* \n\n`
+  body += `> Your [**Weekly Digest**](https://github.com/apps/weekly-digest) bot. :calendar:\n`
+  return body
+}