Skip to content

Commit 0867958

Browse files
authored
[test] Extract try_require_node_version helper function. NFC (#24463)
1 parent cd4761e commit 0867958

File tree

1 file changed

+17
-18
lines changed

1 file changed

+17
-18
lines changed

test/common.py

Lines changed: 17 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -1038,12 +1038,8 @@ def require_wasm64(self):
10381038
if self.is_browser_test():
10391039
return
10401040

1041-
nodejs = self.get_nodejs()
1042-
if nodejs:
1043-
version = shared.get_node_version(nodejs)
1044-
if version >= (24, 0, 0):
1045-
self.js_engines = [nodejs]
1046-
return
1041+
if self.try_require_node_version(24):
1042+
return
10471043

10481044
if config.V8_ENGINE and config.V8_ENGINE in self.js_engines:
10491045
self.emcc_args.append('-sENVIRONMENT=shell')
@@ -1055,16 +1051,23 @@ def require_wasm64(self):
10551051
else:
10561052
self.fail('either d8 or node >= 24 required to run wasm64 tests. Use EMTEST_SKIP_WASM64 to skip')
10571053

1054+
def try_require_node_version(self, major, minor = 0, revision = 0):
1055+
nodejs = self.get_nodejs()
1056+
if not nodejs:
1057+
self.skipTest('Test requires nodejs to run')
1058+
version = shared.get_node_version(nodejs)
1059+
if version < (major, minor, revision):
1060+
return False
1061+
1062+
self.js_engines = [nodejs]
1063+
return True
1064+
10581065
def require_simd(self):
10591066
if self.is_browser_test():
10601067
return
10611068

1062-
nodejs = self.get_nodejs()
1063-
if nodejs:
1064-
version = shared.get_node_version(nodejs)
1065-
if version >= (16, 0, 0):
1066-
self.js_engines = [nodejs]
1067-
return
1069+
if self.try_require_node_version(16):
1070+
return
10681071

10691072
if config.V8_ENGINE and config.V8_ENGINE in self.js_engines:
10701073
self.emcc_args.append('-sENVIRONMENT=shell')
@@ -1078,12 +1081,8 @@ def require_simd(self):
10781081

10791082
def require_wasm_legacy_eh(self):
10801083
self.set_setting('WASM_LEGACY_EXCEPTIONS')
1081-
nodejs = self.get_nodejs()
1082-
if nodejs:
1083-
version = shared.get_node_version(nodejs)
1084-
if version >= (17, 0, 0):
1085-
self.js_engines = [nodejs]
1086-
return
1084+
if self.try_require_node_version(17):
1085+
return
10871086

10881087
if config.V8_ENGINE and config.V8_ENGINE in self.js_engines:
10891088
self.emcc_args.append('-sENVIRONMENT=shell')

0 commit comments

Comments
 (0)