fix: descriptions and unnecessary code in two procedures

PolymathNetwork · Dec 20, 2019 · 59ba9e5 · 59ba9e5
1 parent b19f267
commit 59ba9e5
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 7 deletions.
diff --git a/src/procedures/__tests__/InvestInSimpleSto.ts b/src/procedures/__tests__/InvestInSimpleSto.ts
@@ -24,7 +24,7 @@ import { Factories } from '../../Context';
 import { SimpleSto, SecurityToken } from '../../entities';
 import * as securityTokenFactoryModule from '../../entities/factories/SecurityTokenFactory';
 import { Wallet } from '../../Wallet';
-import { ApproveErc20 } from '~/procedures';
+import { ApproveErc20 } from '../../procedures';
 
 const simpleParams: InvestInSimpleStoProcedureArgs = {
   symbol: 'TEST1',
@@ -231,7 +231,7 @@ describe('InvestInSimpleSto', () => {
       );
     });
 
-    test('should throw an error if the simple sto has not been launched or is archived', async () => {
+    test('should throw an error if the simple sto has not yet been launched or is archived', async () => {
       moduleWrapperFactoryMock.mock('getModuleInstance', Promise.resolve(undefined));
 
       await expect(target.prepareTransactions()).rejects.toThrow(
@@ -272,7 +272,7 @@ describe('InvestInSimpleSto', () => {
       );
     });
 
-    test('should throw an error if the sto is already finalized', async () => {
+    test('should throw an error if the sto has already been finalized', async () => {
       simpleStoFactoryMock.mock('fetch', {
         ...simpleStoObject,
         isFinalized: true,
@@ -286,7 +286,7 @@ describe('InvestInSimpleSto', () => {
       );
     });
 
-    test('should throw an error if the start date is in the future', async () => {
+    test('should throw an error if the sto start date is in the future', async () => {
       simpleStoFactoryMock.mock('fetch', {
         ...simpleStoObject,
         startDate: new Date(2040, 0),
@@ -300,7 +300,7 @@ describe('InvestInSimpleSto', () => {
       );
     });
 
-    test('should throw an error if beneficial investments not allowed, and the parameters include a beneficiary address', async () => {
+    test('should throw an error if beneficial investments are not allowed and the parameters include a beneficiary address', async () => {
       target = new InvestInSimpleSto(
         { ...simpleParams, beneficiary: beneficiaryAddress },
         contextMock.getMockInstance()

diff --git a/src/procedures/__tests__/LaunchSimpleSto.ts b/src/procedures/__tests__/LaunchSimpleSto.ts
@@ -3,7 +3,6 @@ import { restore, spy } from 'sinon';
 import * as contractWrappersModule from '@polymathnetwork/contract-wrappers';
 import {
   BigNumber,
-  CappedSTOFundRaiseType as CappedStoCurrency,
   SecurityTokenEvents,
   TransactionReceiptWithDecodedLogs,
 } from '@polymathnetwork/contract-wrappers';
@@ -28,7 +27,6 @@ import { Wallet } from '../../Wallet';
 import { TransferErc20 } from '..';
 import { mockFactories } from '../../testUtils/mockFactories';
 import { SimpleSto, SecurityToken } from '../../entities';
-import { PostTransactionResolver } from '../../PostTransactionResolver';
 
 const params: LaunchSimpleStoProcedureArgs = {
   symbol: 'TEST1',