Skip to content
This repository was archived by the owner on Sep 30, 2024. It is now read-only.

gitserver: grpc: create proto roundtrip tests for GetObject[Request/Response]#62564

Merged
ggilmore merged 1 commit into
mainfrom
05-08-gitserver_grpc_create_proto_roundtrip_tests_for_getobject_request_response_
May 11, 2024
Merged

gitserver: grpc: create proto roundtrip tests for GetObject[Request/Response]#62564
ggilmore merged 1 commit into
mainfrom
05-08-gitserver_grpc_create_proto_roundtrip_tests_for_getobject_request_response_

Conversation

@ggilmore

@ggilmore ggilmore commented May 8, 2024

Copy link
Copy Markdown
Contributor

Closes https://github.com/sourcegraph/sourcegraph/issues/60411

We didn't have any round trip tests for the GetCommit request or response type. This PR adds them.

Test plan

New unit tests

ggilmore commented May 8, 2024

Copy link
Copy Markdown
Contributor Author

@github-actions github-actions Bot added team/product-platform team/source Tickets under the purview of Source - the one Source to graph it all labels May 8, 2024
@ggilmore ggilmore requested a review from a team May 8, 2024 22:03
@ggilmore ggilmore marked this pull request as ready for review May 8, 2024 22:03
@graphite-app

graphite-app Bot commented May 9, 2024

Copy link
Copy Markdown

Celebrity gif. A young Keanu Reeves stands in the rain smiling. He raises up his arm and gives an enthusiastic thumbs up. (Added via Giphy)

@ggilmore ggilmore force-pushed the 05-08-gitserver_client_grpc_add_tests_for_getobject branch from a0303e5 to 713e225 Compare May 10, 2024 20:55
Base automatically changed from 05-08-gitserver_client_grpc_add_tests_for_getobject to main May 10, 2024 21:40
@ggilmore ggilmore force-pushed the 05-08-gitserver_grpc_create_proto_roundtrip_tests_for_getobject_request_response_ branch from eabfb1b to 7e51609 Compare May 10, 2024 21:41

ggilmore commented May 10, 2024

Copy link
Copy Markdown
Contributor Author

Merge activity

  • May 10, 5:42 PM EDT: Graphite rebased this pull request as part of a merge.
  • May 10, 5:43 PM EDT: Graphite couldn't merge this PR because it was not satisfying all requirements (Failed CI: 'buildkite/sourcegraph', 'scip-go', 'scip-go (lib)', 'scip-go (monitoring)', 'scip-go (dev/ci/images)').

@ggilmore ggilmore merged commit 74f2e25 into main May 11, 2024
@ggilmore ggilmore deleted the 05-08-gitserver_grpc_create_proto_roundtrip_tests_for_getobject_request_response_ branch May 11, 2024 08:37
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

cla-signed team/product-platform team/source Tickets under the purview of Source - the one Source to graph it all

Projects

None yet

Development

Successfully merging this pull request may close these issues.

gitserver: Move GetObject to new gRPC pattern

3 participants