test: move error verbosity test from io tests to spec tests
Signed-off-by: Taimoor Zaeem <taimoorzaeem@gmail.com>
This commit is contained in:
@@ -2087,38 +2087,6 @@ def test_server_timing_transaction_duration_with_role_statement_timeout(
|
||||
assert 2000 <= response_dur < 3000
|
||||
|
||||
|
||||
def test_client_error_verbosity_config(defaultenv):
|
||||
"Test PostgREST errors with different error verbosity settings"
|
||||
|
||||
env = {
|
||||
**defaultenv,
|
||||
"PGRST_CLIENT_ERROR_VERBOSITY": "minimal", # hide details and hint
|
||||
}
|
||||
|
||||
with run(env=env) as postgrest:
|
||||
response = postgrest.session.get("/itemsx")
|
||||
assert response.status_code == 404
|
||||
assert response.json() == {
|
||||
"code": "PGRST205",
|
||||
"message": "Could not find the table 'public.itemsx' in the schema cache",
|
||||
}
|
||||
|
||||
env = {
|
||||
**defaultenv,
|
||||
"PGRST_CLIENT_ERROR_VERBOSITY": "verbose",
|
||||
}
|
||||
|
||||
with run(env=env) as postgrest:
|
||||
response = postgrest.session.get("/itemsx")
|
||||
assert response.status_code == 404
|
||||
assert response.json() == {
|
||||
"code": "PGRST205",
|
||||
"message": "Could not find the table 'public.itemsx' in the schema cache",
|
||||
"details": None,
|
||||
"hint": "Perhaps you meant the table 'public.items'",
|
||||
}
|
||||
|
||||
|
||||
def test_positive_pool_metric(defaultenv):
|
||||
"When a network failure is caused on the pg connection, pgrst_db_pool_available stays positive"
|
||||
|
||||
|
||||
@@ -5,7 +5,7 @@ import Test.Hspec
|
||||
import Test.Hspec.Wai
|
||||
import Test.Hspec.Wai.JSON
|
||||
|
||||
import PostgREST.Config (AppConfig (..))
|
||||
import PostgREST.Config (AppConfig (..), Verbosity (..))
|
||||
|
||||
import Protolude hiding (get)
|
||||
import SpecHelper
|
||||
@@ -223,3 +223,15 @@ spec withConfig = do
|
||||
"message":"Wrong or unsupported encoding algorithm"
|
||||
}|]
|
||||
{ matchStatus = 401 }
|
||||
|
||||
-- By default, error verbosity is set to 'verbose', which returns all error
|
||||
-- fields. No need to test that explicitly.
|
||||
withConfig baseCfg { configClientErrorVerbosity = Minimal } $ describe "Test client-error-verbosity config" $
|
||||
it "hides details and hint when set to 'minimal'" $
|
||||
request methodGet "/itemsx" [] ""
|
||||
`shouldRespondWith`
|
||||
[json|{
|
||||
"code":"PGRST205",
|
||||
"message":"Could not find the table 'test.itemsx' in the schema cache"
|
||||
}|]
|
||||
{ matchStatus = 404 }
|
||||
|
||||
Reference in New Issue
Block a user