Feature/1 adjust token expiration logic (#7)

* removed the else block that obtains a access token by using the refresh token #1

* set node version to 12, package-lock should be checked in, updated dependencies #1

* tweaked the jest config to outline test coverage #1

* bumped to the package version to a new release #1

Author: reggeenr

.gitignore

@@ -1,2 +1,2 @@
 node_modules
-package-lock.json
+coverage

.travis.yml

@@ -1,6 +1,6 @@
 language: node_js
 node_js:
-- '8'
+- '12'
 deploy:
   deploy:
   provider: npm

index.js

@@ -32,18 +32,12 @@ module.exports = class TokenManager {
    */
   getToken () {
     return new Promise((resolve, reject) => {
-      if (!this.tokenInfo.access_token || this.isRefreshTokenExpired()) {
-        // 1. request an initial token
+      if (!this.tokenInfo.access_token || this.isTokenExpired()) {
+        // 1. request an initial token or 2. refresh an expired token
         return this.requestToken().then(tokenResponse => {
           this.saveTokenInfo(tokenResponse)
           resolve(this.tokenInfo.access_token)
         }).catch(error => reject(error))
-      } else if (this.isTokenExpired()) {
-        // 2. refresh a token
-        return this.refreshToken().then(tokenResponse => {
-          this.saveTokenInfo(tokenResponse)
-          resolve(this.tokenInfo.access_token)
-        }).catch(error => reject(error))
       } else {
         // 3. use valid managed token
         resolve(this.tokenInfo.access_token)

jest.config.js

@@ -18,7 +18,7 @@ module.exports = {
   // clearMocks: false,
 
   // Indicates whether the coverage information should be collected while executing the test
-  // collectCoverage: false,
+  collectCoverage: false,
 
   // An array of glob patterns indicating a set of files for which coverage information should be collected
   // collectCoverageFrom: null,
@@ -27,23 +27,22 @@ module.exports = {
   coverageDirectory: 'coverage',
 
   // An array of regexp pattern strings used to skip coverage collection
-  // coveragePathIgnorePatterns: [
-  //   "/node_modules/"
-  // ],
+  coveragePathIgnorePatterns: [
+    '/node_modules/',
+    '/__tests__/'
+  ],
 
   // A list of reporter names that Jest uses when writing coverage reports
-  // coverageReporters: [
-  //   "json",
-  //   "text",
-  //   "lcov",
-  //   "clover"
-  // ],
+  coverageReporters: [
+    'json',
+    'text'
+  ],
 
   // An object that configures minimum threshold enforcement for coverage results
   // coverageThreshold: null,
 
   // Make calling deprecated APIs throw helpful error messages
-  // errorOnDeprecated: false,
+  errorOnDeprecated: true,
 
   // Force coverage collection from ignored files usin a array of glob patterns
   // forceCoverageMatch: [],
@@ -124,7 +123,7 @@ module.exports = {
   // snapshotSerializers: [],
 
   // The test environment that will be used for testing
-  testEnvironment: 'node'
+  testEnvironment: 'node',
 
   // Options that will be passed to the testEnvironment
   // testEnvironmentOptions: {},
@@ -170,7 +169,7 @@ module.exports = {
   // unmockedModulePathPatterns: undefined,
 
   // Indicates whether each individual test should be reported during the run
-  // verbose: null,
+  verbose: false
 
   // An array of regexp patterns that are matched against all source file paths before re-running tests in watch mode
   // watchPathIgnorePatterns: [],