Skip to content

Allow verbosity in websocket testing #1876

Open
@gloriousCode

Description

@gloriousCode

I have to hardcode verbose: true inside stream.NewWebsocket()/websocket.NewManager() because the only way to have a verbose websocket is for the exchange to be verbose. When testing a websocket, the normal path testcode follows is using /testing/exchange/Setup which overrides any local verbosity settings. If you then manually set the exchange to have verbosity, it is too late, the websocket has been setup and you can't set Verbose to true. This makes me sad.

With Shazbert and gk actively changing the websocket, I figure I raise an issue rather than implement it and cause untold rebase issues for them. I hope they will sneak such a change in to suit me. Thanks

Metadata

Metadata

Assignees

No one assigned

    Labels

    low priorityThis enhancement or update will be implemented at a later date.todo

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions