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

test: add regression test of pubsub and timer #413

Merged
merged 10 commits into from
Feb 13, 2023
Merged

test: add regression test of pubsub and timer #413

merged 10 commits into from
Feb 13, 2023

Conversation

wenfengwang
Copy link
Contributor

What problem does this PR solve?

Issue Number: close #xxx

Problem Summary

add regression test of pubsub

What is changed and how does it work?

program sends events to eventbus

  1. create an eventbus
  2. create a subscription
  3. start events sending
  4. start events receiving
  5. check if received all events in next 30s after sending finished

Check List

Tests

  • Unit test
  • Integration test
  • Manual test (add detailed scripts or steps below)
  • No code

@codecov
Copy link

codecov bot commented Feb 4, 2023

Codecov Report

Merging #413 (e679d94) into main (2a78cb5) will decrease coverage by 0.53%.
The diff coverage is 29.46%.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #413      +/-   ##
==========================================
- Coverage   57.71%   57.18%   -0.53%     
==========================================
  Files         180      182       +2     
  Lines       14614    14912     +298     
==========================================
+ Hits         8434     8527      +93     
- Misses       5552     5753     +201     
- Partials      628      632       +4     
Impacted Files Coverage Δ
...ernal/controller/trigger/validation/subscripton.go 69.29% <ø> (+0.85%) ⬆️
internal/gateway/proxy/deadletter.go 0.00% <0.00%> (ø)
internal/gateway/proxy/direct.go 73.33% <0.00%> (-5.24%) ⬇️
internal/gateway/proxy/proxy.go 24.67% <0.00%> (-0.42%) ⬇️
internal/store/segment/server.go 14.07% <0.00%> (-0.06%) ⬇️
internal/trigger/client/gcloud_functions.go 0.00% <0.00%> (ø)
internal/trigger/client/grpc.go 0.00% <0.00%> (ø)
internal/trigger/client/http.go 0.00% <0.00%> (ø)
internal/trigger/client/interface.go 0.00% <0.00%> (-19.24%) ⬇️
internal/trigger/client/lambda.go 0.00% <0.00%> (ø)
... and 18 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2c5fd4d...e679d94. Read the comment docs.

test/regression/regression.yml Show resolved Hide resolved
test/regression/Dockerfile Outdated Show resolved Hide resolved
test/regression/regression.yml Show resolved Hide resolved
test/regression/run.sh Outdated Show resolved Hide resolved
test/regression/run.sh Outdated Show resolved Hide resolved
test/utils/conn.go Outdated Show resolved Hide resolved
@wenfengwang wenfengwang changed the title test: add regression test of pubsub test: add regression test of pubsub and timer Feb 7, 2023
@wenfengwang wenfengwang force-pushed the test-pubsub branch 2 times, most recently from 418c796 to 0410ad3 Compare February 11, 2023 07:45
hwjiangkai
hwjiangkai previously approved these changes Feb 13, 2023
internal/gateway/proxy/proxy.go Show resolved Hide resolved
@wenfengwang wenfengwang merged commit 8b3837f into main Feb 13, 2023
@ifplusor ifplusor deleted the test-pubsub branch March 3, 2023 03:38
wenfengwang added a commit that referenced this pull request Mar 23, 2023
* test: add regression test of pubsub

* remove unused code

* update

* update review commets

* test: trigger read event add log

Signed-off-by: xdlbdy <xdlbdy@gmail.com>


* update golang sdk

---------

Signed-off-by: xdlbdy <xdlbdy@gmail.com>
Co-authored-by: xdlbdy <xdlbdy@gmail.com>
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.

3 participants