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.

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

Speed up unit tests #497

Merged
merged 10 commits into from
Feb 6, 2025
Merged

Speed up unit tests #497

merged 10 commits into from
Feb 6, 2025

Conversation

timokoessler
Copy link
Member

No description provided.

Copy link

codecov bot commented Jan 13, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

📢 Thoughts on this report? Let us know!

@timokoessler timokoessler marked this pull request as ready for review January 13, 2025 08:57
hansott and others added 7 commits January 15, 2025 10:33
* 'main' of github.com:AikidoSec/node-RASP:
  Use separate table for postgres sink tests
  Upgrade Zen internals to v0.1.35
  Use cURL to send requests to server
  Use req.destroy()
  Use t.fail(error)
  Pass body to `req.end(...)` instead of `req.write(...)`
  Assert headers for fetch helper
@hansott hansott merged commit 3005698 into main Feb 6, 2025
11 checks passed
@hansott hansott deleted the speed-up-ci branch February 6, 2025 15:30
hansott added a commit that referenced this pull request Feb 7, 2025
* 'main' of github.com:AikidoSec/node-RASP: (38 commits)
  Speed up unit tests (#497)
  Move outside if
  Keep original code
  Use Number.isInteger(...)
  Update Zen internals to v0.1.37
  Fix timestamp in mock server (#517)
  Remove unused methods (#516)
  Update readme
  Set timeout on workflows
  Use 401
  Increase timeout to 3s and append error message to log
  Check route level access before we check if IP is in bypass list
  Only install library deps when publishing to npm (#511)
  Show connection failures on startup (#510)
  Revert "Add failing test for route level allowed IPs"
  Fix duplicate clickhouse
  Run tests for supported dialects
  Update Zen internals to v0.1.36
  Add failing test for route level allowed IPs
  Only capture hostname if the port is known
  ...
hansott added a commit that referenced this pull request Feb 7, 2025
hansott added a commit that referenced this pull request Feb 7, 2025
…519)

* Select the first valid & non-private IP from x-forwarded-for header

* Add test case for IP with port

* More tests

* Fix unit test

* Fix test

* Revert "Speed up unit tests (#497)"

This reverts commit 3005698.
timokoessler added a commit that referenced this pull request Feb 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants