Skip to content

Commit

Permalink
Skip getIgnoredUpgradesDueToEnginesNode.
Browse files Browse the repository at this point in the history
This test needs to be rewritten because it is run against live data that affects the outcome of the test. The eslint and del versions were mocked in order to prevent this, but now the latest del.enginesNode is >=18 which fails the test. This data should either be mocked or the target packages should be entirely replaced by packages under our control.
  • Loading branch information
raineorshine committed Oct 16, 2024
1 parent 3c347ac commit 3f5f58a
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion test/getIgnoredUpgradesDueToEnginesNode.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,8 @@ const MOCK_DEL_VERSION = '999.0.1'

chaiSetup()

describe('getIgnoredUpgradesDueToEnginesNode', function () {
/* This test needs to be rewritten because it is run against live data that affects the outcome of the test. The eslint and del versions were mocked in order to prevent this, but now the latest del.enginesNode is >=18 which fails the test. This data should either be mocked or the target packages should be entirely replaced by packages under our control. */
describe.skip('getIgnoredUpgradesDueToEnginesNode', function () {
it('ncu-test-peer-update', async () => {
const data = await getIgnoredUpgradesDueToEnginesNode(
{
Expand Down

0 comments on commit 3f5f58a

Please sign in to comment.