Revert "Feat/add external reference (#592)"

This reverts commit 67d51c1.

Author: YOU54F

command/root_test.go

@@ -54,7 +54,7 @@ func captureOutput(action func()) string {
 
 	action()
 
-	_ = w.Close()
+	w.Close()
 	out, _ := io.ReadAll(r)
 	os.Stderr = rescueStderr
 

consumer/http.go

@@ -201,7 +201,7 @@ func (p *httpMockProvider) displayMismatches(t *testing.T, mismatches []native.M
 		fmt.Println("\t\tDiff:")
 		log.Println("[INFO] pact validation failed, errors: ")
 		for _, m := range mismatches {
-			formattedRequest := fmt.Sprintf("%s %s", m.Method, m.Path)
+			formattedRequest := fmt.Sprintf("%s %s", m.Request.Method, m.Request.Path)
 			switch m.Type {
 			case "missing-request":
 				fmt.Printf("\t\texpected: \t%s (Expected request that was not received)\n", formattedRequest)

consumer/http_v2.go

@@ -35,7 +35,7 @@ func NewV2Pact(config MockHTTPProviderConfig) (*V2HTTPMockProvider, error) {
 // AddInteraction to the pact
 func (p *V2HTTPMockProvider) AddInteraction() *V2UnconfiguredInteraction {
 	log.Println("[DEBUG] pact add V2 interaction")
-	interaction := p.mockserver.NewInteraction("")
+	interaction := p.httpMockProvider.mockserver.NewInteraction("")
 
 	i := &V2UnconfiguredInteraction{
 		interaction: &Interaction{

consumer/http_v3.go

@@ -35,7 +35,7 @@ func NewV3Pact(config MockHTTPProviderConfig) (*V3HTTPMockProvider, error) {
 // AddInteraction to the pact
 func (p *V3HTTPMockProvider) AddInteraction() *V3UnconfiguredInteraction {
 	log.Println("[DEBUG] pact add V3 interaction")
-	interaction := p.mockserver.NewInteraction("")
+	interaction := p.httpMockProvider.mockserver.NewInteraction("")
 
 	i := &V3UnconfiguredInteraction{
 		interaction: &Interaction{

consumer/http_v4.go

@@ -36,7 +36,7 @@ func NewV4Pact(config MockHTTPProviderConfig) (*V4HTTPMockProvider, error) {
 // AddInteraction to the pact
 func (p *V4HTTPMockProvider) AddInteraction() *V4UnconfiguredInteraction {
 	log.Println("[DEBUG] pact add V4 interaction")
-	interaction := p.mockserver.NewInteraction("")
+	interaction := p.httpMockProvider.mockserver.NewInteraction("")
 
 	i := &V4UnconfiguredInteraction{
 		interaction: &Interaction{
@@ -92,15 +92,6 @@ func (i *V4UnconfiguredInteraction) UponReceiving(description string) *V4Unconfi
 	return i
 }
 
-// AddExternalReference records a reference to an external resource (such as a ticket or
-// pull request) against the interaction. References appear under
-// comments.references[group][name] in the Pact file. May be called multiple times.
-func (i *V4UnconfiguredInteraction) AddExternalReference(group, name, value string) *V4UnconfiguredInteraction {
-	i.interaction.interaction.WithReference(group, name, value)
-
-	return i
-}
-
 // WithRequest provides a builder for the expected request
 func (i *V4UnconfiguredInteraction) WithCompleteRequest(request Request) *V4InteractionWithCompleteRequest {
 	i.interaction.WithCompleteRequest(request)

consumer/http_v4_test.go

@@ -2,7 +2,6 @@ package consumer
 
 import (
 	"fmt"
-	"net/http"
 	"os"
 	"strings"
 	"testing"
@@ -86,25 +85,6 @@ func TestHttpV4TypeSystem(t *testing.T) {
 
 }
 
-func TestV4HTTPAddExternalReference(t *testing.T) {
-	p, err := NewV4Pact(MockHTTPProviderConfig{
-		Consumer: "consumer",
-		Provider: "provider",
-	})
-	assert.NoError(t, err)
-
-	err = p.AddInteraction().
-		UponReceiving("a request with an external reference").
-		AddExternalReference("Jira", "TICKET-123", "https://jira.example.com/browse/TICKET-123").
-		WithRequest("GET", "/", func(b *V4RequestBuilder) {}).
-		WillRespondWith(200, func(b *V4ResponseBuilder) {}).
-		ExecuteTest(t, func(msc MockServerConfig) error {
-			_, err := http.Get(fmt.Sprintf("http://%s:%d/", msc.Host, msc.Port))
-			return err
-		})
-	assert.NoError(t, err)
-}
-
 var Like = matchers.Like
 var EachLike = matchers.EachLike
 var Term = matchers.Term

examples/avro/codec.go

@@ -1,12 +1,11 @@
 package avro
 
 import (
+	
 	"os"
-
 	"github.com/linkedin/goavro/v2"
 )
 
-//nolint:unused // Retained as a reusable helper for the example package.
 func getCodec() *goavro.Codec {
 	schema, err := os.ReadFile("user.avsc")
 	if err != nil {

examples/grpc/routeguide/server/server.go

@@ -40,8 +40,9 @@ import (
 	"github.com/pact-foundation/pact-go/v2/examples/grpc/routeguide/data"
 	"google.golang.org/grpc/codes"
 	"google.golang.org/grpc/credentials"
-    "google.golang.org/grpc/status"
-    "google.golang.org/protobuf/proto"
+	"google.golang.org/grpc/status"
+
+	"github.com/golang/protobuf/proto"
 
 	pb "github.com/pact-foundation/pact-go/v2/examples/grpc/routeguide"
 )
@@ -54,15 +55,6 @@ var (
 	port       = flag.Int("port", 50051, "The server port")
 )
 
-// Keep example flags/entrypoint available for standalone usage.
-var (
-    _ = tls
-    _ = certFile
-    _ = keyFile
-    _ = port
-    _ = main
-)
-
 type routeGuideServer struct {
 	pb.UnimplementedRouteGuideServer
 	savedFeatures []*pb.Feature // read-only after initialized
@@ -255,9 +247,7 @@ func main() {
 	}
 	grpcServer := grpc.NewServer(opts...)
 	pb.RegisterRouteGuideServer(grpcServer, NewServer())
-    if err := grpcServer.Serve(lis); err != nil {
-        log.Fatalf("failed to serve: %v", err)
-    }
+	grpcServer.Serve(lis)
 }
 
 // exampleData is a copy of testdata/route_guide_db.json. It's to avoid

installer/installer.go

@@ -302,7 +302,7 @@ var setMacOSInstallName = func(file string) error {
 		return fmt.Errorf("error setting install name on pact lib: %s", err)
 	}
 
-	log.Println("[DEBUG] output from command", string(stdoutStderr))
+	log.Println("[DEBUG] output from command", stdoutStderr)
 
 	return err
 }
@@ -388,7 +388,7 @@ const (
 var packages = map[string]packageInfo{
 	FFIPackage: {
 		libName:     "libpact_ffi",
-		version:     "0.5.6",
+		version:     "0.4.28",
 		semverRange: ">= 0.4.0, < 1.0.0",
 	},
 }
@@ -417,17 +417,13 @@ func (d *defaultDownloader) download(src string, dst string) error {
 	if err != nil {
 		return fmt.Errorf("failed to create output file; %w", err)
 	}
-	defer func() {
-		_ = f.Close()
-	}()
+	defer f.Close()
 
 	resp, err := http.Get(src)
 	if err != nil {
 		return fmt.Errorf("failed http call to %s; %w", src, err)
 	}
-	defer func() {
-		_ = resp.Body.Close()
-	}()
+	defer resp.Body.Close()
 
 	archive, err := gzip.NewReader(resp.Body)
 	if err != nil {
@@ -527,9 +523,7 @@ func (d *defaultHasher) hash(src string) (string, error) {
 	if err != nil {
 		return "", err
 	}
-	defer func() {
-		_ = f.Close()
-	}()
+	defer f.Close()
 
 	h := md5.New()
 	if _, err := io.Copy(h, f); err != nil {

internal/native/message_server.go

@@ -232,7 +232,7 @@ func (m *Message) WithContents(part interactionPart, contentType string, body []
 	defer free(cHeader)
 
 	cBody := C.CString(string(body))
-	defer free(cBody)
+    defer free(cBody)
 
 	res := C.pactffi_with_body(m.handle, C.int(part), cHeader, cBody)
 	log.Println("[DEBUG] response from pactffi_interaction_contents", (bool(res)))
@@ -264,7 +264,7 @@ func (m *MessageServer) UsingPlugin(pluginName string, pluginVersion string) err
 		return ErrHandleNotFound
 	default:
 		if res != 0 {
-			return fmt.Errorf("an unknown error (code: %v) occurred when adding a plugin for the test. Received error code", res)
+			return fmt.Errorf("an unknown error (code: %v) occurred when adding a plugin for the test. Received error code:", res)
 		}
 	}
 
@@ -302,7 +302,7 @@ func (m *Message) WithPluginInteractionContents(part interactionPart, contentTyp
 		return ErrPluginSpecificError
 	default:
 		if res != 0 {
-			return fmt.Errorf("an unknown error (code: %v) occurred when adding a plugin for the test. Received error code", res)
+			return fmt.Errorf("an unknown error (code: %v) occurred when adding a plugin for the test. Received error code:", res)
 		}
 	}
 
@@ -543,7 +543,10 @@ func (m *MessageServer) WritePactFile(dir string, overwrite bool) error {
 	cDir := C.CString(dir)
 	defer free(cDir)
 
-	overwritePact := overwrite
+	overwritePact := false
+	if overwrite {
+		overwritePact = true
+	}
 
 	res := int(C.pactffi_write_message_pact_file(m.messagePact.handle, cDir, C.bool(overwritePact)))
 
@@ -569,7 +572,10 @@ func (m *MessageServer) WritePactFileForServer(port int, dir string, overwrite b
 	cDir := C.CString(dir)
 	defer free(cDir)
 
-	overwritePact := overwrite
+	overwritePact := false
+	if overwrite {
+		overwritePact = true
+	}
 
 	res := int(C.pactffi_write_pact_file(C.int(port), cDir, C.bool(overwritePact)))
 
@@ -590,19 +596,3 @@ func (m *MessageServer) WritePactFileForServer(port int, dir string, overwrite b
 		return fmt.Errorf("an unknown error ocurred when writing to pact file")
 	}
 }
-
-// WithReference records an external reference (e.g. a ticket or pull request)
-// against the interaction. References are stored under comments.references[group][name]
-// in the Pact file. This is a V4-only feature.
-func (m *Message) WithReference(group, name, value string) *Message {
-	cGroup := C.CString(group)
-	defer free(cGroup)
-	cName := C.CString(name)
-	defer free(cName)
-	cValue := C.CString(value)
-	defer free(cValue)
-
-	C.pactffi_add_interaction_reference(m.handle, cGroup, cName, cValue)
-
-	return m
-}