Skip to content

Commit

Permalink
proxy: add websocket regression tests (#7121)
Browse files Browse the repository at this point in the history
## Problem

We have no regression tests for websocket flow

## Summary of changes

Add a hacky implementation of the postgres protocol over websockets just
to verify the protocol behaviour does not regress over time.
  • Loading branch information
conradludgate committed Mar 15, 2024
1 parent 76c44dc commit 49bc734
Show file tree
Hide file tree
Showing 3 changed files with 272 additions and 12 deletions.
94 changes: 82 additions & 12 deletions poetry.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

1 change: 1 addition & 0 deletions pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -40,6 +40,7 @@ pytest-split = "^0.8.1"
zstandard = "^0.21.0"
httpx = {extras = ["http2"], version = "^0.26.0"}
pytest-repeat = "^0.9.3"
websockets = "^12.0"

[tool.poetry.group.dev.dependencies]
mypy = "==1.3.0"
Expand Down
Loading

1 comment on commit 49bc734

@github-actions
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

2762 tests run: 2627 passed, 1 failed, 134 skipped (full report)


Failures on Postgres 14

  • test_bulk_insert[neon-github-actions-selfhosted]: release
# Run all failed tests locally:
scripts/pytest -vv -n $(nproc) -k "test_bulk_insert[neon-release-pg14-github-actions-selfhosted]"

Code coverage* (full report)

  • functions: 28.3% (7033 of 24809 functions)
  • lines: 47.1% (43489 of 92413 lines)

* collected from Rust tests only


The comment gets automatically updated with the latest test results
49bc734 at 2024-03-15T10:32:15.603Z :recycle:

Please sign in to comment.