fix(tests): make mocks for rest octokit async

Author: lowlighter

source/app/action/index.mjs

@@ -185,7 +185,7 @@ function quit(reason) {
     //Test token validity and requests count
     else if (!/^NOT_NEEDED$/.test(token)) {
       //Check rate limit
-      const {data} = await api.rest.request("GET /rate_limit", {}).catch(() => ({data:{resources:{}}}))
+      const {data} = await  api.rest.rateLimit.get().catch(() => ({data:{resources:{}}}))
       Object.assign(resources, data.resources)
       info("API requests (REST)", resources.core ? `${resources.core.remaining}/${resources.core.limit}` : "(unknown)")
       info("API requests (GraphQL)", resources.graphql ? `${resources.graphql.remaining}/${resources.graphql.limit}` : "(unknown)")
@@ -600,7 +600,7 @@ function quit(reason) {
       info.break()
       info.section("Consumed API requests")
       info("  * provided that no other app used your quota during execution", "")
-      const {data:current} = await rest.request("GET /rate_limit", {}).catch(() => ({data:{resources:{}}}))
+      const {data:current} = await rest.rateLimit.get().catch(() => ({data:{resources:{}}}))
       for (const type of ["core", "graphql", "search"]) {
         const used = resources[type].remaining - current.resources[type].remaining
         info({core:"REST API", graphql:"GraphQL API", search:"Search API"}[type], (Number.isFinite(used)&&(used >= 0)) ? used : "(unknown)")

tests/mocks/api/github/rest/activity/listEventsForAuthenticatedUser.mjs

@@ -1,5 +1,5 @@
 /**Mocked data */
-export default function({ faker }, target, that, [{ username: login, page, per_page }]) {
+export default async function({ faker }, target, that, [{ username: login, page, per_page }]) {
   console.debug("metrics/compute/mocks > mocking rest api result > rest.activity.listEventsForAuthenticatedUser")
   return ({
     status: 200,

tests/mocks/api/github/rest/activity/listRepoEvents.mjs

@@ -2,7 +2,7 @@
 import listEventsForAuthenticatedUser from "./listEventsForAuthenticatedUser.mjs"
 
 /**Mocked data */
-export default function({ faker }, target, that, [{ username: login, page, per_page }]) {
+export default async function({ faker }, target, that, [{ username: login, page, per_page }]) {
   console.debug("metrics/compute/mocks > mocking rest api result > rest.activity.listRepoEvents")
   return listEventsForAuthenticatedUser(...arguments)
 }

tests/mocks/api/github/rest/emojis/get.mjs

@@ -1,5 +1,5 @@
 /**Mocked data */
-export default function({ faker }, target, that) {
+export default async function({ faker }, target, that) {
   console.debug("metrics/compute/mocks > mocking rest api result > rest.emojis.get")
   return ({
     status: 200,

tests/mocks/api/github/rest/rateLimit/get.mjs

@@ -1,5 +1,5 @@
 /**Mocked data */
-export default function({ faker }, target, that, args) {
+export default async function({ faker }, target, that, args) {
   return ({
     status: 200,
     url: "https://api.github.com/rate_limit",

tests/mocks/api/github/rest/repos/getContributorsStats.mjs

@@ -1,5 +1,5 @@
 /**Mocked data */
-export default function({ faker }, target, that, [{ owner, repo }]) {
+export default async function({ faker }, target, that, [{ owner, repo }]) {
   console.debug("metrics/compute/mocks > mocking rest api result > rest.repos.getContributorsStats")
   return ({
     status: 200,

tests/mocks/api/github/rest/repos/getViews.mjs

@@ -1,5 +1,5 @@
 /**Mocked data */
-export default function({ faker }, target, that, [{ owner, repo }]) {
+export default async function({ faker }, target, that, [{ owner, repo }]) {
   console.debug("metrics/compute/mocks > mocking rest api result > rest.repos.getViews")
   const count = faker.datatype.number(10000) * 2
   const uniques = faker.datatype.number(count) * 2

tests/mocks/api/github/rest/repos/listCommits.mjs

@@ -1,5 +1,5 @@
 /**Mocked data */
-export default function({ faker }, target, that, [{ page, per_page, owner, repo }]) {
+export default async function({ faker }, target, that, [{ page, per_page, owner, repo }]) {
   console.debug("metrics/compute/mocks > mocking rest api result > rest.repos.listCommits")
   return ({
     status: 200,

tests/mocks/api/github/rest/repos/listContributors.mjs

@@ -1,5 +1,5 @@
 /**Mocked data */
-export default function({ faker }, target, that, [{ owner, repo }]) {
+export default async function({ faker }, target, that, [{ owner, repo }]) {
   console.debug("metrics/compute/mocks > mocking rest api result > rest.repos.listContributors")
   return ({
     status: 200,

tests/mocks/api/github/rest/request.mjs

@@ -1,5 +1,5 @@
 /**Mocked data */
-export default function({ faker }, target, that, args) {
+export default async function({ faker }, target, that, args) {
   //Arguments
   const [url] = args
   //Head request