diff options
| author | Alexander Shorin <kxepal@apache.org> | 2014-04-29 17:51:15 +0400 |
|---|---|---|
| committer | Alexander Shorin <kxepal@apache.org> | 2014-04-29 17:51:15 +0400 |
| commit | 0bb567f430c12d8686569398b7be91b699d5f9dc (patch) | |
| tree | c681357c07a9b8ea8bc6a4bdf8c41cd7f855c89c | |
| parent | 079fb84bd6ee716d3831519febfa6631cf6a06b4 (diff) | |
| download | couchdb-0bb567f430c12d8686569398b7be91b699d5f9dc.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(), |