@@ -24,40 +24,57 @@
2424{-# OPTIONS_GHC -fno-warn-name-shadowing #-}
2525
2626#include "overlapping-compat.h"
27−module Servant.ClientSpec where
27+module Servant.ClientSpec (spec) where
2828
29−import Control.Arrow (left)
30−import Control.Concurrent (forkIO, killThread, ThreadId)
31−import Control.Exception (bracket)
32−import Control.Monad.Error.Class (throwError )
29+import Prelude ()
30+import Prelude.Compat
31+
32+import Control.Arrow (left)
33+import Control.Concurrent (ThreadId, forkIO,
34+ killThread)
35+import Control.Exception (bracket)
36+import Control.Monad.Error.Class (throwError)
3337import Data.Aeson
34−import qualified Data.ByteString.Lazy as BS
35−import Data.Char (chr, isPrint)
36−import Data.Foldable (forM_)
37−import Data.Monoid hiding (getLast)
38+import Data.Char (chr, isPrint)
39+import Data.Foldable (forM_)
40+import Data.Monoid hiding (getLast)
3841import Data.Proxy
39−import qualified Generics.SOP as SOP
40−import GHC.Generics (Generic)
41−import qualified Network.HTTP.Client as C
42−import Network.HTTP.Media
43−import qualified Network.HTTP.Types as HTTP
42+import qualified Generics.SOP as SOP
43+import GHC.Generics (Generic)
44+import qualified Network.HTTP.Client as C
45+import qualified Network.HTTP.Types as HTTP
4446import Network.Socket
45−import Network.Wai (Request, requestHeaders, responseLBS)
47+import qualified Network.Wai as Wai
4648import Network.Wai.Handler.Warp
47−import Prelude ()
48−import Prelude.Compat
49−import System.IO.Unsafe (unsafePerformIO)
50−import Test.HUnit
49+import System.IO.Unsafe (unsafePerformIO)
5150import Test.Hspec
5251import Test.Hspec.QuickCheck
52+import Test.HUnit
5353import Test.QuickCheck
54−import Web.FormUrlEncoded (FromForm, ToForm)
55−
56−import Servant.API
54+import Web.FormUrlEncoded (FromForm, ToForm)
55+
56+import Servant.API ((:<|>) ((:<|>)),
57+ (:>), AuthProtect,
58+ BasicAuth,
59+ BasicAuthData (..),
60+ Capture,
61+ CaptureAll, Delete,
62+ DeleteNoContent,
63+ EmptyAPI, addHeader,
64+ FormUrlEncoded,
65+ Get, Header,
66+ Headers, JSON,
67+ NoContent (NoContent),
68+ Post, Put, Raw,
69+ QueryFlag,
70+ QueryParam,
71+ QueryParams,
72+ ReqBody,
73+ getHeaders)
5774import Servant.API.Internal.Test.ComprehensiveAPI
5875import Servant.Client
59−import Servant.Client.Generic
60−import qualified Servant.Common.Req as SCR
76+import qualified Servant.Client.Core.Internal.Request as Req
77+import qualified Servant.Client.Core.Internal.Auth as Auth
6178import Servant.Server
6279import Servant.Server.Experimental.Auth
6380
@@ -75,17 +92,16 @@ spec = describe "Servant.Client" $ do
7592
7693-- * test data types
7794
78−data Person = Person {
79− name :: String,
80− age :: Integer
81− }
82− deriving (Eq, Show, Generic)
95+data Person = Person
96+ { _name :: String
97+ , _age :: Integer
98+ } deriving (Eq, Show, Generic)
8399
84100instance ToJSON Person
85101instance FromJSON Person
86102
87−instance ToForm Person where
88−instance FromForm Person where
103+instance ToForm Person
104+instance FromForm Person
89105
90106alice :: Person
91107alice = Person "Alice" 42
@@ -116,22 +132,20 @@ type Api =
116132api :: Proxy Api
117133api = Proxy
118134
119−getGet :: SCR.ClientM Person
120−getDeleteEmpty :: SCR.ClientM NoContent
121−getCapture :: String -> SCR.ClientM Person
122−getCaptureAll :: [String] -> SCR.ClientM [Person]
123−getBody :: Person -> SCR.ClientM Person
124−getQueryParam :: Maybe String -> SCR.ClientM Person
125−getQueryParams :: [String] -> SCR.ClientM [Person]
126−getQueryFlag :: Bool -> SCR.ClientM Bool
127−getRawSuccess :: HTTP.Method
128− -> SCR.ClientM (Int, BS.ByteString, MediaType, [HTTP.Header], C.Response BS.ByteString)
129−getRawFailure :: HTTP.Method
130− -> SCR.ClientM (Int, BS.ByteString, MediaType, [HTTP.Header], C.Response BS.ByteString)
135+getGet :: ClientM Person
136+getDeleteEmpty :: ClientM NoContent
137+getCapture :: String -> ClientM Person
138+getCaptureAll :: [String] -> ClientM [Person]
139+getBody :: Person -> ClientM Person
140+getQueryParam :: Maybe String -> ClientM Person
141+getQueryParams :: [String] -> ClientM [Person]
142+getQueryFlag :: Bool -> ClientM Bool
143+getRawSuccess :: HTTP.Method -> ClientM Response
144+getRawFailure :: HTTP.Method -> ClientM Response
131145getMultiple :: String -> Maybe Int -> Bool -> [(String, [Rational])]
132− -> SCR.ClientM (String, Maybe Int, Bool, [(String, [Rational])])
133−getRespHeaders :: SCR.ClientM (Headers TestHeaders Bool)
134−getDeleteContentType :: SCR.ClientM NoContent
146+ -> ClientM (String, Maybe Int, Bool, [(String, [Rational])])
147+getRespHeaders :: ClientM (Headers TestHeaders Bool)
148+getDeleteContentType :: ClientM NoContent
135149
136150getGet
137151 :<|> getDeleteEmpty
@@ -161,8 +175,8 @@ server = serve api (
161175 Nothing -> throwError $ ServantErr 400 "missing parameter" "" [])
162176 :<|> (\ names -> return (zipWith Person names [0..]))
163177 :<|> return
164− :<|> (Tagged $ \ _request respond -> respond $ responseLBS HTTP.ok200 [] "rawSuccess")
165− :<|> (Tagged $ \ _request respond -> respond $ responseLBS HTTP.badRequest400 [] "rawFailure")
178+ :<|> (Tagged $ \ _request respond -> respond $ Wai.responseLBS HTTP.ok200 [] "rawSuccess")
179+ :<|> (Tagged $ \ _request respond -> respond $ Wai.responseLBS HTTP.badRequest400 [] "rawFailure")
166180 :<|> (\ a b c d -> return (a, b, c, d))
167181 :<|> (return $ addHeader 1729 $ addHeader "eg2" True)
168182 :<|> return NoContent
@@ -178,9 +192,9 @@ failApi = Proxy
178192
179193failServer :: Application
180194failServer = serve failApi (
181− (Tagged $ \ _request respond -> respond $ responseLBS HTTP.ok200 [] "")
182− :<|> (\ _capture -> Tagged $ \_request respond -> respond $ responseLBS HTTP.ok200 [("content-type", "application/json")] "")
183− :<|> (Tagged $ \_request respond -> respond $ responseLBS HTTP.ok200 [("content-type", "fooooo")] "")
195+ (Tagged $ \ _request respond -> respond $ Wai.responseLBS HTTP.ok200 [] "")
196+ :<|> (\ _capture -> Tagged $ \_request respond -> respond $ Wai.responseLBS HTTP.ok200 [("content-type", "application/json")] "")
197+ :<|> (Tagged $ \_request respond -> respond $ Wai.responseLBS HTTP.ok200 [("content-type", "fooooo")] "")
184198 )
185199
186200-- * basic auth stuff
@@ -214,16 +228,16 @@ genAuthAPI :: Proxy GenAuthAPI
214228genAuthAPI = Proxy
215229
216230type instance AuthServerData (AuthProtect "auth-tag") = ()
217−type instance AuthClientData (AuthProtect "auth-tag") = ()
231+type instance Auth.AuthClientData (AuthProtect "auth-tag") = ()
218232
219−genAuthHandler :: AuthHandler Request ()
233+genAuthHandler :: AuthHandler Wai.Request ()
220234genAuthHandler =
221− let handler req = case lookup "AuthHeader" (requestHeaders req) of
235+ let handler req = case lookup "AuthHeader" (Wai.requestHeaders req) of
222236 Nothing -> throwError (err401 { errBody = "Missing auth header" })
223237 Just _ -> return ()
224238 in mkAuthHandler handler
225239
226−genAuthServerContext :: Context '[ AuthHandler Request () ]
240+genAuthServerContext :: Context '[ AuthHandler Wai.Request () ]
227241genAuthServerContext = genAuthHandler :. EmptyContext
228242
229243genAuthServer :: Application
@@ -236,11 +250,11 @@ type GenericClientAPI
236250 :<|> Capture "foo" String :> NestedAPI1
237251
238252data GenericClient = GenericClient
239− { getSqr :: Maybe Int -> SCR.ClientM Int
253+ { getSqr :: Maybe Int -> ClientM Int
240254 , mkNestedClient1 :: String -> NestedClient1
241255 } deriving Generic
242256instance SOP.Generic GenericClient
243−instance (Client GenericClientAPI ~ client) => ClientLike client GenericClient
257+instance (Client ClientM GenericClientAPI ~ client) => ClientLike client GenericClient
244258
245259type NestedAPI1
246260 = QueryParam "int" Int :> NestedAPI2
@@ -248,21 +262,21 @@ type NestedAPI1
248262
249263data NestedClient1 = NestedClient1
250264 { mkNestedClient2 :: Maybe Int -> NestedClient2
251− , idChar :: Maybe Char -> SCR.ClientM Char
265+ , idChar :: Maybe Char -> ClientM Char
252266 } deriving Generic
253267instance SOP.Generic NestedClient1
254−instance (Client NestedAPI1 ~ client) => ClientLike client NestedClient1
268+instance (Client ClientM NestedAPI1 ~ client) => ClientLike client NestedClient1
255269
256270type NestedAPI2
257271 = "sum" :> Capture "first" Int :> Capture "second" Int :> Get '[JSON] Int
258272 :<|> "void" :> Post '[JSON] ()
259273
260274data NestedClient2 = NestedClient2
261− { getSum :: Int -> Int -> SCR.ClientM Int
262− , doNothing :: SCR.ClientM ()
275+ { getSum :: Int -> Int -> ClientM Int
276+ , doNothing :: ClientM ()
263277 } deriving Generic
264278instance SOP.Generic NestedClient2
265−instance (Client NestedAPI2 ~ client) => ClientLike client NestedClient2
279+instance (Client ClientM NestedAPI2 ~ client) => ClientLike client NestedClient2
266280
267281genericClientServer :: Application
268282genericClientServer = serve (Proxy :: Proxy GenericClientAPI) (
@@ -276,67 +290,70 @@ genericClientServer = serve (Proxy :: Proxy GenericClientAPI) (
276290 nestedServer1 _str = nestedServer2 :<|> (maybe (throwError $ ServantErr 400 "missing parameter" "" []) return)
277291 nestedServer2 _int = (\ x y -> return (x + y)) :<|> return ()
278292
279−{-# NOINLINE manager #-}
280−manager :: C.Manager
281−manager = unsafePerformIO $ C.newManager C.defaultManagerSettings
293+{-# NOINLINE manager' #-}
294+manager' :: C.Manager
295+manager' = unsafePerformIO $ C.newManager C.defaultManagerSettings
296+
297+runClient :: ClientM a -> BaseUrl -> IO (Either ServantError a)
298+runClient x baseUrl' = runClientM x (ClientEnv manager' baseUrl')
282299
283300sucessSpec :: Spec
284301sucessSpec = beforeAll (startWaiApp server) $ afterAll endWaiApp $ do
285302
286303 it "Servant.API.Get" $ \(_, baseUrl) -> do
287− (left show <$> (runClientM getGet (ClientEnv manager baseUrl))) `shouldReturn` Right alice
304+ left show <$> runClient getGet baseUrl `shouldReturn` Right alice
288305
289306 describe "Servant.API.Delete" $ do
290307 it "allows empty content type" $ \(_, baseUrl) -> do
291− (left show <$> (runClientM getDeleteEmpty (ClientEnv manager baseUrl))) `shouldReturn` Right NoContent
308+ left show <$> runClient getDeleteEmpty baseUrl `shouldReturn` Right NoContent
292309
293310 it "allows content type" $ \(_, baseUrl) -> do
294− (left show <$> (runClientM getDeleteContentType (ClientEnv manager baseUrl))) `shouldReturn` Right NoContent
311+ left show <$> runClient getDeleteContentType baseUrl `shouldReturn` Right NoContent
295312
296313 it "Servant.API.Capture" $ \(_, baseUrl) -> do
297− (left show <$> (runClientM (getCapture "Paula") (ClientEnv manager baseUrl))) `shouldReturn` Right (Person "Paula" 0)
314+ left show <$> runClient (getCapture "Paula") baseUrl `shouldReturn` Right (Person "Paula" 0)
298315
299316 it "Servant.API.CaptureAll" $ \(_, baseUrl) -> do
300317 let expected = [(Person "Paula" 0), (Person "Peta" 1)]
301− (left show <$> (runClientM (getCaptureAll ["Paula", "Peta"]) (ClientEnv manager baseUrl))) `shouldReturn` Right expected
318+ left show <$> runClient (getCaptureAll ["Paula", "Peta"]) baseUrl `shouldReturn` Right expected
302319
303320 it "Servant.API.ReqBody" $ \(_, baseUrl) -> do
304321 let p = Person "Clara" 42
305− (left show <$> runClientM (getBody p) (ClientEnv manager baseUrl)) `shouldReturn` Right p
322+ left show <$> runClient (getBody p) baseUrl `shouldReturn` Right p
306323
307324 it "Servant.API.QueryParam" $ \(_, baseUrl) -> do
308− left show <$> runClientM (getQueryParam (Just "alice")) (ClientEnv manager baseUrl) `shouldReturn` Right alice
309− Left FailureResponse{..} <- runClientM (getQueryParam (Just "bob")) (ClientEnv manager baseUrl)
310− responseStatus `shouldBe` HTTP.Status 400 "bob not found"
325+ left show <$> runClient (getQueryParam (Just "alice")) baseUrl `shouldReturn` Right alice
326+ Left (FailureResponse r) <- runClient (getQueryParam (Just "bob")) baseUrl
327+ responseStatusCode r `shouldBe` HTTP.Status 400 "bob not found"
311328
312329 it "Servant.API.QueryParam.QueryParams" $ \(_, baseUrl) -> do
313− (left show <$> runClientM (getQueryParams []) (ClientEnv manager baseUrl)) `shouldReturn` Right []
314− (left show <$> runClientM (getQueryParams ["alice", "bob"]) (ClientEnv manager baseUrl))
330+ left show <$> runClient (getQueryParams []) baseUrl `shouldReturn` Right []
331+ left show <$> runClient (getQueryParams ["alice", "bob"]) baseUrl
315332 `shouldReturn` Right [Person "alice" 0, Person "bob" 1]
316333
317334 context "Servant.API.QueryParam.QueryFlag" $
318335 forM_ [False, True] $ \ flag -> it (show flag) $ \(_, baseUrl) -> do
319− (left show <$> runClientM (getQueryFlag flag) (ClientEnv manager baseUrl)) `shouldReturn` Right flag
336+ left show <$> runClient (getQueryFlag flag) baseUrl `shouldReturn` Right flag
320337
321338 it "Servant.API.Raw on success" $ \(_, baseUrl) -> do
322− res <- runClientM (getRawSuccess HTTP.methodGet) (ClientEnv manager baseUrl)
339+ res <- runClient (getRawSuccess HTTP.methodGet) baseUrl
323340 case res of
324341 Left e -> assertFailure $ show e
325− Right (code, body, ct, _, response) -> do
326− (code, body, ct) `shouldBe` (200, "rawSuccess", "application"//"octet-stream")
327− C.responseBody response `shouldBe` body
328− C.responseStatus response `shouldBe` HTTP.ok200
342+ Right r -> do
343+ responseStatusCode r `shouldBe` HTTP.status200
344+ responseBody r `shouldBe` "rawSuccess"
329345
330346 it "Servant.API.Raw should return a Left in case of failure" $ \(_, baseUrl) -> do
331− res <- runClientM (getRawFailure HTTP.methodGet) (ClientEnv manager baseUrl)
347+ res <- runClient (getRawFailure HTTP.methodGet) baseUrl
332348 case res of
333349 Right _ -> assertFailure "expected Left, but got Right"
334− Left e -> do
335− Servant.Client.responseStatus e `shouldBe` HTTP.status400
336− Servant.Client.responseBody e `shouldBe` "rawFailure"
350+ Left (FailureResponse r) -> do
351+ responseStatusCode r `shouldBe` HTTP.status400
352+ responseBody r `shouldBe` "rawFailure"
353+ Left e -> assertFailure $ "expected FailureResponse, but got " ++ show e
337354
338355 it "Returns headers appropriately" $ \(_, baseUrl) -> do
339− res <- runClientM getRespHeaders (ClientEnv manager baseUrl)
356+ res <- runClient getRespHeaders baseUrl
340357 case res of
341358 Left e -> assertFailure $ show e
342359 Right val -> getHeaders val `shouldBe` [("X-Example1", "1729"), ("X-Example2", "eg2")]
@@ -345,7 +362,7 @@ sucessSpec = beforeAll (startWaiApp server) $ afterAll endWaiApp $ do
345362 it "works for a combination of Capture, QueryParam, QueryFlag and ReqBody" $ \(_, baseUrl) ->
346363 property $ forAllShrink pathGen shrink $ \(NonEmpty cap) num flag body ->
347364 ioProperty $ do
348− result <- left show <$> runClientM (getMultiple cap num flag body) (ClientEnv manager baseUrl)
365+ result <- left show <$> runClient (getMultiple cap num flag body) baseUrl
349366 return $
350367 result === Right (cap, num, flag, body)
351368
@@ -357,10 +374,10 @@ wrappedApiSpec = describe "error status codes" $ do
357374 let test :: (WrappedApi, String) -> Spec
358375 test (WrappedApi api, desc) =
359376 it desc $ bracket (startWaiApp $ serveW api) endWaiApp $ \(_, baseUrl) -> do
360− let getResponse :: SCR.ClientM ()
377+ let getResponse :: ClientM ()
361378 getResponse = client api
362− Left FailureResponse{..} <- runClientM getResponse (ClientEnv manager baseUrl)
363− responseStatus `shouldBe` (HTTP.Status 500 "error message")
379+ Left (FailureResponse r) <- runClient getResponse baseUrl
380+ responseStatusCode r `shouldBe` (HTTP.Status 500 "error message")
364381 in mapM_ test $
365382 (WrappedApi (Proxy :: Proxy (Delete '[JSON] ())), "Delete") :
366383 (WrappedApi (Proxy :: Proxy (Get '[JSON] ())), "Get") :
@@ -374,42 +391,42 @@ failSpec = beforeAll (startWaiApp failServer) $ afterAll endWaiApp $ do
374391 context "client returns errors appropriately" $ do
375392 it "reports FailureResponse" $ \(_, baseUrl) -> do
376393 let (_ :<|> getDeleteEmpty :<|> _) = client api
377− Left res <- runClientM getDeleteEmpty (ClientEnv manager baseUrl)
394+ Left res <- runClient getDeleteEmpty baseUrl
378395 case res of
379− FailureResponse _ (HTTP.Status 404 "Not Found") _ _ -> return ()
396+ FailureResponse r | responseStatusCode r == HTTP.status404 -> return ()
380397 _ -> fail $ "expected 404 response, but got " <> show res
381398
382399 it "reports DecodeFailure" $ \(_, baseUrl) -> do
383400 let (_ :<|> _ :<|> getCapture :<|> _) = client api
384− Left res <- runClientM (getCapture "foo") (ClientEnv manager baseUrl)
401+ Left res <- runClient (getCapture "foo") baseUrl
385402 case res of
386− DecodeFailure _ ("application/json") _ -> return ()
403+ DecodeFailure _ _ -> return ()
387404 _ -> fail $ "expected DecodeFailure, but got " <> show res
388405
389406 it "reports ConnectionError" $ \_ -> do
390407 let (getGetWrongHost :<|> _) = client api
391− Left res <- runClientM getGetWrongHost (ClientEnv manager (BaseUrl Http "127.0.0.1" 19872 ""))
408+ Left res <- runClient getGetWrongHost (BaseUrl Http "127.0.0.1" 19872 "")
392409 case res of
393410 ConnectionError _ -> return ()
394411 _ -> fail $ "expected ConnectionError, but got " <> show res
395412
396413 it "reports UnsupportedContentType" $ \(_, baseUrl) -> do
397414 let (getGet :<|> _ ) = client api
398− Left res <- runClientM getGet (ClientEnv manager baseUrl)
415+ Left res <- runClient getGet baseUrl
399416 case res of
400417 UnsupportedContentType ("application/octet-stream") _ -> return ()
401418 _ -> fail $ "expected UnsupportedContentType, but got " <> show res
402419
403420 it "reports InvalidContentTypeHeader" $ \(_, baseUrl) -> do
404421 let (_ :<|> _ :<|> _ :<|> _ :<|> getBody :<|> _) = client api
405− Left res <- runClientM (getBody alice) (ClientEnv manager baseUrl)
422+ Left res <- runClient (getBody alice) baseUrl
406423 case res of
407− InvalidContentTypeHeader "fooooo" _ -> return ()
424+ InvalidContentTypeHeader _ -> return ()
408425 _ -> fail $ "expected InvalidContentTypeHeader, but got " <> show res
409426
410427data WrappedApi where
411428 WrappedApi :: (HasServer (api :: *) '[], Server api ~ Handler a,
412− HasClient api, Client api ~ SCR.ClientM ()) =>
429+ HasClient ClientM api, Client ClientM api ~ ClientM ()) =>
413430 Proxy api -> WrappedApi
414431
415432basicAuthSpec :: Spec
@@ -419,15 +436,15 @@ basicAuthSpec = beforeAll (startWaiApp basicAuthServer) $ afterAll endWaiApp $ d
419436 it "Authenticates a BasicAuth protected server appropriately" $ \(_,baseUrl) -> do
420437 let getBasic = client basicAuthAPI
421438 let basicAuthData = BasicAuthData "servant" "server"
422− (left show <$> runClientM (getBasic basicAuthData) (ClientEnv manager baseUrl)) `shouldReturn` Right alice
439+ left show <$> runClient (getBasic basicAuthData) baseUrl `shouldReturn` Right alice
423440
424441 context "Authentication is rejected when requests are not authenticated properly" $ do
425442
426443 it "Authenticates a BasicAuth protected server appropriately" $ \(_,baseUrl) -> do
427444 let getBasic = client basicAuthAPI
428445 let basicAuthData = BasicAuthData "not" "password"
429− Left FailureResponse{..} <- runClientM (getBasic basicAuthData) (ClientEnv manager baseUrl)
430− responseStatus `shouldBe` HTTP.Status 403 "Forbidden"
446+ Left (FailureResponse r) <- runClient (getBasic basicAuthData) baseUrl
447+ responseStatusCode r `shouldBe` HTTP.Status 403 "Forbidden"
431448
432449genAuthSpec :: Spec
433450genAuthSpec = beforeAll (startWaiApp genAuthServer) $ afterAll endWaiApp $ do
@@ -435,16 +452,16 @@ genAuthSpec = beforeAll (startWaiApp genAuthServer) $ afterAll endWaiApp $ do
435452
436453 it "Authenticates a AuthProtect protected server appropriately" $ \(_, baseUrl) -> do
437454 let getProtected = client genAuthAPI
438− let authRequest = mkAuthenticateReq () (\_ req -> SCR.addHeader "AuthHeader" ("cool" :: String) req)
439− (left show <$> runClientM (getProtected authRequest) (ClientEnv manager baseUrl)) `shouldReturn` Right alice
455+ let authRequest = Auth.mkAuthenticatedRequest () (\_ req -> Req.addHeader "AuthHeader" ("cool" :: String) req)
456+ left show <$> runClient (getProtected authRequest) baseUrl `shouldReturn` Right alice
440457
441458 context "Authentication is rejected when requests are not authenticated properly" $ do
442459
443460 it "Authenticates a AuthProtect protected server appropriately" $ \(_, baseUrl) -> do
444461 let getProtected = client genAuthAPI
445− let authRequest = mkAuthenticateReq () (\_ req -> SCR.addHeader "Wrong" ("header" :: String) req)
446− Left FailureResponse{..} <- runClientM (getProtected authRequest) (ClientEnv manager baseUrl)
447− responseStatus `shouldBe` (HTTP.Status 401 "Unauthorized")
462+ let authRequest = Auth.mkAuthenticatedRequest () (\_ req -> Req.addHeader "Wrong" ("header" :: String) req)
463+ Left (FailureResponse r) <- runClient (getProtected authRequest) baseUrl
464+ responseStatusCode r `shouldBe` (HTTP.Status 401 "Unauthorized")
448465
449466genericClientSpec :: Spec
450467genericClientSpec = beforeAll (startWaiApp genericClientServer) $ afterAll endWaiApp $ do
@@ -454,13 +471,13 @@ genericClientSpec = beforeAll (startWaiApp genericClientServer) $ afterAll endWa
454471 NestedClient1{..} = mkNestedClient1 "example"
455472 NestedClient2{..} = mkNestedClient2 (Just 42)
456473
457− it "works for top-level client function" $ \(_, baseUrl) -> do
458− (left show <$> (runClientM (getSqr (Just 5)) (ClientEnv manager baseUrl))) `shouldReturn` Right 25
474+ it "works for top-level client inClientM function" $ \(_, baseUrl) -> do
475+ left show <$> runClient (getSqr (Just 5)) baseUrl `shouldReturn` Right 25
459476
460477 it "works for nested clients" $ \(_, baseUrl) -> do
461− (left show <$> (runClientM (idChar (Just 'c')) (ClientEnv manager baseUrl))) `shouldReturn` Right 'c'
462− (left show <$> (runClientM (getSum 3 4) (ClientEnv manager baseUrl))) `shouldReturn` Right 7
463− (left show <$> (runClientM doNothing (ClientEnv manager baseUrl))) `shouldReturn` Right ()
478+ left show <$> runClient (idChar (Just 'c')) baseUrl `shouldReturn` Right 'c'
479+ left show <$> runClient (getSum 3 4) baseUrl `shouldReturn` Right 7
480+ left show <$> runClient doNothing baseUrl `shouldReturn` Right ()
464481
465482-- * utils
466483