Remove SSHUploadPack from TestGitalyServer

SSHUploadPack has been removed from Gitaly proto file, so
TestGitalyServer cannot refer to it anymore. This commit removes
SSHUploadPack related code.

Author: Eric Ju

client/testserver/gitalyserver.go

@@ -35,18 +35,6 @@ func (s *TestGitalyServer) SSHReceivePack(stream pb.SSHService_SSHReceivePackSer
 	return stream.Send(&pb.SSHReceivePackResponse{Stdout: response})
 }
 
-func (s *TestGitalyServer) SSHUploadPack(stream pb.SSHService_SSHUploadPackServer) error {
-	req, err := stream.Recv()
-	if err != nil {
-		return err
-	}
-
-	s.ReceivedMD, _ = metadata.FromIncomingContext(stream.Context())
-
-	response := []byte("UploadPack: " + req.Repository.GlRepository)
-	return stream.Send(&pb.SSHUploadPackResponse{Stdout: response})
-}
-
 func (s *TestGitalyServer) SSHUploadPackWithSidechannel(ctx context.Context, req *pb.SSHUploadPackWithSidechannelRequest) (*pb.SSHUploadPackWithSidechannelResponse, error) {
 	conn, err := client.OpenServerSidechannel(ctx)
 	if err != nil {

support/lint_last_known_acceptable.txt

@@ -18,12 +18,11 @@ client/httpsclient_test.go:109:5: expected-actual: need to reverse actual and ex
 client/testserver/gitalyserver.go:1:1: package-comments: should have a package comment (revive)
 client/testserver/gitalyserver.go:21:6: exported: exported type TestGitalyServer should have comment or be unexported (revive)
 client/testserver/gitalyserver.go:26:1: exported: exported method TestGitalyServer.SSHReceivePack should have comment or be unexported (revive)
-client/testserver/gitalyserver.go:38:1: exported: exported method TestGitalyServer.SSHUploadPack should have comment or be unexported (revive)
-client/testserver/gitalyserver.go:50:1: exported: exported method TestGitalyServer.SSHUploadPackWithSidechannel should have comment or be unexported (revive)
-client/testserver/gitalyserver.go:55:18: Error return value of `conn.Close` is not checked (errcheck)
-client/testserver/gitalyserver.go:70:1: exported: exported method TestGitalyServer.SSHUploadArchive should have comment or be unexported (revive)
-client/testserver/gitalyserver.go:82:1: exported: exported function StartGitalyServer should have comment or be unexported (revive)
-client/testserver/gitalyserver.go:131:3: go-require: require must only be used in the goroutine running the test function (testifylint)
+client/testserver/gitalyserver.go:38:1: exported: exported method TestGitalyServer.SSHUploadPackWithSidechannel should have comment or be unexported (revive)
+client/testserver/gitalyserver.go:43:18: Error return value of `conn.Close` is not checked (errcheck)
+client/testserver/gitalyserver.go:58:1: exported: exported method TestGitalyServer.SSHUploadArchive should have comment or be unexported (revive)
+client/testserver/gitalyserver.go:70:1: exported: exported function StartGitalyServer should have comment or be unexported (revive)
+client/testserver/gitalyserver.go:119:3: go-require: require must only be used in the goroutine running the test function (testifylint)
 client/testserver/testserver.go:1:1: ST1000: at least one file in a package should have a package comment (stylecheck)
 client/testserver/testserver.go:20:6: exported: exported type TestRequestHandler should have comment or be unexported (revive)
 client/testserver/testserver.go:46:12: G112: Potential Slowloris Attack because ReadHeaderTimeout is not configured in the http.Server (gosec)