-
Notifications
You must be signed in to change notification settings - Fork 297
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
chore: integration test for rETL flow #4324
Conversation
Important Auto Review SkippedAuto reviews are disabled on this repository. Please check the settings in the CodeRabbit UI or the To trigger a single review, invoke the Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configration File (
|
c3d6c14
to
d4bd862
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #4324 +/- ##
==========================================
+ Coverage 74.02% 74.08% +0.05%
==========================================
Files 381 381
Lines 54404 54407 +3
==========================================
+ Hits 40272 40306 +34
+ Misses 11849 11816 -33
- Partials 2283 2285 +2 ☔ View full report in Codecov by Sentry. |
t.Fail() | ||
return | ||
} | ||
if res.Status != "200 OK" { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why not check the status code?
if res.Status != "200 OK" { | |
if res.StatusCode != http.StatusOK { |
398e69f
to
49db216
Compare
5bf9b11
to
f4fa181
Compare
36ed083
to
b840876
Compare
a163d83
to
8bd20b9
Compare
8bd20b9
to
483378b
Compare
integration_test/retl_test/sut.go
Outdated
var transformerContainer *destination.TransformerResource | ||
|
||
containersGroup.Go(func() (err error) { | ||
postgresContainer, err = resource.SetupPostgres(pool, t) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think you need to change these, it now is postgres.Setup(pool, t)
.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I pushed the changes @lvrach
--------- Co-authored-by: siddarth.msv <siddarthmsv+1@gmail.com> Co-authored-by: Francesco Casula <fra.casula@gmail.com>
Description
An integration test for the recently added
/retl
endpoint. This new endpoint lets users also provide destination id to which they want to route their events to. Tests added to verify sources stats' behaviour has remained the same.Linear Ticket
linear resolves PIPE-701
Security