diff options
| author | Alexander Shorin <kxepal@apache.org> | 2014-04-29 17:51:15 +0400 |
|---|---|---|
| committer | Dirkjan Ochtman <dirkjan@ochtman.nl> | 2014-04-29 16:03:36 +0200 |
| commit | cf3e19f88d224a828cef96cc1155c6c23484bc22 (patch) | |
| tree | cda6b3ddde5944e658025c841ef5c071e5837d10 | |
| parent | 26cff2d51473cbc8c0dfc655b2dbb986493bbcb8 (diff) | |
| download | couchdb-cf3e19f88d224a828cef96cc1155c6c23484bc22.tar.gz | |
Fix 231-cors.t test suite (again)
That's interesting issue: couch_passwords:hash_admin_password accepts
password as binary string, but list one had been passed instead. This
causes crush with function_clause reason. Ok, but this crush left
hidden for R15/R16 - only R14 shows stack trace in output and alerts
that's something wrong. To be honest, *sometimes* it's also possible
to reproduce this test suite crush with modern Erlang releases,
but it will be about Bad Plan: planned 27 test, but run only 26.
Nothing specific.
So, silent crush prevented other tests to be run and also counted by
the plan. Now this is fixed.
| -rw-r--r-- | test/etap/231-cors.t | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/test/etap/231-cors.t b/test/etap/231-cors.t index c568e79a1..2f420d1c4 100644 --- a/test/etap/231-cors.t +++ b/test/etap/231-cors.t @@ -32,7 +32,7 @@ server() -> main(_) -> test_util:init_code_path(), - etap:plan(26), + etap:plan(29), case (catch test()) of ok -> etap:end_tests(); @@ -133,7 +133,7 @@ test() -> % test multiple per-host configuration %% do tests with auth - ok = set_admin_password("test", "test"), + ok = set_admin_password("test", <<"test">>), test_db_preflight_auth_request(), test_db_origin_auth_request(), |