end-to-end testing shell scripts #270
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
robur-team
force-pushed
the
e2e-testing
branch
3 times, most recently
from
539b273 to
838179e
Compare
In static client configuration, the "ifconfig A B" is IP address of client followed by the IP address of the server In other configurations, B is a netmask. Co-Authored-By: Reynir Björnsson <[email protected]> Co-Authored-By: Hannes Mehnert <[email protected]>
Move ip_config back to State to avoid circular dependency Co-Authored-By: Reynir Björnsson <[email protected]> Co-Authored-By: Hannes Mehnert <[email protected]>
This mode establishes a tunnel and sends a single ping, waits for a reply, and exits Co-Authored-By: Reynir Björnsson <[email protected]> Co-Authored-By: Hannes Mehnert <[email protected]>
The test script sets up a openvpn server with varying configuration and runs miragevpn-client-notun --test against the server. Co-Authored-By: Reynir Björnsson <[email protected]> Co-Authored-By: Hannes Mehnert <[email protected]>
Upon receiving an echo request the server sends back a HALT control message to the client and then exits with exit code 0. The HALT control message is sent back in order to tell the client to exit and not try to reconnect.
On VPN establish we send an echo request immediately instead of waiting one second.
* Use rm -f for the pidfile in case OpenVPN removes it first * Use --verb 4 instead of --verb 5 in static/server.conf as otherwise OpenVPN will print rwWwrrRrW to stderr as read/write feedback.
send HALT to client and exit after replying to the ping Co-Authored-By: Reynir Björnsson <[email protected]> Co-Authored-By: Hannes Mehnert <[email protected]>
Co-Authored-By: Reynir Björnsson <[email protected]> Co-Authored-By: Hannes Mehnert <[email protected]>
- turn coverage off at pretty printers - execute e2e tests with verbose (to cover logs) Co-Authored-By: Reynir Björnsson <[email protected]> Co-Authored-By: Hannes Mehnert <[email protected]>
robur-team
force-pushed
the
e2e-testing
branch
5 times, most recently
from
a9b7dff to
7617068
Compare
hannesm
force-pushed
the
e2e-testing
branch
2 times, most recently
from
aa578a1 to
2f5d285
Compare
|
The last commit re-adds AES-GCM to tls-crypt in data-ciphers, since the GitHub runner is ubuntu 22.04 with openvpn 2.5.9 which doesn't support poly1305-chacha20. I tried to use ubuntu 24.04, but that failed to install darcs (which somehow opam requires in the setup-ocaml CI action). Anyways, coverage around 72%. |
|
|
I think this is very nice now! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.