From 92e8c6fdca0faca21b1d06065797220345770c8c Mon Sep 17 00:00:00 2001 From: joostdecock Date: Fri, 19 May 2023 10:34:51 +0200 Subject: [PATCH] chore(core): Code QA fixes --- packages/core/src/path.mjs | 2 +- packages/core/tests/part.test.mjs | 4 ++-- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/packages/core/src/path.mjs b/packages/core/src/path.mjs index 6783b5b57f4..bdaa2c05774 100644 --- a/packages/core/src/path.mjs +++ b/packages/core/src/path.mjs @@ -187,7 +187,7 @@ Path.prototype.clean = function () { if (!(op.cp1.sitsRoughlyOn(cur) && op.cp2.sitsRoughlyOn(cur) && op.to.sitsRoughlyOn(cur))) ops.push(ops) } - cur = op?.to + cur = op.to } if (ops.length < this.ops.length) this.ops = ops diff --git a/packages/core/tests/part.test.mjs b/packages/core/tests/part.test.mjs index 2dc5338b258..19e645e119c 100644 --- a/packages/core/tests/part.test.mjs +++ b/packages/core/tests/part.test.mjs @@ -304,7 +304,7 @@ describe('Part', () => { const part = { name: 'test', draft: ({ options, part }) => { - if (options.test) return part + if (options.test === 'This should never match') return null else return part }, } @@ -321,7 +321,7 @@ describe('Part', () => { const part = { name: 'test', draft: ({ absoluteOptions, part }) => { - if (absoluteOptions.test) return part + if (absoluteOptions.test === 'this check should always fail') return null else return part }, }