send checksum callback separately as range requests will not send one.

cloudant · Feb 17, 2011 · 889e791 · 889e791
1 parent 95bfacf
commit 889e791
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/src/monic_file.erl b/src/monic_file.erl
@@ -137,9 +137,10 @@ handle_call({read, #handle{location=Location,cookie=Cookie}, Fun}, _From, #state
                     case read_item_footer(Fd, Location + ?ITEM_HEADER_SIZE + Len) of
                         {ok, #item_footer{sha=RecordedSha}} ->
                             case RecordedSha of
-                                CalculatedSha -> Fun({eof, checksum_verified});
-                                _ -> Fun({eof, checksum_failed})
+                                CalculatedSha -> Fun({checksum, valid});
+                                _ -> Fun({checksum, invalid})
                             end,
+                            Fun(eof),
                             {reply, ok, State};
                         Else ->
                             {reply, {error, Else}, State}

diff --git a/test/monic_test.erl b/test/monic_test.erl
@@ -56,7 +56,9 @@ read_fun() ->
                       undefined ->
                           ?assertEqual({ok, <<"foobar">>}, R);
                       {ok, <<"foobar">>} ->
-                          ?assertEqual({eof, checksum_verified}, R)
+                          ?assertEqual({checksum, valid}, R);
+                      {checksum, valid} ->
+                          ?assertEqual(eof, R)
                   end,
                   put(last, R) end,
     ok = monic:read("foo", Handle, Fun).