[couchdb] branch more-flaky-test-fixes-in-couchjs created (now bfbcdd7a7)

[va...@apache.org]

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

vatamane pushed a change to branch more-flaky-test-fixes-in-couchjs
in repository https://gitbox.apache.org/repos/asf/couchdb.git


      at bfbcdd7a7 Another flaky couch_js fix

This branch includes the following new commits:

     new bfbcdd7a7 Another flaky couch_js fix

The 1 revisions listed above as "new" are entirely new to this
repository and will be described in separate emails.  The revisions
listed as "add" were already present in the repository and have only
been added to this reference.

[couchdb] 01/01: Another flaky couch_js fix

[va...@apache.org]

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

vatamane pushed a commit to branch more-flaky-test-fixes-in-couchjs
in repository https://gitbox.apache.org/repos/asf/couchdb.git

commit bfbcdd7a7355b9ee88cc40599156d2f0375ebeee
Author: Nick Vatamaniuc <va...@gmail.com>
AuthorDate: Wed Apr 26 18:41:22 2023 -0400

    Another flaky couch_js fix
    
    After the previous fix, now the flakiness moved on to the next line.
    
    Remove the extra assertion to avoid it generating flaky tests. The main
    assertion is already checked above that we get a crash.
---
 src/couch/test/eunit/couch_js_tests.erl | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/src/couch/test/eunit/couch_js_tests.erl b/src/couch/test/eunit/couch_js_tests.erl
index 09cab59a5..ccd2cd0b5 100644
--- a/src/couch/test/eunit/couch_js_tests.erl
+++ b/src/couch/test/eunit/couch_js_tests.erl
@@ -274,9 +274,7 @@ should_exit_on_internal_error() ->
         % It may fail and just exit the process. That's expected as well
         throw:{os_process_error, _} ->
             ok
-    end,
-    % Expect the process to be dead
-    ?assertThrow({os_process_error, _}, couch_query_servers:proc_prompt(Proc, [<<"reset">>])).
+    end.
 
 trigger_oom(Proc) ->
     Status =