remote capability negociation

Author: mcuadros

common_test.go

@@ -119,6 +119,13 @@ func (c *MockFetchPackSession) AdvertisedReferences() (*packp.AdvRefs, error) {
 func (c *MockFetchPackSession) FetchPack(
 	r *packp.UploadPackRequest) (io.ReadCloser, error) {
 
+	if !r.Capabilities.Supports(capability.Agent) {
+		return nil, fmt.Errorf("" +
+			"invalid test rquest, missing Agent capability, the request" +
+			"should be created using NewUploadPackRequestFromCapabilities",
+		)
+	}
+
 	f := fixtures.ByURL(c.endpoint.String())
 
 	if len(r.Wants) == 1 {

remote.go

@@ -171,11 +171,11 @@ func (r *Remote) getWantedReferences(spec []config.RefSpec) ([]*plumbing.Referen
 func (r *Remote) buildRequest(
 	s storer.ReferenceStorer, o *FetchOptions, refs []*plumbing.Reference,
 ) (*packp.UploadPackRequest, error) {
-	req := packp.NewUploadPackRequest()
+	req := packp.NewUploadPackRequestFromCapabilities(r.advRefs.Capabilities)
 	req.Depth = packp.DepthCommits(o.Depth)
 
 	for _, ref := range refs {
-		req.Want(ref.Hash())
+		req.Wants = append(req.Wants, ref.Hash())
 	}
 
 	i, err := s.IterReferences()
@@ -188,7 +188,7 @@ func (r *Remote) buildRequest(
 			return nil
 		}
 
-		req.Have(ref.Hash())
+		req.Haves = append(req.Haves, ref.Hash())
 		return nil
 	})