Skip to content

Commit

Permalink
Apply suggestions from code review
Browse files Browse the repository at this point in the history
Co-authored-by: jacekwegr <[email protected]>
  • Loading branch information
arcusfelis and jacekwegr authored Dec 12, 2024
1 parent c6131e5 commit 01ec22e
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 6 deletions.
2 changes: 1 addition & 1 deletion test/common/mongoose_helper.erl
Original file line number Diff line number Diff line change
Expand Up @@ -355,7 +355,7 @@ inject_module(Module, ReloadIfAlreadyLoaded) ->
Module :: module(),
ReloadIfAlreadyLoaded :: no_reload | reload) ->
ok | already_loaded.
inject_module(#{node := NodeAtom} = Node, Module, _) when NodeAtom =:= node() ->
inject_module(#{node := NodeAtom} = _Node, _, _) when NodeAtom =:= node() ->
already_loaded;
inject_module(#{} = Node, Module, no_reload) ->
case successful_rpc(Node, code, is_loaded, [Module]) of
Expand Down
7 changes: 2 additions & 5 deletions test/mnesia_db_SUITE.erl
Original file line number Diff line number Diff line change
@@ -1,10 +1,7 @@
-module(mnesia_db_SUITE).
-compile([export_all, nowarn_export_all]).

-include_lib("exml/include/exml.hrl").
-include_lib("eunit/include/eunit.hrl").
-include("mongoose.hrl").
-include("jlib.hrl").

local() ->
#{node => node()}.
Expand All @@ -15,7 +12,7 @@ all() ->
init_per_suite(Config) ->
Config.

end_per_suite(Config) ->
end_per_suite(_Config) ->
ok.

init_per_testcase(_, C) ->
Expand All @@ -31,7 +28,7 @@ mock_mnesia() ->
meck:new(mnesia, []),
meck:expect(mnesia, system_info, fun(local_tables) -> [test_table_fast, test_table_slow] end),
meck:expect(mnesia, table_info, fun(_, _) -> [] end),
meck:expect(mnesia, wait_for_tables, fun(Tables, Interval) ->
meck:expect(mnesia, wait_for_tables, fun(Tables, _Interval) ->
case meck:num_calls(mnesia, wait_for_tables, '_') > 5 of
true -> ok;
false -> {timeout, Tables -- [test_table_fast]}
Expand Down

0 comments on commit 01ec22e

Please sign in to comment.