Attention is currently required from: plaisthos.
Hello plaisthos,
I'd like you to do a code review.
Please visit
http://gerrit.openvpn.net/c/openvpn/+/1487?usp=email
to review the following change.
Change subject: GHA: Run openvpnserv UT for MinGW builds
......................................................................
GHA: Run openvpnserv UT for MinGW builds
Should have been added in commit
b10ee38ccde5f155d02fcd3e8f1b1ab454826ab0.
Note that test_openvpnserv.exe lives in a
separate directory, so we need to make the
code a bit more flexible.
Change-Id: If61a91b4580864fd22162c94467ba3dda2045b7b
Signed-off-by: Frank Lichtenheld <[email protected]>
---
M .github/workflows/build.yaml
1 file changed, 6 insertions(+), 4 deletions(-)
git pull ssh://gerrit.openvpn.net:29418/openvpn refs/changes/87/1487/1
diff --git a/.github/workflows/build.yaml b/.github/workflows/build.yaml
index ef9b3f5..5f5d624 100644
--- a/.github/workflows/build.yaml
+++ b/.github/workflows/build.yaml
@@ -100,13 +100,13 @@
path: |
${{ github.workspace }}/out/build/mingw/${{ matrix.arch }}/**/${{
matrix.build }}/*.exe
${{ github.workspace }}/out/build/mingw/${{ matrix.arch }}/**/${{
matrix.build }}/*.dll
- !${{ github.workspace }}/out/build/mingw/${{ matrix.arch }}/${{
matrix.build }}/test_*.exe
+ !${{ github.workspace }}/out/build/mingw/${{ matrix.arch }}/**/${{
matrix.build }}/test_*.exe
- uses: actions/upload-artifact@b7c566a772e6b6bfb58ed0dc250532a479d7789f
# v6.0.0
with:
name: openvpn-mingw-${{ matrix.arch }}-${{ matrix.build }}-tests
path: |
- ${{ github.workspace }}/out/build/mingw/${{ matrix.arch }}/${{
matrix.build }}/test_*.exe
+ ${{ github.workspace }}/out/build/mingw/${{ matrix.arch }}/**/${{
matrix.build }}/test_*.exe
${{ github.workspace }}/out/build/mingw/${{ matrix.arch }}/${{
matrix.build }}/*.dll
mingw-unittest:
@@ -115,7 +115,7 @@
fail-fast: false
matrix:
arch: [x86, x64]
- test: [argv, auth_token, buffer, cryptoapi, crypto, misc,
options_parse, ncp, packet_id, pkt, provider, ssl, tls_crypt, user_pass]
+ test: [argv, auth_token, buffer, cryptoapi, crypto, misc,
options_parse, ncp, openvpnserv, packet_id, pkt, provider, ssl, tls_crypt,
user_pass]
build: [Release, Debug]
runs-on: windows-2025
@@ -129,7 +129,9 @@
name: openvpn-mingw-${{ matrix.arch }}-${{ matrix.build }}-tests
path: unittests
- name: Run ${{ matrix.test }} unit test
- run: ./unittests/test_${{ matrix.test }}.exe
+ run: |
+ $test_file=(Get-ChildItem -Path unittests -Filter test_${{
matrix.test }}.exe -Recurse).fullname
+ & $test_file
env:
srcdir: "${{ github.workspace }}/tests/unit_tests/openvpn"
--
To view, visit http://gerrit.openvpn.net/c/openvpn/+/1487?usp=email
To unsubscribe, or for help writing mail filters, visit
http://gerrit.openvpn.net/settings?usp=email
Gerrit-MessageType: newchange
Gerrit-Project: openvpn
Gerrit-Branch: master
Gerrit-Change-Id: If61a91b4580864fd22162c94467ba3dda2045b7b
Gerrit-Change-Number: 1487
Gerrit-PatchSet: 1
Gerrit-Owner: flichtenheld <[email protected]>
Gerrit-Reviewer: plaisthos <[email protected]>
Gerrit-CC: openvpn-devel <[email protected]>
Gerrit-Attention: plaisthos <[email protected]>
_______________________________________________
Openvpn-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/openvpn-devel
