Fix file newlines & TS error, normalize permissions comments naming

Author: cgewecke

contracts/lib/AddressArrayUtils.sol

@@ -219,4 +219,4 @@ library AddressArrayUtils {
         require(A.length > 0, "Array length must be > 0");
         require(!hasDuplicate(A), "Cannot duplicate addresses");
     }
-}
+}

contracts/lib/PreciseUnitMath.sol

@@ -194,4 +194,4 @@ library PreciseUnitMath {
     function approximatelyEquals(uint256 a, uint256 b, uint256 range) internal pure returns (bool) {
         return a <= b.add(range) && a >= b.sub(range);
     }
-}
+}

contracts/mocks/AddressArrayUtilsMock.sol

@@ -78,4 +78,4 @@ contract AddressArrayUtilsMock {
     function getStorageArray() external view returns(address[] memory) {
         return storageArray;
     }
-}
+}

contracts/mocks/PreciseUnitMathMock.sol

@@ -85,4 +85,4 @@ contract PreciseUnitMathMock {
     function approximatelyEquals(uint256 a, uint256 b, uint256 range) external pure returns (bool) {
         return a.approximatelyEquals(b, range);
     }
-}
+}

contracts/protocol/modules/GeneralIndexModule.sol

@@ -78,7 +78,7 @@ contract GeneralIndexModule is ModuleBase, ReentrancyGuard {
 
     struct TradePermissionInfo {
         bool anyoneTrade;                               // Boolean indicating if anyone can execute a trade
-        address[] tradersHistory;                       // Tracks whitelisted traders to be deleted on module removal
+        address[] tradersHistory;                       // Tracks permissioned traders to be deleted on module removal
         mapping(address => bool) tradeAllowList;        // Mapping indicating which addresses are allowed to execute trade
     }
 
@@ -1043,7 +1043,7 @@ contract GeneralIndexModule is ModuleBase, ReentrancyGuard {
      */
 
     /*
-     * Trader must be whitelisted for SetToken
+     * Trader must be permissioned for SetToken
      */
     function _validateOnlyAllowedTrader(ISetToken _setToken) internal view {
         require(_isAllowedTrader(_setToken, msg.sender), "Address not permitted to trade");

test/lib/preciseUnitMath.spec.ts

@@ -474,7 +474,7 @@ describe("PreciseUnitMath", () => {
       subjectRange = ether(2);
     });
 
-    async function subject(): Promise<[BigNumber]> {
+    async function subject(): Promise<boolean> {
       return mathMock.approximatelyEquals(subjectA, subjectB, subjectRange);
     }