Skip to content
This repository was archived by the owner on Jul 24, 2024. It is now read-only.

Commit d0d8865

Browse files
committed
chore: Skip constructor tests on v14.6+
Related to tracking issue #2972
1 parent ee3984d commit d0d8865

File tree

6 files changed

+73
-8
lines changed

6 files changed

+73
-8
lines changed

.github/workflows/alpine.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -15,13 +15,13 @@ jobs:
1515
node:
1616
- 10
1717
- 12
18-
- 14.5
18+
- 14
1919
include:
2020
- node: 10
2121
alpine: "3.9"
2222
- node: 12
2323
alpine: "3.9"
24-
- node: 14.5
24+
- node: 14
2525
alpine: "3.10"
2626
steps:
2727
- name: Install Alpine build tools

.github/workflows/linux.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ jobs:
1212
node:
1313
- 10
1414
- 12
15-
- 14.5
15+
- 14
1616
include:
1717
- node: 10
1818
gcc: "gcc-4.9"
@@ -22,7 +22,7 @@ jobs:
2222
gcc: "gcc-6"
2323
gpp: "g++-6"
2424
os: ubuntu-18.04
25-
- node: 14.5
25+
- node: 14
2626
gcc: "gcc-6"
2727
gpp: "g++-6"
2828
os: ubuntu-18.04

.github/workflows/macos.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ jobs:
1212
node:
1313
- 10
1414
- 12
15-
- 14.5
15+
- 14
1616

1717
steps:
1818
- uses: actions/checkout@v2

.github/workflows/windows.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,7 +12,7 @@ jobs:
1212
node:
1313
- 10
1414
- 12
15-
- 14.5
15+
- 14
1616

1717
steps:
1818
- uses: actions/checkout@v2

appveyor.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@
3939
- nodejs_version: 12
4040
GYP_MSVS_VERSION: 2017
4141
APPVEYOR_BUILD_WORKER_IMAGE: Visual Studio 2017
42-
- nodejs_version: 14.5
42+
- nodejs_version: 14
4343
GYP_MSVS_VERSION: 2017
4444
APPVEYOR_BUILD_WORKER_IMAGE: Visual Studio 2017
4545

@@ -115,7 +115,7 @@
115115
- nodejs_version: 12
116116
GYP_MSVS_VERSION: 2017
117117
APPVEYOR_BUILD_WORKER_IMAGE: Visual Studio 2017
118-
- nodejs_version: 14.5
118+
- nodejs_version: 14
119119
GYP_MSVS_VERSION: 2017
120120
APPVEYOR_BUILD_WORKER_IMAGE: Visual Studio 2017
121121

test/types.js

Lines changed: 65 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@
33

44
var assert = require('assert').strict;
55
var sass = require('../');
6+
var semver = require('semver');
67

78
describe('sass.types', function() {
89
describe('Boolean', function() {
@@ -15,6 +16,10 @@ describe('sass.types', function() {
1516
});
1617

1718
it('supports call constructor', function() {
19+
if(semver.gt(process.version, 'v14.5.0')) {
20+
// v8 issue tracked in https://github.com/sass/node-sass/issues/2972
21+
this.skip();
22+
}
1823
var t = sass.types.Boolean(true);
1924
assert.strictEqual(t.toString(), '[object SassBoolean]');
2025

@@ -89,11 +94,21 @@ describe('sass.types', function() {
8994
});
9095

9196
it('supports call constructor', function() {
97+
if(semver.gt(process.version, 'v14.5.0')) {
98+
// v8 issue tracked in https://github.com/sass/node-sass/issues/2972
99+
this.skip();
100+
}
101+
92102
var t = sass.types.Color();
93103
assert.strictEqual(t.toString(), '[object SassColor]');
94104
});
95105

96106
it('supports new constructor', function() {
107+
if(semver.gt(process.version, 'v14.5.0')) {
108+
// v8 issue tracked in https://github.com/sass/node-sass/issues/2972
109+
this.skip();
110+
}
111+
97112
var t = new sass.types.Color(1);
98113
assert.strictEqual(t.toString(), '[object SassColor]');
99114
});
@@ -235,6 +250,11 @@ describe('sass.types', function() {
235250
});
236251

237252
it('supports call constructor', function() {
253+
if(semver.gt(process.version, 'v14.5.0')) {
254+
// v8 issue tracked in https://github.com/sass/node-sass/issues/2972
255+
this.skip();
256+
}
257+
238258
var e = sass.types.Error('Such Error');
239259
assert.ok(e instanceof sass.types.Error);
240260
assert.strictEqual(e.toString(), '[object SassError]');
@@ -243,6 +263,11 @@ describe('sass.types', function() {
243263
});
244264

245265
it('supports new constructor', function() {
266+
if(semver.gt(process.version, 'v14.5.0')) {
267+
// v8 issue tracked in https://github.com/sass/node-sass/issues/2972
268+
this.skip();
269+
}
270+
246271
var e = new sass.types.Error('Such Error');
247272
assert.ok(e instanceof sass.types.Error);
248273
assert.strictEqual(e.toString(), '[object SassError]');
@@ -260,12 +285,22 @@ describe('sass.types', function() {
260285
});
261286

262287
it('support call constructor', function() {
288+
if(semver.gt(process.version, 'v14.5.0')) {
289+
// v8 issue tracked in https://github.com/sass/node-sass/issues/2972
290+
this.skip();
291+
}
292+
263293
var list = sass.types.List();
264294
assert.ok(list instanceof sass.types.List);
265295
assert.strictEqual(list.toString(), '[object SassList]');
266296
});
267297

268298
it('support new constructor', function() {
299+
if(semver.gt(process.version, 'v14.5.0')) {
300+
// v8 issue tracked in https://github.com/sass/node-sass/issues/2972
301+
this.skip();
302+
}
303+
269304
var list = new sass.types.List();
270305
assert.ok(list instanceof sass.types.List);
271306
assert.strictEqual(list.toString(), '[object SassList]');
@@ -411,11 +446,21 @@ describe('sass.types', function() {
411446
});
412447

413448
it('supports call constructor', function() {
449+
if(semver.gt(process.version, 'v14.5.0')) {
450+
// v8 issue tracked in https://github.com/sass/node-sass/issues/2972
451+
this.skip();
452+
}
453+
414454
var x = sass.types.Map();
415455
assert.strictEqual(x.toString(), '[object SassMap]');
416456
});
417457

418458
it('supports new constructor', function() {
459+
if(semver.gt(process.version, 'v14.5.0')) {
460+
// v8 issue tracked in https://github.com/sass/node-sass/issues/2972
461+
this.skip();
462+
}
463+
419464
var x = new sass.types.Map();
420465
assert.strictEqual(x.toString(), '[object SassMap]');
421466
});
@@ -490,11 +535,21 @@ describe('sass.types', function() {
490535
});
491536

492537
it('supports new constructor', function() {
538+
if(semver.gt(process.version, 'v14.5.0')) {
539+
// v8 issue tracked in https://github.com/sass/node-sass/issues/2972
540+
this.skip();
541+
}
542+
493543
var number = new sass.types.Number();
494544
assert.strictEqual(number.toString(), '[object SassNumber]');
495545
});
496546

497547
it('supports call constructor', function() {
548+
if(semver.gt(process.version, 'v14.5.0')) {
549+
// v8 issue tracked in https://github.com/sass/node-sass/issues/2972
550+
this.skip();
551+
}
552+
498553
var number = sass.types.Number();
499554
assert.strictEqual(number.toString(), '[object SassNumber]');
500555
});
@@ -585,13 +640,23 @@ describe('sass.types', function() {
585640
});
586641

587642
it('supports call constructor', function() {
643+
if(semver.gt(process.version, 'v14.5.0')) {
644+
// v8 issue tracked in https://github.com/sass/node-sass/issues/2972
645+
this.skip();
646+
}
647+
588648
var x = sass.types.String('OMG');
589649

590650
assert.strictEqual(x.toString(), '[object SassString]');
591651
assert.strictEqual(x.getValue(), 'OMG');
592652
});
593653

594654
it('supports new constructor', function() {
655+
if(semver.gt(process.version, 'v14.5.0')) {
656+
// v8 issue tracked in https://github.com/sass/node-sass/issues/2972
657+
this.skip();
658+
}
659+
595660
var x = new sass.types.String('OMG');
596661

597662
assert.strictEqual(x.toString(), '[object SassString]');

0 commit comments

Comments
 (0)