-
Notifications
You must be signed in to change notification settings - Fork 2.8k
add test for subscriptions, improve coverage (fix #191) #193
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
support subsriptions in console graphiql
Fixes [object Object] error parsing for websocket responses.
…to console-test-coverage
server/test/Main.hs
Outdated
|
|
||
| ravenApp :: L.LoggerCtx -> PGQ.PGPool -> IO Application | ||
| ravenApp loggerCtx pool = do | ||
| <<<<<<< HEAD |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Resolve conflicts and commit.
server/test/Main.hs
Outdated
| ravenApp loggerCtx pool = do | ||
| <<<<<<< HEAD | ||
| let corsCfg = CorsConfigG "*" True -- cors is disabled | ||
| ======= |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Here!
server/test/Main.hs
Outdated
| specs <- mkSpecs | ||
| loggerCtx <- L.mkLoggerCtx L.defaultLoggerSettings | ||
| -- run the tests | ||
| <<<<<<< HEAD |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Here!
GitOrigin-RevId: 37c51daf8d5232f0a31fe0c14d99a4ecd75b511b
No description provided.