couchdb-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From d..@apache.org
Subject couchdb commit: updated refs/heads/1.6.x to cf3e19f
Date Tue, 29 Apr 2014 14:03:47 GMT
Repository: couchdb
Updated Branches:
  refs/heads/1.6.x 26cff2d51 -> cf3e19f88


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.


Project: http://git-wip-us.apache.org/repos/asf/couchdb/repo
Commit: http://git-wip-us.apache.org/repos/asf/couchdb/commit/cf3e19f8
Tree: http://git-wip-us.apache.org/repos/asf/couchdb/tree/cf3e19f8
Diff: http://git-wip-us.apache.org/repos/asf/couchdb/diff/cf3e19f8

Branch: refs/heads/1.6.x
Commit: cf3e19f88d224a828cef96cc1155c6c23484bc22
Parents: 26cff2d
Author: Alexander Shorin <kxepal@apache.org>
Authored: Tue Apr 29 17:51:15 2014 +0400
Committer: Dirkjan Ochtman <dirkjan@ochtman.nl>
Committed: Tue Apr 29 16:03:36 2014 +0200

----------------------------------------------------------------------
 test/etap/231-cors.t | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/couchdb/blob/cf3e19f8/test/etap/231-cors.t
----------------------------------------------------------------------
diff --git a/test/etap/231-cors.t b/test/etap/231-cors.t
index c568e79..2f420d1 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(),


Mime
View raw message