mirror of
https://github.com/d0zingcat/gocryptotrader.git
synced 2026-05-14 15:09:51 +00:00
* Bitstamp: Add auth channel subscription handling * Bitstamp: Avoid searching for asset type We've hardcoded asset.Spot in order to find the pair. Looking the asset up from the pair makes no sense. * Bitstamp: Add type for wsOrders * Bitstamp: Working test of Generic DataHandler * Bitstamp: WS Order chan tests and remove type orderType could be derived from status == New and Buy & price == 9+e9 or Sell & price == 0 But it would only be true for the first update and it really doesn't feel worth the risk Consumers are going to have to merge to original request anyway * Bitstamp: Linter fixes * Kraken: Switch to shared fixture test * Bitstamp: Fix lint on TestFixtureToDataHandler * Engine: Add Clone for PairsManager go-vet highlighted that the mutex here is a value when we copied the PairsManager in a test. Options to fix: * Add a deep clone method to PairsManager * Add a shallow clone method with a disclaimer * Make the mutex a pointer * Make the PairsManager itself a pointer Options 3 and 4 are too invasive to justify changing at this point. There's an inherent risk of PM being passed by value, but govet should catch the copylock. There's more risk in changing everything to use a pointer at this stage. * Engine: Fix linter again, ironically * Bitstamp: Rename OHLC const * Bitstamp: Minor fixes to syntax * Bitstamp: Simplify chanSymb=>pair * Bitstamp: Still process order updates without ID If there's a ClientOrderID we'll still process the order. It doesn't seem likely we'd have this happen, but if it does we still want consumers to get something. * Bitstamp: Replace Clone with Lock methods * Engine: Expose PairsManager's Mutex Makes more sense than wrapping functions * Bitstamp: Fix linter copylock (again) * fixup! Engine: Expose PairsManager's Mutex Omit Mutex from Json * fixup! Bitstamp: Add auth channel subscription handling Remove unused wsAuthToken * Bitstamp: Simplify OrderData Unmarshal * Bitstamp: Remove unused contexts I added these following best practices, but the reality is that when/if we get context awareness in GCT, there will be a lot more to fix and this will be a drop in the ocean anyway. * Bitstamp: Only call handleWSOrder for MyOrders * Bitstamp: Avoid allocating again in handleWSOrder * CurrencyPairs: Remove public mutex Simplified to a Load method to avoid making mutex public * Tests: Improve test readability and clarity * Bitstamp: Wrap errWSPairParsingError Co-authored-by: Scott <gloriousCode@users.noreply.github.com> * Bitstamp: FetchWSAuth mock and live test --------- Co-authored-by: Scott <gloriousCode@users.noreply.github.com>
185 lines
6.2 KiB
Go
185 lines
6.2 KiB
Go
package sharedtestvalues
|
|
|
|
import (
|
|
"bufio"
|
|
"bytes"
|
|
"fmt"
|
|
"os"
|
|
"path/filepath"
|
|
"regexp"
|
|
"strings"
|
|
"sync"
|
|
"testing"
|
|
"time"
|
|
|
|
"github.com/stretchr/testify/assert"
|
|
exchange "github.com/thrasher-corp/gocryptotrader/exchanges"
|
|
"github.com/thrasher-corp/gocryptotrader/exchanges/stream"
|
|
)
|
|
|
|
// This package is only to be referenced in test files
|
|
const (
|
|
// WebsocketResponseDefaultTimeout used in websocket testing
|
|
// Defines wait time for receiving websocket response before cancelling
|
|
WebsocketResponseDefaultTimeout = (3 * time.Second)
|
|
// WebsocketResponseExtendedTimeout used in websocket testing
|
|
// Defines wait time for receiving websocket response before cancelling
|
|
WebsocketResponseExtendedTimeout = (15 * time.Second)
|
|
// WebsocketChannelOverrideCapacity used in websocket testing
|
|
// Defines channel capacity as defaults size can block tests
|
|
WebsocketChannelOverrideCapacity = 500
|
|
|
|
MockTesting = "Mock testing framework in use for %s exchange on REST endpoints only"
|
|
LiveTesting = "Mock testing bypassed; live testing of REST endpoints in use for %s exchange"
|
|
|
|
warningSkip = "Skipping test"
|
|
warningKeys = "API test keys have not been set"
|
|
warningManipulateOrders = "variable `canManipulateRealOrders` is false"
|
|
warningHowTo = "these values can be set at the top of the test file."
|
|
)
|
|
|
|
// GetWebsocketInterfaceChannelOverride returns a new interface based channel
|
|
// with the capacity set to WebsocketChannelOverrideCapacity
|
|
func GetWebsocketInterfaceChannelOverride() chan interface{} {
|
|
return make(chan interface{}, WebsocketChannelOverrideCapacity)
|
|
}
|
|
|
|
// GetWebsocketStructChannelOverride returns a new struct based channel
|
|
// with the capacity set to WebsocketChannelOverrideCapacity
|
|
func GetWebsocketStructChannelOverride() chan struct{} {
|
|
return make(chan struct{}, WebsocketChannelOverrideCapacity)
|
|
}
|
|
|
|
// NewTestWebsocket returns a test websocket object
|
|
func NewTestWebsocket() *stream.Websocket {
|
|
return &stream.Websocket{
|
|
Init: true,
|
|
DataHandler: make(chan interface{}, WebsocketChannelOverrideCapacity),
|
|
ToRoutine: make(chan interface{}, 1000),
|
|
TrafficAlert: make(chan struct{}),
|
|
ReadMessageErrors: make(chan error),
|
|
Subscribe: make(chan []stream.ChannelSubscription, 10),
|
|
Unsubscribe: make(chan []stream.ChannelSubscription, 10),
|
|
Match: stream.NewMatch(),
|
|
}
|
|
}
|
|
|
|
// SkipTestIfCredentialsUnset is a test helper function checking if the
|
|
// authenticated function can perform the required test.
|
|
func SkipTestIfCredentialsUnset(t *testing.T, exch exchange.IBotExchange, canManipulateOrders ...bool) {
|
|
t.Helper()
|
|
|
|
if len(canManipulateOrders) > 1 {
|
|
t.Fatal("more than one canManipulateOrders boolean value has been supplied, please remove")
|
|
}
|
|
|
|
areTestAPICredentialsSet := AreAPICredentialsSet(exch)
|
|
supportsManipulatingOrders := len(canManipulateOrders) > 0
|
|
allowedToManipulateOrders := supportsManipulatingOrders && canManipulateOrders[0]
|
|
|
|
if (areTestAPICredentialsSet && !supportsManipulatingOrders) ||
|
|
(areTestAPICredentialsSet && allowedToManipulateOrders) {
|
|
return
|
|
}
|
|
|
|
message := []string{warningSkip}
|
|
if !areTestAPICredentialsSet {
|
|
message = append(message, warningKeys)
|
|
}
|
|
|
|
if supportsManipulatingOrders && !allowedToManipulateOrders {
|
|
message = append(message, warningManipulateOrders)
|
|
}
|
|
message = append(message, warningHowTo)
|
|
t.Skip(strings.Join(message, ", "))
|
|
}
|
|
|
|
// SkipTestIfCannotManipulateOrders will only skip if the credentials are set
|
|
// correctly and can manipulate orders is set to false. It will continue normal
|
|
// operations if credentials are not set, giving better code coverage.
|
|
func SkipTestIfCannotManipulateOrders(t *testing.T, exch exchange.IBotExchange, canManipulateOrders bool) {
|
|
t.Helper()
|
|
|
|
if !AreAPICredentialsSet(exch) || canManipulateOrders {
|
|
return
|
|
}
|
|
|
|
t.Skip(warningSkip + ", " + warningManipulateOrders)
|
|
}
|
|
|
|
// AreAPICredentialsSet returns if the API credentials are set.
|
|
func AreAPICredentialsSet(exch exchange.IBotExchange) bool {
|
|
return exch.VerifyAPICredentials(exch.GetDefaultCredentials()) == nil
|
|
}
|
|
|
|
// EmptyStringPotentialPattern is a regular expression pattern for a potential
|
|
// empty string into float64
|
|
var EmptyStringPotentialPattern = `.*float64.*json:"[^"]*,string".*`
|
|
|
|
// ForceFileStandard will check all files in the current directory for a regular
|
|
// expression pattern. If the pattern is found the test will fail.
|
|
func ForceFileStandard(t *testing.T, pattern string) error {
|
|
t.Helper()
|
|
|
|
r := regexp.MustCompile(pattern)
|
|
|
|
root := "." // Specify the root directory to start walking from
|
|
err := filepath.Walk(root, func(path string, info os.FileInfo, err error) error {
|
|
if err != nil {
|
|
return err
|
|
}
|
|
|
|
if !info.IsDir() && strings.HasSuffix(path, ".go") {
|
|
fileContents, err := os.ReadFile(path)
|
|
if err != nil {
|
|
t.Fatalf("Failed to read file: %v", err)
|
|
}
|
|
|
|
lines := bytes.Split(fileContents, []byte("\n"))
|
|
for x, line := range lines {
|
|
if r.Match(line) {
|
|
t.Errorf("File: %s line contains pattern [%s] match with [%s] at line %d", path, pattern, string(line), x+1)
|
|
}
|
|
}
|
|
}
|
|
return nil
|
|
})
|
|
|
|
if err != nil {
|
|
return fmt.Errorf("failed to walk directory: %w", err)
|
|
}
|
|
return nil
|
|
}
|
|
|
|
// TestFixtureToDataHandler takes a new empty exchange and configures a new websocket handler for it, and squirts the json path contents to it
|
|
// It accepts a reader function, which is probably e.wsHandleData but could be anything
|
|
func TestFixtureToDataHandler(t *testing.T, seed, e exchange.IBotExchange, fixturePath string, reader func([]byte) error) {
|
|
b := e.GetBase()
|
|
seedBase := seed.GetBase()
|
|
|
|
err := b.CurrencyPairs.Load(&seedBase.CurrencyPairs)
|
|
assert.NoError(t, err, "Loading currency pairs should not error")
|
|
|
|
b.Name = "fixture"
|
|
b.Websocket = &stream.Websocket{
|
|
Wg: new(sync.WaitGroup),
|
|
DataHandler: make(chan interface{}, 128),
|
|
}
|
|
b.API.Endpoints = b.NewEndpoints()
|
|
|
|
fixture, err := os.Open(fixturePath)
|
|
assert.NoError(t, err, "Opening fixture '%s' should not error", fixturePath)
|
|
defer func() {
|
|
assert.NoError(t, fixture.Close(), "Closing the fixture file should not error")
|
|
}()
|
|
|
|
s := bufio.NewScanner(fixture)
|
|
for s.Scan() {
|
|
msg := s.Bytes()
|
|
if err := reader(msg); err != nil {
|
|
t.Errorf("%v from message: %s", err, msg)
|
|
}
|
|
}
|
|
assert.NoError(t, s.Err(), "Fixture Scanner should not error")
|
|
}
|