Skip to content

Commit

Permalink
fix: missing return in invokeRequest for stream rpc (#206)
Browse files Browse the repository at this point in the history
* fix: missing return in invokeRequest for stream rpc

fix Got error when running the sample grpc testsuite file #200

Signed-off-by: Ink33 <[email protected]>

* re-order the test cases in .github/testing/core.yaml

---------

Signed-off-by: Ink33 <[email protected]>
Co-authored-by: Rick <[email protected]>
  • Loading branch information
Ink-33 and LinuxSuRen authored Sep 6, 2023
1 parent 69e53b6 commit ada9ca0
Show file tree
Hide file tree
Showing 3 changed files with 6 additions and 5 deletions.
8 changes: 4 additions & 4 deletions .github/testing/core.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -5,16 +5,16 @@ name: atest
api: |
{{default "http://localhost:8080/server.Runner" (env "SERVER")}}
items:
- name: suites
request:
api: /GetSuites
method: POST
- name: createSuite
request:
api: /CreateTestSuite
method: POST
body: |
{"name": "{{randAlpha 6}}"}
- name: suites
request:
api: /GetSuites
method: POST
- name: suite
request:
api: /GetTestSuite
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/build.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -46,6 +46,7 @@ jobs:
sudo atest service restart
sudo atest service status
atest run -p .github/testing/core.yaml --request-ignore-error --report md --report-file .github/workflows/report.md
atest run -p sample/grpc-sample.yaml
sudo atest service status
atest convert -p .github/testing/core.yaml --converter jmeter -t sample.jmx
Expand Down
2 changes: 1 addition & 1 deletion pkg/runner/grpc.go
Original file line number Diff line number Diff line change
Expand Up @@ -129,7 +129,7 @@ func invokeRequest(ctx context.Context, md protoreflect.MethodDescriptor, payloa
if err != nil {
return nil, err
}

return buildResponses(resps)
}
request, err := getReqMessagePb(md, payload)
if err != nil {
Expand Down

0 comments on commit ada9ca0

Please sign in to comment.