Applying linter to packages/core/src

Author: bigmontz

packages/core/src/auth.ts

@@ -29,7 +29,7 @@
  */
 const auth = {
   basic: (username: string, password: string, realm?: string) => {
-    if (realm) {
+    if (realm != null) {
       return {
         scheme: 'basic',
         principal: username,
@@ -77,12 +77,12 @@ const auth = {
   }
 }
 
-function isNotEmpty<T extends object | string>(value: T | null | undefined): boolean {
+function isNotEmpty<T extends object | string> (value: T | null | undefined): boolean {
   return !(
     value === null ||
     value === undefined ||
     value === '' ||
-    Object.getPrototypeOf(value) === Object.prototype && Object.keys(value).length === 0
+    (Object.getPrototypeOf(value) === Object.prototype && Object.keys(value).length === 0)
   )
 }
 

packages/core/src/connection-provider.ts

@@ -16,12 +16,12 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
+/* eslint-disable @typescript-eslint/promise-function-async */
 
 import Connection from './connection'
 import { bookmarks } from './internal'
 import { ServerInfo } from './result-summary'
 
-
 /**
  * Inteface define a common way to acquire a connection
  *
@@ -44,11 +44,11 @@ class ConnectionProvider {
    * @property {string} param.impersonatedUser - the impersonated user
    * @property {function (databaseName:string?)} param.onDatabaseNameResolved - Callback called when the database name get resolved
    */
-  acquireConnection(param?: {
+  acquireConnection (param?: {
     accessMode?: string
     database?: string
-    bookmarks: bookmarks.Bookmarks,
-    impersonatedUser?: string,
+    bookmarks: bookmarks.Bookmarks
+    impersonatedUser?: string
     onDatabaseNameResolved?: (databaseName?: string) => void
   }): Promise<Connection> {
     throw Error('Not implemented')
@@ -60,7 +60,7 @@ class ConnectionProvider {
    *
    * @returns {Promise<boolean>}
    */
-  supportsMultiDb(): Promise<boolean> {
+  supportsMultiDb (): Promise<boolean> {
     throw Error('Not implemented')
   }
 
@@ -70,7 +70,7 @@ class ConnectionProvider {
    *
    * @returns {Promise<boolean>}
    */
-  supportsTransactionConfig(): Promise<boolean> {
+  supportsTransactionConfig (): Promise<boolean> {
     throw Error('Not implemented')
   }
 
@@ -80,7 +80,7 @@ class ConnectionProvider {
    *
    * @returns {Promise<boolean>}
    */
-  supportsUserImpersonation(): Promise<boolean> {
+  supportsUserImpersonation (): Promise<boolean> {
     throw Error('Not implemented')
   }
 
@@ -94,7 +94,7 @@ class ConnectionProvider {
    *
    * @returns {Promise<ServerInfo>} promise resolved with server info or rejected with error.
    */
-  verifyConnectivityAndGetServerInfo(param? :{ database?: string, accessMode?: string }): Promise<ServerInfo> {
+  verifyConnectivityAndGetServerInfo (param?: { database?: string, accessMode?: string }): Promise<ServerInfo> {
     throw Error('Not implemented')
   }
 
@@ -103,7 +103,7 @@ class ConnectionProvider {
    *
    * @returns {Promise<void>}
    */
-  close(): Promise<void> {
+  close (): Promise<void> {
     throw Error('Not implemented')
   }
 }

packages/core/src/connection.ts

@@ -16,6 +16,7 @@
  * See the License for the specific language governing permissions and
  * limitations under the License.
  */
+/* eslint-disable @typescript-eslint/promise-function-async */
 
 import { ServerAddress } from './internal/server-address'
 
@@ -25,28 +26,28 @@ import { ServerAddress } from './internal/server-address'
  */
 class Connection {
   get id (): string {
-    return ""
+    return ''
   }
 
-  get databaseId(): string {
-    return ""
+  get databaseId (): string {
+    return ''
   }
 
-  get server(): any {
+  get server (): any {
     return {}
   }
 
   /**
    * @property {ServerAddress} the server address this connection is opened against
    */
-  get address(): ServerAddress | undefined {
+  get address (): ServerAddress | undefined {
     return undefined
   }
 
   /**
    * @property {ServerVersion} the version of the server this connection is connected to
    */
-  get version(): any {
+  get version (): any {
     return undefined
   }