server: add test cases for config using combinators

servant-server/test/Servant/Server/UsingConfigSpec.hs

@@ -13,27 +13,64 @@ import           Servant.Server.UsingConfigSpec.CustomCombinator
 
 -- * API
 
-type API =
+data Tag1
-  CustomCombinator :> Get '[JSON] String
+data Tag2
 
-api :: Proxy API
+type OneEntryAPI =
-api = Proxy
+  CustomCombinator Tag1 :> Get '[JSON] String
 
-testServer :: Server API
+testServer :: Server OneEntryAPI
 testServer (CustomConfig s) = return s
 
-app :: Application
+oneEntryApp :: Application
-app =
+oneEntryApp =
-  serve api config testServer
+  serve (Proxy :: Proxy OneEntryAPI) config testServer
   where
-    config :: Config '[ConfigEntry Tag CustomConfig]
+    config :: Config '[ConfigEntry Tag1 CustomConfig]
     config = CustomConfig "configValue" .:. EmptyConfig
 
+type OneEntryTwiceAPI =
+  "foo" :> CustomCombinator Tag1 :> Get '[JSON] String :<|>
+  "bar" :> CustomCombinator Tag1 :> Get '[JSON] String
+
+oneEntryTwiceApp :: Application
+oneEntryTwiceApp = serve (Proxy :: Proxy OneEntryTwiceAPI) config $
+  testServer :<|>
+  testServer
+  where
+    config :: Config '[ConfigEntry Tag1 CustomConfig]
+    config = CustomConfig "configValueTwice" .:. EmptyConfig
+
+type TwoDifferentEntries =
+  "foo" :> CustomCombinator Tag1 :> Get '[JSON] String :<|>
+  "bar" :> CustomCombinator Tag2 :> Get '[JSON] String
+
+twoDifferentEntries :: Application
+twoDifferentEntries = serve (Proxy :: Proxy TwoDifferentEntries) config $
+  testServer :<|>
+  testServer
+  where
+    config :: Config '[ConfigEntry Tag1 CustomConfig, ConfigEntry Tag2 CustomConfig]
+    config =
+      CustomConfig "firstConfigValue" .:.
+      CustomConfig "secondConfigValue" .:.
+      EmptyConfig
+
 -- * tests
 
 spec :: Spec
 spec = do
   describe "using Config in a custom combinator" $ do
-    with (return app) $ do
+    with (return oneEntryApp) $ do
-      it "allows to retrieve the ConfigEntry" $ do
+      it "allows to retrieve a ConfigEntry" $ do
         get "/" `shouldRespondWith` "\"configValue\""
+
+    with (return oneEntryTwiceApp) $ do
+      it "allows to retrieve the same ConfigEntry twice" $ do
+        get "/foo" `shouldRespondWith` "\"configValueTwice\""
+        get "/bar" `shouldRespondWith` "\"configValueTwice\""
+
+    with (return twoDifferentEntries) $ do
+      it "allows to retrieve different ConfigEntries for the same combinator" $ do
+        get "/foo" `shouldRespondWith` "\"firstConfigValue\""
+        get "/bar" `shouldRespondWith` "\"secondConfigValue\""

servant-server/test/Servant/Server/UsingConfigSpec/CustomCombinator.hs

@@ -13,29 +13,22 @@
 -- the config.
 module Servant.Server.UsingConfigSpec.CustomCombinator where
 
--- import           Network.Wai
--- import           Test.Hspec (Spec, describe, it)
--- import           Test.Hspec.Wai
-
 import           Servant
 import           Servant.Server.Internal.Config
--- import           Servant.Server.Internal.Router
 import           Servant.Server.Internal.RoutingApplication
 
-data CustomCombinator
+data CustomCombinator (tag :: *)
 
 data CustomConfig = CustomConfig String
 
-data Tag
+instance forall subApi (c :: [*]) tag .
-
-instance forall subApi (c :: [*]) .
   (HasServer subApi) =>
-  HasServer (CustomCombinator :> subApi) where
+  HasServer (CustomCombinator tag :> subApi) where
 
-  type ServerT (CustomCombinator :> subApi) m =
+  type ServerT (CustomCombinator tag :> subApi) m =
     CustomConfig -> ServerT subApi m
-  type HasCfg (CustomCombinator :> subApi) c =
+  type HasCfg (CustomCombinator tag :> subApi) c =
-    (HasConfigEntry c Tag CustomConfig, HasCfg subApi c)
+    (HasConfigEntry c tag CustomConfig, HasCfg subApi c)
 
   route Proxy config delayed =
     route subProxy config (fmap (inject config) delayed :: Delayed (Server subApi))
@@ -43,5 +36,5 @@ instance forall subApi (c :: [*]) .
       subProxy :: Proxy subApi
       subProxy = Proxy
 
-      inject config f = f (getConfigEntry (Proxy :: Proxy Tag) config)
+      inject config f = f (getConfigEntry (Proxy :: Proxy tag) config)