[Blocked on #137, #277] Update Oppiabot dependencies

[vojtechjelinek]

Update the versions of all the oppiabot dependencies, to keep them recent. This might help with the timeouts issue #136

[BenHenning]

@vojtechjelinek can you add context on which dependencies need to be updated & why? It's a bit hard to act upon this issue as-is.

[MohdImran001]

@vojtechjelinek Can I work on it?

[MohdImran001]

Hey @vojtechjelinek, I ran npm update to update dependencies to thier latest minor releases Then to check if everything is working fine I ran npm run test. I m getting the following output. But it is not showing which tests are failing. How I can find out failing tests?

487 specs, 0 failures
Finished in 89.019 seconds
----------------------------------|---------|----------|---------|---------|------------------------
File                              | % Stmts | % Branch | % Funcs | % Lines | Uncovered Line #s
----------------------------------|---------|----------|---------|---------|------------------------
All files                         |     100 |    94.93 |     100 |     100 |
 oppiabot                         |     100 |    79.71 |     100 |     100 |
  constants.js                    |     100 |      100 |     100 |     100 |
  index.js                        |     100 |    79.71 |     100 |     100 | ...232-258,270,282-297
 oppiabot/actions/src             |     100 |      100 |     100 |     100 |
  dispatcher.js                   |     100 |      100 |     100 |     100 |
 oppiabot/actions/src/issues      |     100 |      100 |     100 |     100 |
  checkIssueLabels.js             |     100 |      100 |     100 |     100 |
 oppiabot/lib                     |     100 |    99.49 |     100 |     100 |
  apiForSheets.js                 |     100 |      100 |     100 |     100 |
  checkBranchPush.js              |     100 |      100 |     100 |     100 |
  checkCriticalPullRequest.js     |     100 |      100 |     100 |     100 |
  checkForNewCodeowner.js         |     100 |      100 |     100 |     100 |
  checkIssueAssignee.js           |     100 |      100 |     100 |     100 |
  checkMergeConflicts.js          |     100 |      100 |     100 |     100 |
  checkPullRequestBranch.js       |     100 |      100 |     100 |     100 |
  checkPullRequestJob.js          |     100 |      100 |     100 |     100 |
  checkPullRequestLabels.js       |     100 |      100 |     100 |     100 |
  checkPullRequestReview.js       |     100 |    97.06 |     100 |     100 | 242
  checkPullRequestTemplate.js     |     100 |      100 |     100 |     100 |
  checkWipDraftPR.js              |     100 |      100 |     100 |     100 |
  ciChecks.js                     |     100 |      100 |     100 |     100 |
  periodicChecks.js               |     100 |      100 |     100 |     100 |
  scheduler.js                    |     100 |      100 |     100 |     100 |
  staleBuildChecks.js             |     100 |      100 |     100 |     100 |
  utils.js                        |     100 |      100 |     100 |     100 |
 oppiabot/spec                    |     100 |      100 |     100 |     100 |
  apiForSheetsSpec.js             |     100 |      100 |     100 |     100 |
  checkBranchPushSpec.js          |     100 |      100 |     100 |     100 |
  checkCriticalPullRequestSpec.js |     100 |      100 |     100 |     100 |
  checkForNewCodeownerSpec.js     |     100 |      100 |     100 |     100 |
  checkIssueAssignedSpec.js       |     100 |      100 |     100 |     100 |
  checkIssueLabelsSpec.js         |     100 |      100 |     100 |     100 |
  checkMergeConflictSpec.js       |     100 |      100 |     100 |     100 |
  checkPullRequestBranchSpec.js   |     100 |      100 |     100 |     100 |
  checkPullRequestJobSpec.js      |     100 |      100 |     100 |     100 |
  checkPullRequestLabelsSpec.js   |     100 |      100 |     100 |     100 |
  checkPullRequestReviewSpec.js   |     100 |      100 |     100 |     100 |
  checkPullRequestTemplateSpec.js |     100 |      100 |     100 |     100 |
  checkWipDraftPRSpec.js          |     100 |      100 |     100 |     100 |
  ciChecksSpec.js                 |     100 |      100 |     100 |     100 |
  periodicChecksSpec.js           |     100 |      100 |     100 |     100 |
  staleBuildChecksSpec.js         |     100 |      100 |     100 |     100 |
  utilsSpec.js                    |     100 |      100 |     100 |     100 |
----------------------------------|---------|----------|---------|---------|------------------------

[@octokit/app] Deprecated. Use @octokit/app-auth instead. See https://github.com/octokit/app.js/#deprecated

Reverting .env
Tests failed.
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR! [email protected] test: `node ./spec/setup.js`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the [email protected] test script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR! A complete log of this run can be found in:
npm ERR!     /home/skull/.npm/_logs/2021-03-21T15_46_31_406Z-debug.log

[vojtechjelinek]

@jameesjohn Any help here?

[MohdImran001]

Hey @jameesjohn, please guide me about the above-mentioned error. I will really appreciate it.

[jameesjohn]

Hi @MohdImran001
I'm not sure why it fails, it probably has something to do with

oppiabot/spec/setup.js

Lines 145 to 151 in a2b19c5

	if (error) {
	console.log(error);
	reject();
	}
	if (stderr) {
	console.log(stderr);
	reject();

[mridul-netizen]

Can you assign me this issue?

[winnie368c]

@seanlip Can I be assigned to this?

[U8NWXD]

Moving from high priority to priority since it isn't currently blocking developers

[seanlip]

Note: @vojtechjelinek thinks this might not be possible until we fully move Oppiabot to update Probot, or move to GitHub Actions (see #137, #277) to remove the need for the Probot dependency.

[Shivkant-Chauhan]

marking this as TODO as blocked on 137 and 277