🐛 bug: fixed test bugs

.env.sample

@@ -1,5 +1,5 @@
 PUBLIC_KEY=
-SECRET_KEY=
+PRIVATE_KEY=
 ACCOUNT_NUMBER=
 ACCOUNT_BANK=
 ACCOUNT_NAME=

README.md

@@ -7,12 +7,12 @@
 ```bash
 const CoinForBarter = require('coinforbarter      -node-v3');
 
-const c4b = new CoinForBarter(PUBLIC_KEY, SECRET_KEY, SECRET_HASH);
+const c4b = new CoinForBarter(PUBLIC_KEY, PRIVATE_KEY, SECRET_HASH);
 
 const customers = c4b.Customers.findAll();
 ```
 
-For staging, Use TEST API Keys. For production, use LIVE API KEYS. You can get your PUBLIC_KEY and SECRET_KEY from the CoinForBarter dashboard.
+For staging, Use TEST API Keys. For production, use LIVE API KEYS. You can get your PUBLIC_KEY and PRIVATE_KEY from the CoinForBarter dashboard.
 
 Go [here](https://dashboard.coinforbarter.com/settings/api) to get your API Keys.
 

package.json

@@ -26,7 +26,7 @@
     "lint": "eslint \"{src,apps,libs,test}/**/*.ts\" --fix"
   },
   "engines": {
-    "node": "<=14.17.1"
+    "node": ">=14.1"
   },
   "keywords": [
     "bitcoin",

src/index.ts

@@ -28,10 +28,10 @@ export default class CoinForBarter {
   private readonly Request: CoinForBarterRequest;
   constructor(
     private readonly publicKey: string,
-    private readonly secretKey: string,
+    private readonly privateKey: string,
     private readonly secretHash: string = '',
   ) {
-    this.Request = new CoinForBarterRequest(this.publicKey, this.secretKey);
+    this.Request = new CoinForBarterRequest(this.publicKey, this.privateKey);
     this.Payment = new Payment(this.Request, this.publicKey);
     this.Payout = new Payout(this.Request);
     this.Transfer = new Transfer(this.Request);
@@ -41,6 +41,7 @@ export default class CoinForBarter {
     this.BankAccount = new BankAccount(this.Request);
     this.Customer = new Customer(this.Request);
     this.Misc = new Misc(this.Request);
+    this.Transaction = new Transaction(this.Request);
     this.Webhook = new Webhook(this.secretHash);
   }
 }

src/lib/BankAccount.ts

@@ -107,4 +107,19 @@ export class BankAccount {
       statusCode,
     };
   }
+
+  async remove(bankAccountId: string) {
+    const { data, status, message, statusCode } = await this.request.call(
+      `${this.path}/account/archive/${bankAccountId}`,
+      'get',
+      {},
+      true,
+    );
+    return {
+      data,
+      status,
+      message,
+      statusCode,
+    };
+  }
 }

src/lib/Misc.ts

@@ -18,7 +18,7 @@ export class Misc {
     return { status, message, statusCode, data };
   }
 
-  async getBalance(): Promise<CoinForBarterResponse<BalanceResponse[]>> {
+  async getBalances(): Promise<CoinForBarterResponse<BalanceResponse[]>> {
     const { status, message, statusCode, data } = await this.request.call(
       '/balances',
       'get',

src/lib/Wallet.ts

@@ -0,0 +1,79 @@
+import { CoinForBarterRequest } from 'src/services';
+import {
+  CoinForBarterResponse,
+  CreateWalletBody,
+  CreateWalletResponse,
+  WalletQuery,
+} from 'src/types';
+
+export class Wallet {
+  path = '/wallets';
+  constructor(private readonly request: CoinForBarterRequest) {}
+
+  async create(
+    body: CreateWalletBody,
+  ): Promise<CoinForBarterResponse<CreateWalletResponse>> {
+    const { status, message, statusCode, data } = await this.request.call(
+      this.path,
+      'post',
+      body,
+      true,
+    );
+    return {
+      status,
+      message,
+      statusCode,
+      data,
+    };
+  }
+
+  async findAll(
+    query: WalletQuery = {},
+  ): Promise<CoinForBarterResponse<CreateWalletResponse[]>> {
+    const queryString = this.request.makeQueryString(query);
+    const { status, message, statusCode, data } = await this.request.call(
+      `${this.path}${queryString}`,
+      'get',
+      {},
+      true,
+    );
+    return {
+      status,
+      message,
+      statusCode,
+      data,
+    };
+  }
+
+  async findOne(
+    id: string,
+  ): Promise<CoinForBarterResponse<CreateWalletResponse>> {
+    const { status, message, statusCode, data } = await this.request.call(
+      `${this.path}/${id}`,
+      'get',
+      {},
+      true,
+    );
+    return {
+      status,
+      message,
+      statusCode,
+      data,
+    };
+  }
+
+  async remove(id: string): Promise<CoinForBarterResponse<null>> {
+    const { status, message, statusCode, data } = await this.request.call(
+      `${this.path}/${id}`,
+      'patch',
+      undefined,
+      true,
+    );
+    return {
+      status,
+      message,
+      statusCode,
+      data,
+    };
+  }
+}

src/services/Request.ts

@@ -3,10 +3,10 @@ import { CoinForBarterStatus } from '../types';
 import { RequestResponseSchema } from '../types/request';
 
 export class CoinForBarterRequest {
-  private url = 'https://api.coinforbarter.com/v1';
+  private url = 'https://coinforbarter-api.herokuapp.com/v1';
   constructor(
     private readonly publicKey: string,
-    private readonly secretKey: string,
+    private readonly privateKey: string,
   ) {}
 
   async call(
@@ -22,7 +22,7 @@ export class CoinForBarterRequest {
       if (method === 'get' || method === 'delete') {
         if (useToken) {
           const headers = {
-            Authorization: `Bearer ${this.secretKey}`,
+            Authorization: `Bearer ${this.privateKey}`,
             'Content-Type': contentType,
           };
           request = await axios[method](url, { headers });
@@ -37,7 +37,7 @@ export class CoinForBarterRequest {
       if (method !== 'get' && method !== 'delete') {
         if (useToken) {
           const headers = {
-            Authorization: `Bearer ${this.secretKey}`,
+            Authorization: `Bearer ${this.privateKey}`,
             'Content-Type': contentType,
           };
           request = await axios[method](url, body, { headers });

src/test/bank-account.spec.ts

@@ -9,12 +9,14 @@ import {
 
 describe('BankAccount', () => {
   const publicKey = process.env.PUBLIC_KEY;
-  const secretKey = process.env.SECRET_KEY;
+  const privateKey = process.env.PRIVATE_KEY;
 
-  const request = new CoinForBarterRequest(publicKey, secretKey);
+  const request = new CoinForBarterRequest(publicKey, privateKey);
 
   const bankAccount = new BankAccount(request);
 
+  let bankAccountId = '';
+
   it('should be defined', () => {
     expect(bankAccount).toBeDefined();
   });
@@ -85,6 +87,7 @@ describe('BankAccount', () => {
       const respWithParam = await bankAccount.findAll(param);
       const resp = await bankAccount.findAll();
       if (resp.status === CoinForBarterStatus.Success) {
+        bankAccountId = resp.data[0].id;
         expect(resp.data.length).toBeGreaterThan(0);
       } else {
         expect(resp.statusCode).toEqual(404);
@@ -103,10 +106,9 @@ describe('BankAccount', () => {
     });
 
     it('should find one bank account from id', async () => {
-      const id = process.env.BANK_ACCOUNT_ID;
-      const resp = await bankAccount.findOne(id);
+      const resp = await bankAccount.findOne(bankAccountId);
       if (resp.status === CoinForBarterStatus.Success) {
-        return expect(resp.data.id).toBe(id);
+        return expect(resp.data.id).toBe(bankAccountId);
       } else {
         expect(resp.statusCode).not.toEqual(200);
       }
@@ -145,4 +147,19 @@ describe('BankAccount', () => {
       }
     });
   });
+
+  describe('remove', () => {
+    it('should be defined', () => {
+      expect(bankAccount.remove).toBeDefined();
+    });
+
+    it('should make an account primary', async () => {
+      const resp = await bankAccount.remove(bankAccountId);
+      if (resp.status === CoinForBarterStatus.Success) {
+        return expect(resp.statusCode).toEqual(204);
+      } else {
+        return expect(resp.statusCode).not.toEqual(200);
+      }
+    });
+  });
 });

src/test/customer.spec.ts

@@ -5,12 +5,12 @@ import { CoinForBarterRequest } from '../services';
 
 describe('customer', () => {
   const publicKey = process.env.PUBLIC_KEY;
-  const secretKey = process.env.SECRET_KEY;
+  const privateKey = process.env.PRIVATE_KEY;
 
   const email = process.env.EMAIL;
   const phoneNumber = process.env.PHONE_NUMBER;
   const fullName = process.env.EMAIL;
-  const request = new CoinForBarterRequest(publicKey, secretKey);
+  const request = new CoinForBarterRequest(publicKey, privateKey);
 
   const customer = new Customer(request);
 

src/test/misc.spec.ts

@@ -4,9 +4,9 @@ import { CoinForBarterRequest } from '../services/Request';
 
 describe('misc', () => {
   const publicKey = process.env.PUBLIC_KEY;
-  const secretKey = process.env.SECRET_KEY;
+  const privateKey = process.env.PRIVATE_KEY;
 
-  const request = new CoinForBarterRequest(publicKey, secretKey);
+  const request = new CoinForBarterRequest(publicKey, privateKey);
 
   const misc = new Misc(request);
 
@@ -30,22 +30,6 @@ describe('misc', () => {
     });
   });
 
-  describe('getBalance', () => {
-    it('should be defined', () => {
-      expect(misc.getBalance).toBeDefined();
-    });
-
-    it('should find all countries', async () => {
-      const resp = await misc.getBalance();
-
-      if (resp.status === CoinForBarterStatus.Success) {
-        expect(resp.data.length).toBeGreaterThan(0);
-      } else {
-        expect(resp.statusCode).toEqual(404);
-      }
-    });
-  });
-
   describe('getCurrencies', () => {
     it('should be defined', () => {
       expect(misc.getCurrencies).toBeDefined();

src/test/payment-plan-subscribers.spec.ts

@@ -8,13 +8,13 @@ import { CoinForBarterRequest } from '../services/Request';
 
 describe('paymentPlanSubscriber', () => {
   const publicKey = process.env.PUBLIC_KEY;
-  const secretKey = process.env.SECRET_KEY;
+  const privateKey = process.env.PRIVATE_KEY;
 
   const email = process.env.EMAIL;
   const phoneNumber = process.env.PHONE_NUMBER;
   const fullName = process.env.EMAIL;
   const paymentPlanId = process.env.PAYMENT_PLAN_ID;
-  const request = new CoinForBarterRequest(publicKey, secretKey);
+  const request = new CoinForBarterRequest(publicKey, privateKey);
 
   const paymentPlanSubscriber = new PaymentPlanSubscriber(request);
 

src/test/payment-plan.spec.ts

@@ -10,9 +10,9 @@ import { CoinForBarterStatus } from '../types/response.types';
 
 describe('paymentPlan', () => {
   const publicKey = process.env.PUBLIC_KEY;
-  const secretKey = process.env.SECRET_KEY;
+  const privateKey = process.env.PRIVATE_KEY;
 
-  const request = new CoinForBarterRequest(publicKey, secretKey);
+  const request = new CoinForBarterRequest(publicKey, privateKey);
 
   const paymentPlan = new PaymentPlan(request);
 

src/test/payment.spec.ts

@@ -5,9 +5,9 @@ import { CoinForBarterStatus } from '../types/response.types';
 
 describe('Payment', () => {
   const publicKey = process.env.PUBLIC_KEY;
-  const secretKey = process.env.SECRET_KEY;
+  const privateKey = process.env.PRIVATE_KEY;
 
-  const request = new CoinForBarterRequest(publicKey, secretKey);
+  const request = new CoinForBarterRequest(publicKey, privateKey);
   const id = process.env.PAYMENT_ID;
 
   const currency = 'DOGE';

src/test/payout.spec.ts

@@ -5,9 +5,9 @@ import { CoinForBarterStatus } from '../types/response.types';
 
 describe('payout', () => {
   const publicKey = process.env.PUBLIC_KEY;
-  const secretKey = process.env.SECRET_KEY;
+  const privateKey = process.env.PRIVATE_KEY;
 
-  const request = new CoinForBarterRequest(publicKey, secretKey);
+  const request = new CoinForBarterRequest(publicKey, privateKey);
 
   const payout = new Payout(request);
 

src/test/transaction.spec.ts

@@ -4,9 +4,9 @@ import { TransactionQuery } from '../types/transaction.type';
 import { CoinForBarterStatus } from '../types/response.types';
 describe('Transaction', () => {
   const publicKey = process.env.PUBLIC_KEY;
-  const secretKey = process.env.SECRET_KEY;
+  const privateKey = process.env.PRIVATE_KEY;
 
-  const request = new CoinForBarterRequest(publicKey, secretKey);
+  const request = new CoinForBarterRequest(publicKey, privateKey);
 
   const transaction = new Transaction(request);
 

src/test/transfer.spec.ts

@@ -8,12 +8,12 @@ import { CoinForBarterStatus } from '../types/response.types';
 
 describe('Transfer', () => {
   const publicKey = process.env.PUBLIC_KEY;
-  const secretKey = process.env.SECRET_KEY;
+  const privateKey = process.env.PRIVATE_KEY;
   const accountNumber = process.env.ACCOUNT_NUMBER;
   const walletAddress = process.env.WALLET_ADDRESS;
   const walletNetwork = process.env.WALLET_NETWORK;
 
-  const request = new CoinForBarterRequest(publicKey, secretKey);
+  const request = new CoinForBarterRequest(publicKey, privateKey);
 
   const transfer = new Transfer(request);
 

src/test/wallet-address.spec.ts

@@ -8,9 +8,9 @@ import { CoinForBarterStatus } from '../types/response.types';
 
 describe('WalletAddress', () => {
   const publicKey = process.env.PUBLIC_KEY;
-  const secretKey = process.env.SECRET_KEY;
+  const privateKey = process.env.PRIVATE_KEY;
 
-  const request = new CoinForBarterRequest(publicKey, secretKey);
+  const request = new CoinForBarterRequest(publicKey, privateKey);
 
   const walletAddress = new WalletAddress(request);