probing 3.0.1 → 4.0.0

package/CHANGELOG.md

@@ -1,5 +1,9 @@
 # Versions
 
+## Version 4.0.0
+
+- Node.js 16 or higher is now required
+
 ## Version 3.0.1
 
 - Node.js 14 or higher is now required

package/package.json

@@ -8,16 +8,13 @@
   },
   "dependencies": {
     "async": "3.2.4",
-    "asyncjs-util": "1.2.11",
-    "bolt07": "1.8.3",
-    "ln-service": "54.9.0"
+    "asyncjs-util": "1.2.12",
+    "bolt07": "1.8.4",
+    "ln-service": "56.7.1"
   },
   "description": "Lightning Network probing utilities",
-  "devDependencies": {
-    "@alexbosworth/tap": "15.0.12"
-  },
   "engines": {
-    "node": ">=14"
+    "node": ">=16"
   },
   "keywords": [
     "lightning",
@@ -32,7 +29,7 @@
     "url": "https://github.com/alexbosworth/probing.git"
   },
   "scripts": {
-    "test": "tap --branches=1 --functions=1 --lines=1 --statements=1 test/graph/*.js test/liquidity/*.js test/payments/*.js test/routing/*.js"
+    "test": "npx nyc@15.1 node --experimental-test-coverage --test"
   },
-  "version": "3.0.1"
+  "version": "4.0.0"
 }

package/test/graph/test_get_policies_for_channels.js

@@ -1,4 +1,6 @@
-const {test} = require('@alexbosworth/tap');
+const {rejects} = require('node:assert').strict;
+const strictSame = require('node:assert').strict.deepStrictEqual;
+const test = require('node:test');
 
 const {channels} = require('./../fixtures/hop_hints');
 const {getPoliciesForChannels} = require('./../../graph');
@@ -182,7 +184,7 @@ const tests = [
 ];
 
 tests.forEach(({args, description, error, expected}) => {
-  return test(description, async ({end, rejects, strictSame}) => {
+  return test(description, async () => {
     if (!!error) {
       await rejects(getPoliciesForChannels(args), error, 'Got expected error');
     } else {
@@ -191,6 +193,6 @@ tests.forEach(({args, description, error, expected}) => {
       strictSame(channels, expected.channels, 'Got expected channels');
     }
 
-    return end();
+    return;
   });
 });

package/test/graph/test_max_htlc_across_route.js

@@ -1,4 +1,6 @@
-const {test} = require('@alexbosworth/tap');
+const strictSame = require('node:assert').strict.deepStrictEqual;
+const test = require('node:test');
+const {throws} = require('node:assert').strict;
 
 const {maxHtlcAcrossRoute} = require('./../../graph');
 
@@ -73,7 +75,7 @@ const tests = [
 ];
 
 tests.forEach(({args, description, error, expected}) => {
-  return test(description, async ({end, strictSame, throws}) => {
+  return test(description, (t, end) => {
     if (!!error) {
       throws(() => maxHtlcAcrossRoute(args), error, 'Got expected error');
     } else {

package/test/liquidity/test_find_max_payable.js

@@ -1,4 +1,6 @@
-const {test} = require('@alexbosworth/tap');
+const {rejects} = require('node:assert').strict;
+const strictSame = require('node:assert').strict.deepStrictEqual;
+const test = require('node:test');
 
 const findMaxPayable = require('./../../liquidity/find_max_payable');
 const {getInfoResponse} = require('./../fixtures');
@@ -208,19 +210,19 @@ const tests = [
 ];
 
 tests.forEach(({args, description, error, expected}) => {
-  return test(description, async ({end, equal, rejects}) => {
+  return test(description, async () => {
     if (!!error) {
       await rejects(findMaxPayable(args), error, 'Got expected error');
     } else if (expected.maximum) {
       const {maximum} = await findMaxPayable(args);
 
-      equal(maximum > expected.maximum - 100000, true, 'Got expected maximum');
+      strictSame(maximum > expected.maximum - 100000, true, 'Got maximum');
     } else {
       const {maximum} = await findMaxPayable(args);
 
-      equal(maximum, Number(), 'Max payable is zero');
+      strictSame(maximum, Number(), 'Max payable is zero');
     }
 
-    return end();
+    return;
   });
 });

package/test/liquidity/test_get_synthetic_out_ignores.js

@@ -1,4 +1,6 @@
-const {test} = require('@alexbosworth/tap');
+const {rejects} = require('node:assert').strict;
+const strictSame = require('node:assert').strict.deepStrictEqual;
+const test = require('node:test');
 
 const {getInfoResponse} = require('./../fixtures');
 const {getSyntheticOutIgnores} = require('./../../');
@@ -100,7 +102,7 @@ const tests = [
 ];
 
 tests.forEach(({args, description, error, expected}) => {
-  return test(description, async ({end, rejects, strictSame}) => {
+  return test(description, async () => {
     if (!!error) {
       await rejects(getSyntheticOutIgnores(args), error, 'Got expected error');
     } else {
@@ -109,6 +111,6 @@ tests.forEach(({args, description, error, expected}) => {
       strictSame(ignore, expected.ignore, 'Got expected ignores');
     }
 
-    return end();
+    return;
   });
 });

package/test/liquidity/test_hops_for_find_max_path.js

@@ -1,4 +1,6 @@
-const {test} = require('@alexbosworth/tap');
+const strictSame = require('node:assert').strict.deepStrictEqual;
+const test = require('node:test');
+const {throws} = require('node:assert').strict;
 
 const hopsForFindMaxPath = require('./../../liquidity/hops_for_find_max_path');
 
@@ -94,7 +96,7 @@ const tests = [
 ];
 
 tests.forEach(({args, description, error, expected}) => {
-  return test(description, async ({end, strictSame, throws}) => {
+  return test(description, (t, end) => {
     if (!!error) {
       throws(() => hopsForFindMaxPath(args), new Error(error), 'Got error');
     } else {

package/test/liquidity/test_multi_probe_ignores.js

@@ -1,4 +1,6 @@
-const {test} = require('@alexbosworth/tap');
+const strictSame = require('node:assert').strict.deepStrictEqual;
+const test = require('node:test');
+const {throws} = require('node:assert').strict;
 
 const multiProbeIgnores = require('./../../liquidity/multi_probe_ignores');
 
@@ -265,7 +267,7 @@ const tests = [
 ];
 
 tests.forEach(({args, description, error, expected}) => {
-  return test(description, async ({end, strictSame, throws}) => {
+  return test(description, (t, end) => {
     if (!!error) {
       throws(() => multiProbeIgnores(args), new Error(error), 'Got error');
     } else {

package/test/liquidity/test_subscribe_to_find_max_payable.js

@@ -1,7 +1,8 @@
 const {once} = require('events');
 const {promisify} = require('util');
-
-const {test} = require('@alexbosworth/tap');
+const strictSame = require('node:assert').strict.deepStrictEqual;
+const test = require('node:test');
+const {throws} = require('node:assert').strict;
 
 const {getChanInfoResponse} = require('./../fixtures');
 const {getInfoResponse} = require('./../fixtures');
@@ -77,7 +78,7 @@ const tests = [
 ];
 
 tests.forEach(({args, description, error, expected}) => {
-  return test(description, async ({end, strictSame, throws}) => {
+  return test(description, async () => {
     if (!!error) {
       throws(() => subscribeToFindMaxPayable(args), error, 'Got error');
     } else {
@@ -102,6 +103,6 @@ tests.forEach(({args, description, error, expected}) => {
       strictSame(events, expected.events, 'Got expected events');
     }
 
-    return end();
+    return;
   });
 });

package/test/liquidity/test_subscribe_to_find_path.js

@@ -1,7 +1,8 @@
 const {once} = require('events');
 const {promisify} = require('util');
-
-const {test} = require('@alexbosworth/tap');
+const strictSame = require('node:assert').strict.deepStrictEqual;
+const test = require('node:test');
+const {throws} = require('node:assert').strict;
 
 const {getChanInfoResponse} = require('./../fixtures');
 const {getInfoResponse} = require('./../fixtures');
@@ -451,7 +452,7 @@ const tests = [
 ];
 
 tests.forEach(({args, description, error, expected}) => {
-  return test(description, async ({end, strictSame, throws}) => {
+  return test(description, async () => {
     if (!!error) {
       throws(() => method(args), error, 'Got error');
     } else {
@@ -478,7 +479,5 @@ tests.forEach(({args, description, error, expected}) => {
 
       strictSame(events, expected.events, 'Got expected events');
     }
-
-    return end();
   });
 });

package/test/liquidity/test_subscribe_to_multi_path_probe.js

@@ -1,7 +1,8 @@
 const {once} = require('events');
 const {promisify} = require('util');
-
-const {test} = require('@alexbosworth/tap');
+const strictSame = require('node:assert').strict.deepStrictEqual;
+const test = require('node:test');
+const {throws} = require('node:assert').strict;
 
 const {getChanInfoResponse} = require('./../fixtures');
 const {getInfoResponse} = require('./../fixtures');
@@ -325,7 +326,7 @@ const tests = [
 ];
 
 tests.forEach(({args, description, error, expected}) => {
-  return test(description, async ({end, strictSame, throws}) => {
+  return test(description, async () => {
     if (!!error) {
       throws(() => subscribeToMultiPathProbe(args), error, 'Got error');
     } else {
@@ -353,6 +354,6 @@ tests.forEach(({args, description, error, expected}) => {
       strictSame(events, expected.events, 'Got expected events');
     }
 
-    return end();
+    return;
   });
 });

package/test/payments/test_get_route_for_payment.js

@@ -1,4 +1,6 @@
-const {test} = require('@alexbosworth/tap');
+const {rejects} = require('node:assert').strict;
+const strictSame = require('node:assert').strict.deepStrictEqual;
+const test = require('node:test');
 
 const {getChanInfoResponse} = require('./../fixtures');
 const {getInfoResponse} = require('./../fixtures');
@@ -158,7 +160,7 @@ const tests = [
 ];
 
 tests.forEach(({args, description, error, expected}) => {
-  return test(description, async ({end, rejects, strictSame}) => {
+  return test(description, async () => {
     if (!!error) {
       await rejects(getRouteForPayment(args), error, 'Got expected error');
     } else {
@@ -167,6 +169,6 @@ tests.forEach(({args, description, error, expected}) => {
       strictSame(route, expected, 'Got expected route');
     }
 
-    return end();
+    return;
   });
 });

package/test/payments/test_mtokens_for_multi_path_payment.js

@@ -1,4 +1,6 @@
-const {test} = require('@alexbosworth/tap');
+const strictSame = require('node:assert').strict.deepStrictEqual;
+const test = require('node:test');
+const {throws} = require('node:assert').strict;
 
 const method = require('./../../payments/mtokens_for_multi_path_payment');
 
@@ -59,13 +61,13 @@ const tests = [
 ];
 
 tests.forEach(({args, description, error, expected}) => {
-  return test(description, async ({end, strictSame, throws}) => {
+  return test(description, (t, end) => {
     if (!!error) {
       throws(() => method(args), error, 'Got expected error');
     } else {
       const {sorted} = method(args);
 
-      strictSame(sorted, expected.sorted);
+      strictSame(sorted, expected.sorted, 'Got expected result');
     }
 
     return end();

package/test/payments/test_sort_multiple_payment_paths.js

@@ -1,4 +1,6 @@
-const {test} = require('@alexbosworth/tap');
+const strictSame = require('node:assert').strict.deepStrictEqual;
+const test = require('node:test');
+const {throws} = require('node:assert').strict;
 
 const method = require('./../../payments/sort_multiple_payment_paths');
 
@@ -75,7 +77,7 @@ const tests = [
 ];
 
 tests.forEach(({args, description, error, expected}) => {
-  return test(description, async ({end, strictSame, throws}) => {
+  return test(description, (t, end) => {
     if (!!error) {
       throws(() => method(args), error, 'Got expected error');
     } else {

package/test/payments/test_subscribe_to_multi_path_pay.js

@@ -1,7 +1,8 @@
 const {once} = require('events');
 const {promisify} = require('util');
-
-const {test} = require('@alexbosworth/tap');
+const strictSame = require('node:assert').strict.deepStrictEqual;
+const test = require('node:test');
+const {throws} = require('node:assert').strict;
 
 const {getChanInfoResponse} = require('./../fixtures');
 const {getInfoResponse} = require('./../fixtures');
@@ -895,7 +896,7 @@ const tests = [
 ];
 
 tests.forEach(({args, description, error, expected}) => {
-  return test(description, async ({end, strictSame, throws}) => {
+  return test(description, async () => {
     if (!!error) {
       throws(() => subscribeToMultiPathPay(args), error, 'Got error');
     } else {
@@ -916,7 +917,7 @@ tests.forEach(({args, description, error, expected}) => {
 
       await nextTick();
 
-      await delay(50)
+      await delay(50);
 
       // Make sure that no listener to error doesn't cause an issue
       const sub2 = subscribeToMultiPathPay(args);
@@ -926,6 +927,6 @@ tests.forEach(({args, description, error, expected}) => {
       strictSame(events, expected.events, 'Got expected events');
     }
 
-    return end();
+    return;
   });
 });

package/test/routing/test_channels_from_hints.js

@@ -1,4 +1,6 @@
-const {test} = require('@alexbosworth/tap');
+const strictSame = require('node:assert').strict.deepStrictEqual;
+const test = require('node:test');
+const {throws} = require('node:assert').strict;
 
 const {channels} = require('./../fixtures/hop_hints');
 const {channelsFromHints} = require('./../../routing');
@@ -37,7 +39,7 @@ const tests = [
 ];
 
 tests.forEach(({args, description, error, expected}) => {
-  return test(description, async ({end, strictSame, throws}) => {
+  return test(description, (t, end) => {
     if (!!error) {
       throws(() => findMaxPayable(args), error, 'Got expected error');
     } else {

package/test/routing/test_is_route_payable.js

@@ -1,7 +1,8 @@
-const {test} = require('@alexbosworth/tap');
+const {rejects} = require('node:assert').strict;
+const strictSame = require('node:assert').strict.deepStrictEqual;
+const test = require('node:test');
 
 const {getInfoResponse} = require('./../fixtures');
-
 const isRoutePayable = require('./../../routing/is_route_payable');
 
 const getInfoRes = () => JSON.parse(JSON.stringify(getInfoResponse));
@@ -206,15 +207,15 @@ const tests = [
 ];
 
 tests.forEach(({args, description, error, expected}) => {
-  return test(description, async ({end, equal, rejects}) => {
+  return test(description, async () => {
     if (!!error) {
-      rejects(isRoutePayable(args), error, 'Got expected error');
+      await rejects(isRoutePayable(args), error, 'Got expected error');
     } else {
       const payable = await isRoutePayable(args);
 
-      equal(payable.is_payable, expected.is_payable, 'Got is_payable');
+      strictSame(payable.is_payable, expected.is_payable, 'Got is_payable');
     }
 
-    return end();
+    return;
   });
 });