Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Add tests with scheduled runs #22

Open
wants to merge 3 commits into
base: main
Choose a base branch
from
Open

Add tests with scheduled runs #22

wants to merge 3 commits into from

Conversation

bryopsida
Copy link
Owner

@bryopsida bryopsida commented Dec 13, 2023 •
edited
Loading

What

Adds a test that can/will be run daily and on PR that confirms.

  1. container on main or targeted tag runs
  2. can connect through wireguard to a http bin/echo service

Will be used to vet incoming base changes as desired by #21 and will prevent something like #18 from occurring again.

Todo

  • Finish docker-compose.test.yml so it has a test runner with wg client, wg server, backend http bin/echo server behind wg network. Run something like a postman test after connecting and verify connectivity
  • Connect/add as a GitHub job, it should run after building the image in PR, the built tag in PR should be used for wg server. When running on main it should use the latest main tag.
  • Run build on schedule to catch any issues introduced by base patches.
  • Add analysis of WG container logs as a test case, logs should be clean, no errors, warnings.

@bryopsida bryopsida linked an issue Dec 13, 2023 that may be closed by this pull request
Add scheduled/cron test #20
Open
@bryopsida bryopsida self-assigned this Dec 13, 2023
@bryopsida
wip: start of test runner
10355a7 
add postman collection shell

add empty wg conf placeholder

set correct capabilities for wg on in service

start of devcontainer env to iterate on wireguard setup for verification of container
hound[bot]
hound bot reviewed Dec 16, 2023
View reviewed changes
test.js Show resolved Hide resolved
@bryopsida bryopsida mentioned this pull request Dec 17, 2023
Closed
2 tasks
@socket-security Socket Security
Copy link

New dependencies detected. Learn more about Socket for GitHub ↗︎

Package New capabilities Transitives Size Publisher
npm/[email protected] environment, filesystem Transitive: eval, network, shell, unsafe +116 41 MB codenirvana

View full report↗︎

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hound hound[bot] hound[bot] left review comments

Assignees

@bryopsida bryopsida

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add scheduled/cron test
1 participant
@bryopsida