feat: additional swaps-related logging

Author: sangaman

Diff for: lib/p2p/Pool.ts

@@ -2,7 +2,7 @@ import assert from 'assert';
 import { EventEmitter } from 'events';
 import net, { Server, Socket } from 'net';
 import semver from 'semver';
-import { DisconnectionReason, ReputationEvent, XuNetwork } from '../constants/enums';
+import { DisconnectionReason, ReputationEvent, XuNetwork, SwapFailureReason } from '../constants/enums';
 import { Models } from '../db/DB';
 import { ReputationEventInstance } from '../db/types';
 import Logger from '../Logger';
@@ -825,7 +825,7 @@ class Pool extends EventEmitter {
       }
       case PacketType.SwapFailed: {
         const { body } = (packet as packets.SwapFailedPacket);
-        this.logger.debug(`received swapFailed due to ${body?.failureReason} from ${peer.label}: ${JSON.stringify(body)}`);
+        this.logger.debug(`received swapFailed due to ${SwapFailureReason[body!.failureReason]} from ${peer.label}: ${JSON.stringify(body)}`);
         this.emit('packet.swapFailed', packet);
         break;
       }

Diff for: lib/swaps/Swaps.ts

@@ -1209,12 +1209,13 @@ class Swaps extends EventEmitter {
           deal.role === SwapRole.Maker && deal.phase === SwapPhase.SwapAccepted,
             'SendingPayment can only be set after SwapRequested (taker) or SwapAccepted (maker)');
         deal.executeTime = Date.now();
+        this.logger.debug(`Setting SendingPayment phase for deal ${deal.rHash}`);
         break;
       case SwapPhase.PaymentReceived:
         assert(deal.phase === SwapPhase.SendingPayment, 'PaymentReceived can be only be set after SendingPayment');
         deal.completeTime = Date.now();
         deal.state = SwapState.Completed;
-        this.logger.debug(`Payment received for deal with hash ${deal.rHash} - preimage is ${deal.rPreimage}`);
+        this.logger.debug(`Setting PaymentReceived phase for deal ${deal.rHash} - preimage is ${deal.rPreimage}`);
         break;
       default:
         assert.fail('unknown deal phase');