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 #65

Merged
merged 9 commits into from
Dec 20, 2024
Merged

Add tests #65

merged 9 commits into from
Dec 20, 2024

Conversation

michelemin
Copy link
Collaborator

@michelemin michelemin commented Dec 5, 2024 •
edited
Loading

This PR adds the following tests:

  • log back system (with delayed log back)
  • reading from / writing to the DB
  • opening a file on the host from a rule (impossible)
  • get random bytes
  • MNRs: not allowed, with URL templating, with dynamic headers

@michelemin michelemin changed the title [WIP] Add tests Add tests Dec 9, 2024
@michelemin michelemin requested a review from obelisk December 9, 2024 23:05
@michelemin michelemin marked this pull request as ready for review December 9, 2024 23:05
@michelemin michelemin force-pushed the test_modules branch 4 times, most recently from c237197 to 6b84baa Compare December 10, 2024 12:04
@obelisk obelisk merged commit 4813434 into obelisk:main Dec 20, 2024
3 checks passed
@michelemin michelemin deleted the test_modules branch December 20, 2024 08:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@obelisk obelisk obelisk approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@michelemin @obelisk