6165158762
60%

Ran 12 Sep 2023 09:44PM UTC

Jobs 2

Files 116

Run time 4s

Badge

Embed ▾

Committed 12 Sep 2023 09:40PM UTC coverage: 75.761% (+0.01%) from 75.748%

Build # 6165158762

Build Type

Pull #2791

github

Committed by

web-flow

Commit Message

Allow WebsocketInitFunc to add payload to Ack (#4)

* Allow WebsocketInitFunc to add payload to Ack

The connection ACK message in the protocol for both
graphql-ws and graphql-transport-ws allows for a payload in the
connection ack message.

We really wanted to use this to establish better telemetry in our use of
websockets in graphql.

* Fix lint error in test

* Switch argument ordering.

---------

Co-authored-by: Chris Pride <cpride@observeinc.com>

Pull Request Pull Request #2791: Allow WebsocketInitFunc to add payload to Ack (#4)

Run Details

11 of 11 new or added lines in 1 file covered. (100.0%)

6767 of 8932 relevant lines covered (75.76%)

1408.23 hits per line

New Missed Lines in Diff

Lines	Coverage	∆	File
1	86.71	0.07%	graphql/handler/transport/websocket.go

Jobs

ID	Job ID	Ran	Files	Coverage
1	6165158762.1	12 Sep 2023 09:44PM UTC	116	75.67	GitHub Action Run
2	6165158762.2	12 Sep 2023 09:44PM UTC	107	79.3	GitHub Action Run

99designs / gqlgen / 6165158762
60%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

New Missed Lines in Diff

Jobs

Source Files on build 6165158762

99designs / gqlgen / 6165158762 60%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

New Missed Lines in Diff

Jobs

Source Files on build 6165158762

99designs / gqlgen / 6165158762
60%

README BADGES
x