Lints, more tests

protocol/daemons/slinky/client/client_test.go

@@ -33,7 +33,6 @@ type ClientTestSuite struct {
 	daemonFlags           daemonflags.DaemonFlags
 	appFlags              appflags.Flags
 	daemonServer          *daemonserver.Server
-	exchangePriceCache    *pricefeed_types.MarketToExchangePrices
 	grpcServer            *grpc.Server
 	pricesMockQueryServer *mocks.QueryServer
 	wg                    sync.WaitGroup

protocol/daemons/slinky/client/market_pair_fetcher_test.go

@@ -88,5 +88,4 @@ func TestMarketPairFetcher(t *testing.T) {
 		err := fetcher.FetchIdMappings(context.Background())
 		require.Error(t, err, "test error")
 	})
-
 }

protocol/daemons/slinky/client/price_fetcher_test.go

@@ -31,12 +31,11 @@ func TestPriceFetcherTestSuite(t *testing.T) {
 
 type PriceFetcherTestSuite struct {
 	suite.Suite
-	daemonFlags        daemonflags.DaemonFlags
-	appFlags           appflags.Flags
-	daemonServer       *daemonserver.Server
-	exchangePriceCache *pricefeed_types.MarketToExchangePrices
-	pricesGrpcServer   *grpc.Server
-	wg                 sync.WaitGroup
+	daemonFlags      daemonflags.DaemonFlags
+	appFlags         appflags.Flags
+	daemonServer     *daemonserver.Server
+	pricesGrpcServer *grpc.Server
+	wg               sync.WaitGroup
 }
 
 func (p *PriceFetcherTestSuite) SetupTest() {
@@ -189,7 +188,9 @@ func (p *PriceFetcherTestSuite) TestPriceFetcher() {
 			logger,
 		)
 		p.Require().NoError(fetcher.Start(context.Background()))
-		p.Require().Errorf(fetcher.FetchPrices(context.Background()), "slinky client returned price %s not parsable as uint64", "abc123")
+		p.Require().Errorf(
+			fetcher.FetchPrices(context.Background()),
+			"slinky client returned price %s not parsable as uint64", "abc123")
 		fetcher.Stop()
 	})
 

protocol/lib/slinky/utils.go

@@ -2,8 +2,9 @@ package slinky
 
 import (
 	"fmt"
-	"github.com/skip-mev/slinky/x/oracle/types"
 	"strings"
+
+	"github.com/skip-mev/slinky/x/oracle/types"
 )
 
 /*

protocol/lib/slinky/utils_test.go

@@ -0,0 +1,34 @@
+package slinky_test
+
+import (
+	"fmt"
+	"testing"
+
+	"github.com/skip-mev/slinky/x/oracle/types"
+	"github.com/stretchr/testify/require"
+
+	"github.com/dydxprotocol/v4-chain/protocol/lib/slinky"
+)
+
+func TestMarketPairToCurrencyPair(t *testing.T) {
+	testCases := []struct {
+		mp  string
+		cp  types.CurrencyPair
+		err error
+	}{
+		{mp: "FOO-BAR", cp: types.CurrencyPair{Base: "FOO", Quote: "BAR"}, err: nil},
+		{mp: "FOOBAR", cp: types.CurrencyPair{}, err: fmt.Errorf("incorrectly formatted CurrencyPair: FOOBAR")},
+		{mp: "FOO/BAR", cp: types.CurrencyPair{}, err: fmt.Errorf("incorrectly formatted CurrencyPair: FOOBAR")},
+	}
+	for _, tc := range testCases {
+		t.Run(fmt.Sprintf("TestMarketPair %s", tc.mp), func(t *testing.T) {
+			cp, err := slinky.MarketPairToCurrencyPair(tc.mp)
+			if tc.err != nil {
+				require.Error(t, err, tc.err)
+			} else {
+				require.NoError(t, err)
+				require.Equal(t, tc.cp, cp)
+			}
+		})
+	}
+}