Skip to content

Commit

Permalink
Mark test_sigaction_default as cross platform. NFC (emscripten-core#2…
Browse files Browse the repository at this point in the history
…0272)

It looks like this test is failing on windows on the emscripten-releases
builder.  Marking it as cross platform to see if it also fails on github
CI on windows.
  • Loading branch information
sbc100 authored Sep 18, 2023
1 parent b57effb commit cb9471c
Showing 1 changed file with 7 additions and 4 deletions.
11 changes: 7 additions & 4 deletions test/test_core.py
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,6 @@
import random
import re
import shutil
import signal
import sys
import time
import unittest
Expand All @@ -37,6 +36,9 @@

logger = logging.getLogger("test_core")

EM_SIGINT = 2
EM_SIGABRT = 6


def wasm_simd(f):
@wraps(f)
Expand Down Expand Up @@ -6184,14 +6186,15 @@ def test_signals(self):
self.do_core_test(test_file('test_signals.c'))

@parameterized({
'sigint': (signal.SIGINT, 128 + signal.SIGINT, True),
'sigabrt': (signal.SIGABRT, 7, False)
'sigint': (EM_SIGINT, 128 + EM_SIGINT, True),
'sigabrt': (EM_SIGABRT, 7, False)
})
@crossplatform
def test_sigaction_default(self, signal, exit_code, assert_identical):
self.set_setting('EXIT_RUNTIME')
self.do_core_test(
test_file('test_sigaction_default.c'),
args=[str(int(signal))],
args=[str(signal)],
assert_identical=assert_identical,
assert_returncode=exit_code
)
Expand Down

0 comments on commit cb9471c

Please sign in to comment.