Use target if provided

modules/node/src/channel/channel.service.ts

@@ -112,7 +112,7 @@ export class ChannelService {
     multisigAddress: string,
     assetId: string = AddressZero,
     rebalanceType: RebalanceType,
-    requestedTarget: BigNumber = Zero,
+    requestedTarget?: BigNumber,
   ): Promise<
     | {
         completed?: () => Promise<FreeBalanceResponse>;
@@ -184,12 +184,12 @@ export class ChannelService {
         );
       }
 
-      const targetToUse = maxBN([profileTarget, requestedTarget]);
-      const thresholdToUse = maxBN([collateralizeThreshold, requestedTarget]);
+      const targetToUse = requestedTarget ? requestedTarget : profileTarget;
+      const thresholdToUse = requestedTarget ? requestedTarget : collateralizeThreshold;
 
       if (nodeFreeBalance.lt(thresholdToUse)) {
         this.log.info(
-          `nodeFreeBalance ${nodeFreeBalance.toString()} < thresholdToUse ${thresholdToUse.toString()}, depositing to target ${requestedTarget.toString()}`,
+          `nodeFreeBalance ${nodeFreeBalance.toString()} < thresholdToUse ${thresholdToUse.toString()}, depositing to target ${targetToUse.toString()}`,
         );
         const amount = targetToUse.sub(nodeFreeBalance);
         rebalanceRes = (await this.depositService.deposit(channel, amount, normalizedAssetId))!;

modules/test-runner/src/collateral/request.test.ts

@@ -62,6 +62,19 @@ describe(name, () => {
     expect(freeBalance[nodeSignerAddress]).to.be.least(requestedTarget);
   });
 
+  it("should collateralize tokens with a target lower than profile", async () => {
+    const requestedTarget = utils.parseEther("10"); // requested < 20
+    const response = (await client.requestCollateral(tokenAddress, requestedTarget))!;
+    expect(response).to.be.ok;
+    expect(response.completed).to.be.ok;
+    expect(response.transaction).to.be.ok;
+    expect(response.transaction.hash).to.be.ok;
+    expect(response.depositAppIdentityHash).to.be.ok;
+    const { freeBalance } = await response.completed();
+    expect(freeBalance[client.signerAddress]).to.be.eq(Zero);
+    expect(freeBalance[nodeSignerAddress]).to.be.eq(requestedTarget);
+  });
+
   it("should properly handle concurrent collateral requests", async () => {
     const appDef = client.config.contractAddresses[client.chainId].DepositApp;
     let depositAppCount = 0;