add OpenFIGI adaptar with tests

internal/open_figi.go

@@ -6,28 +6,23 @@ import (
 	"encoding/json"
 	"fmt"
 	"net/http"
-	"time"
 )
 
 type OpenFIGI struct {
-	client http.Client
-
-	cache map[string]string
+	client *http.Client
 }
 
-func NewOpenFIGI() *OpenFIGI {
+func NewOpenFIGI(c *http.Client) *OpenFIGI {
 	return &OpenFIGI{
-		client: http.Client{
-			Timeout: 5 * time.Second,
-		},
+		client: c,
 	}
 }
 
-func (of *OpenFIGI) Category(ctx context.Context, isin, ticker string) (string, error) {
-	rawBody, err := json.Marshal(mappingRequestBody{
-		IdType:  "ID_ISIN",
-		IdValue: isin,
-	})
+func (of *OpenFIGI) SecurityTypeByISIN(ctx context.Context, isin string) (string, error) {
+	rawBody, err := json.Marshal([]mappingRequestBody{{
+		IDType:  "ID_ISIN",
+		IDValue: isin,
+	}})
 	if err != nil {
 		return "", fmt.Errorf("marshal mapping request body: %w", err)
 	}
@@ -37,24 +32,46 @@ func (of *OpenFIGI) Category(ctx context.Context, isin, ticker string) (string,
 		return "", fmt.Errorf("create mapping request: %w", err)
 	}
 
+	req.Header.Add("Content-Type", "application/json")
+
 	res, err := of.client.Do(req)
 	if err != nil {
 		return "", fmt.Errorf("make mapping request: %w", err)
 	}
 	defer res.Body.Close()
 
-	var resBody mappingResponseBody
+	if res.StatusCode >= 400 {
+		return "", fmt.Errorf("bad mapping response status code: %s", res.Status)
+	}
+
+	var resBody []mappingResponseBody
 	err = json.NewDecoder(res.Body).Decode(&resBody)
 	if err != nil {
 		return "", fmt.Errorf("unmarshal response: %w", err)
 	}
 
-	return "", nil
+	if len(resBody) == 0 {
+		return "", fmt.Errorf("missing top-level elements")
+	}
+
+	if len(resBody[0].Data) == 0 {
+		return "", fmt.Errorf("missing data elements")
+	}
+
+	// It is not possible that an isin is assign to diferent security types, therefore we can assume
+	// all entries have the same securityType value.
+	return resBody[0].Data[0].SecurityType, nil
 }
 
 type mappingRequestBody struct {
-	IdType  string `json:"idType"`
-	IdValue string `json:"idValue"`
+	IDType  string `json:"idType"`
+	IDValue string `json:"idValue"`
 }
 
-type mappingResponseBody struct{}
+type mappingResponseBody struct {
+	Data []struct {
+		FIGI         string `json:"figi"`
+		SecurityType string `json:"securityType"`
+		Ticker       string `json:"ticker"`
+	} `json:"data"`
+}

internal/open_figi_test.go

@@ -0,0 +1,101 @@
+package internal_test
+
+import (
+	"bytes"
+	"context"
+	"io"
+	"net/http"
+	"testing"
+	"time"
+
+	"github.com/nmoniz/any2anexoj/internal"
+)
+
+func TestOpenFIGI_SecurityTypeByISIN(t *testing.T) {
+	tests := []struct {
+		name     string // description of this test case
+		response *http.Response
+		isin     string
+		want     string
+		wantErr  bool
+	}{
+		{
+			name: "all good",
+			response: &http.Response{
+				Status:     http.StatusText(http.StatusOK),
+				StatusCode: http.StatusOK,
+				Body:       io.NopCloser(bytes.NewBufferString(`[{"data":[{"figi":"BBG000BJJR23","name":"AIRBUS SE","ticker":"EADSF","exchCode":"US","compositeFIGI":"BBG000BJJR23","securityType":"Common Stock","marketSector":"Equity","shareClassFIGI":"BBG001S8TFZ6","securityType2":"Common Stock","securityDescription":"EADSF"},{"figi":"BBG000BJJXJ2","name":"AIRBUS SE","ticker":"EADSF","exchCode":"PQ","compositeFIGI":"BBG000BJJR23","securityType":"Common Stock","marketSector":"Equity","shareClassFIGI":"BBG001S8TFZ6","securityType2":"Common Stock","securityDescription":"EADSF"}]}]`)),
+			},
+			isin: "NL0000235190",
+			want: "Common Stock",
+		},
+		{
+			name: "bas status code",
+			response: &http.Response{
+				Status:     http.StatusText(http.StatusTooManyRequests),
+				StatusCode: http.StatusTooManyRequests,
+			},
+			isin:    "NL0000235190",
+			wantErr: true,
+		},
+		{
+			name: "empty top-level",
+			response: &http.Response{
+				Status:     http.StatusText(http.StatusOK),
+				StatusCode: http.StatusOK,
+				Body:       io.NopCloser(bytes.NewBufferString(`[]`)),
+			},
+			isin:    "NL0000235190",
+			wantErr: true,
+		},
+		{
+			name: "empty data elements",
+			response: &http.Response{
+				Status:     http.StatusText(http.StatusOK),
+				StatusCode: http.StatusOK,
+				Body:       io.NopCloser(bytes.NewBufferString(`[{"data":[]}]`)),
+			},
+			isin:    "NL0000235190",
+			wantErr: true,
+		},
+	}
+	for _, tt := range tests {
+		t.Run(tt.name, func(t *testing.T) {
+			c := NewTestClient(t, func(req *http.Request) (*http.Response, error) {
+				return tt.response, nil
+			})
+
+			of := internal.NewOpenFIGI(c)
+
+			got, gotErr := of.SecurityTypeByISIN(context.Background(), tt.isin)
+			if gotErr != nil {
+				if !tt.wantErr {
+					t.Errorf("want success but failed: %v", gotErr)
+				}
+				return
+			}
+			if tt.wantErr {
+				t.Fatal("want error but none")
+			}
+
+			if tt.want != got {
+				t.Fatalf("want security type to be %s but got %s", tt.want, got)
+			}
+		})
+	}
+}
+
+type RoundTripFunc func(req *http.Request) (*http.Response, error)
+
+func (f RoundTripFunc) RoundTrip(req *http.Request) (*http.Response, error) {
+	return f(req)
+}
+
+func NewTestClient(t testing.TB, fn RoundTripFunc) *http.Client {
+	t.Helper()
+
+	return &http.Client{
+		Timeout:   time.Second,
+		Transport: fn,
+	}
+}