Skip to content

Commit 2f5e048

Browse files
committed
Enable testing on Windows
It was previously disabled on Windows as it was believed that the gnat_arm_elf toolchain would behave the same on both Windows and Linux. However, from a post on the Ada forum, it seems that there may be cases where projects build on Linux but not on Windows. Therefore, we re-enable our testsuite on Windows out of an abundance of caution.
1 parent 9c1aebf commit 2f5e048

1 file changed

Lines changed: 1 addition & 11 deletions

File tree

.github/workflows/main.yaml

Lines changed: 1 addition & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -43,28 +43,18 @@ jobs:
4343
echo "::endgroup::"
4444
done
4545
46-
# Don't run tests on Windows since it's so slow compared to Linux.
47-
#
48-
# Once we've proven that the runtimes can be built on Windows (in the
49-
# above step), there's not much point in also building test apps against
50-
# the runtimes since the Windows and Linux gnat_arm_elf toolchains should
51-
# behave the same at this point, and we're not interested in checking
52-
# for differences in those toolchains here.
53-
5446
- name: Install Test Dependencies
55-
if: ${{ matrix.os != 'windows-latest' }}
5647
run: |
5748
pip install -r tests/requirements.txt
5849
5950
- name: Run tests
60-
if: ${{ matrix.os != 'windows-latest' }}
6151
run: |
6252
cd tests
6353
pytest . -v --junit-xml=reports/runtime-tests.xml
6454
6555
- name: Test Report
6656
uses: dorny/test-reporter@v2.6.0
67-
if: ${{ !cancelled() && (matrix.os != 'windows-latest') }} # run this step even if previous step failed
57+
if: ${{ !cancelled() }} # run this step even if previous step failed
6858
with:
6959
name: Runtime Tests
7060
path: tests/reports/*.xml

0 commit comments

Comments
 (0)