diff options
author | Eric Avdey <eiri@eiri.ca> | 2019-02-01 12:39:42 -0400 |
---|---|---|
committer | Eric Avdey <eiri@eiri.ca> | 2019-02-01 12:39:42 -0400 |
commit | 4959bc598b6c9bb2ab2b7972c7606fe8783d1fc8 (patch) | |
tree | 64ed11275697ada8ba6a7644e291a48d51a09d61 | |
parent | 4127726f26fc0e3d7423a058d4c1934a47a464a2 (diff) | |
download | couchdb-4959bc598b6c9bb2ab2b7972c7606fe8783d1fc8.tar.gz |
Make from_json_error_cases tests idiomatic
-rw-r--r-- | src/couch/test/couch_doc_json_tests.erl | 16 |
1 files changed, 6 insertions, 10 deletions
diff --git a/src/couch/test/couch_doc_json_tests.erl b/src/couch/test/couch_doc_json_tests.erl index cc5dc3222..848bae888 100644 --- a/src/couch/test/couch_doc_json_tests.erl +++ b/src/couch/test/couch_doc_json_tests.erl @@ -288,18 +288,14 @@ from_json_error_cases() -> lists:map(fun ({Fun, Expect, Msg}) when is_function(Fun, 0) -> - Error = (catch couch_doc:from_json_obj_validate(Fun())), - {Msg, ?_assertMatch(Expect, Error)}; + {Msg, + ?_assertThrow(Expect, couch_doc:from_json_obj_validate(Fun()))}; ({EJson, Expect, Msg}) -> - Error = (catch couch_doc:from_json_obj_validate(EJson)), - {Msg, ?_assertMatch(Expect, Error)}; + {Msg, + ?_assertThrow(Expect, couch_doc:from_json_obj_validate(EJson))}; ({EJson, Msg}) -> - try - couch_doc:from_json_obj_validate(EJson), - {"Conversion failed to raise an exception", ?_assert(false)} - catch - _:_ -> {Msg, ?_assert(true)} - end + {Msg, + ?_assertThrow(_, couch_doc:from_json_obj_validate(EJson))} end, Cases). from_json_with_dbname_error_cases() -> |