This is an automated email from the ASF dual-hosted git repository.

jaydoane pushed a commit to branch config-delete-reload
in repository https://gitbox.apache.org/repos/asf/couchdb.git

commit 4132b272f486aee5cd2646ab95bd91bae7823fb7
Author: Jay Doane <[email protected]>
AuthorDate: Wed Oct 18 17:32:29 2023 -0700

    Delete confusing test config delete reason
    
    The third argument to `config:delete/3`, if not a boolean, should be a
    reason for the deletion. This "8080" that looks like a port number was
    presumably a copy-pasta error.
---
 src/config/test/config_tests.erl | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/config/test/config_tests.erl b/src/config/test/config_tests.erl
index 77b961872..c10878a7e 100644
--- a/src/config/test/config_tests.erl
+++ b/src/config/test/config_tests.erl
@@ -437,7 +437,7 @@ should_write_changes(_, _) ->
         ?assertEqual("5986", config:get("httpd", "port")),
         ?assertEqual(ok, config:set("httpd", "port", "8080")),
         ?assertEqual("8080", config:get("httpd", "port")),
-        ?assertEqual(ok, config:delete("httpd", "bind_address", "8080")),
+        ?assertEqual(ok, config:delete("httpd", "bind_address")),
         ?assertEqual(undefined, config:get("httpd", "bind_address"))
     end).
 

Reply via email to