Fix test

Author: acogoluegnes

test/osiris_log_SUITE.erl

@@ -480,25 +480,25 @@ iterator_read_chunk_with_read_ahead(Config) ->
              W1;
         (read, #{r := R0, tracer := T}) ->
              %% first chunk, there won't be any data size hints in the reader
-             {ok, H, I0, R1} = osiris_log:chunk_iterator(R0),
+             {ok, _, I0, R1} = osiris_log:chunk_iterator(R0),
              {{_, <<"ho">>}, I1} = osiris_log:iterator_next(I0),
              {{_, <<"hi">>}, I2} = osiris_log:iterator_next(I1),
              ?assertMatch(end_of_chunk, osiris_log:iterator_next(I2)),
              ?assertEqual(2, length(osiris_tracer:calls(T))),
-             {H, R1}
+             R1
      end,
      fun(write, #{w := W0}) ->
              %% this one will be read ahead
              EntriesRev = [<<"foo">>, <<"bar">>],
              {_, W1} = write_committed(EntriesRev, W0),
              W1;
         (read, #{r := R0, tracer := T}) ->
-             {ok, H, I0, R1} = osiris_log:chunk_iterator(R0),
+             {ok, _, I0, R1} = osiris_log:chunk_iterator(R0),
              {{_, <<"bar">>}, I1} = osiris_log:iterator_next(I0),
              {{_, <<"foo">>}, I2} = osiris_log:iterator_next(I1),
              ?assertMatch(end_of_chunk, osiris_log:iterator_next(I2)),
              ?assertEqual(0, length(osiris_tracer:calls(T))),
-             {H, R1}
+             R1
      end,
      fun(write, #{w := W0}) ->
              %% this one will be read ahead
@@ -508,12 +508,12 @@ iterator_read_chunk_with_read_ahead(Config) ->
              W1;
         (read, #{r := R0, tracer := T}) ->
              E1 = binary:copy(<<"b">>, RAL - 200),
-             {ok, H, I0, R1} = osiris_log:chunk_iterator(R0),
+             {ok, _, I0, R1} = osiris_log:chunk_iterator(R0),
              {{_, <<"aaa">>}, I1} = osiris_log:iterator_next(I0),
              {{_, E1}, I2} = osiris_log:iterator_next(I1),
              ?assertMatch(end_of_chunk, osiris_log:iterator_next(I2)),
              ?assertEqual(0, length(osiris_tracer:calls(T))),
-             {H, R1}
+             R1
      end,
      fun(write, #{w := W0}) ->
              %% this one is too big to be read ahead
@@ -523,12 +523,12 @@ iterator_read_chunk_with_read_ahead(Config) ->
              W1;
         (read, #{r := R0, tracer := T}) ->
              E1 = binary:copy(<<"b">>, RAL * 2),
-             {ok, H, I0, R1} = osiris_log:chunk_iterator(R0),
+             {ok, _, I0, R1} = osiris_log:chunk_iterator(R0),
              {{_, <<"aaa">>}, I1} = osiris_log:iterator_next(I0),
              {{_, E1}, I2} = osiris_log:iterator_next(I1),
              ?assertMatch(end_of_chunk, osiris_log:iterator_next(I2)),
              ?assertEqual(2, length(osiris_tracer:calls(T))),
-             {H, R1}
+             R1
      end
     ],
 

test/osiris_tracer.erl

@@ -41,7 +41,10 @@ calls(P) ->
 
 calls(P, M, F) ->
     Calls = calls(P),
-    lists:filter(fun({M1, F1, _A}) -> M1 =:= M, F1 =:= F; (_) -> false end, Calls).
+    lists:filter(fun({M1, F1, _A}) when M1 == M andalso F1 == F ->
+                         true;
+                    (_) -> false
+                 end, Calls).
 
 stop(P) ->
     erlang:trace(self(), false, [call]),