You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@couchdb.apache.org by ei...@apache.org on 2016/05/03 21:05:34 UTC
[1/3] couch commit: updated refs/heads/master to aef448d
Repository: couchdb-couch
Updated Branches:
refs/heads/master 78f575ec2 -> aef448db0
Add tests for couch_compress module
Project: http://git-wip-us.apache.org/repos/asf/couchdb-couch/repo
Commit: http://git-wip-us.apache.org/repos/asf/couchdb-couch/commit/21b2847d
Tree: http://git-wip-us.apache.org/repos/asf/couchdb-couch/tree/21b2847d
Diff: http://git-wip-us.apache.org/repos/asf/couchdb-couch/diff/21b2847d
Branch: refs/heads/master
Commit: 21b2847db6c50db1a5bc25406e34084f8c9689d0
Parents: 78f575e
Author: Eric Avdey <ei...@eiri.ca>
Authored: Tue May 3 13:36:52 2016 -0300
Committer: Eric Avdey <ei...@eiri.ca>
Committed: Tue May 3 13:36:52 2016 -0300
----------------------------------------------------------------------
test/couch_compress_tests.erl | 66 ++++++++++++++++++++++++++++++++++++++
1 file changed, 66 insertions(+)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/couchdb-couch/blob/21b2847d/test/couch_compress_tests.erl
----------------------------------------------------------------------
diff --git a/test/couch_compress_tests.erl b/test/couch_compress_tests.erl
new file mode 100644
index 0000000..f47bf47
--- /dev/null
+++ b/test/couch_compress_tests.erl
@@ -0,0 +1,66 @@
+% Licensed under the Apache License, Version 2.0 (the "License"); you may not
+% use this file except in compliance with the License. You may obtain a copy of
+% the License at
+%
+% http://www.apache.org/licenses/LICENSE-2.0
+%
+% Unless required by applicable law or agreed to in writing, software
+% distributed under the License is distributed on an "AS IS" BASIS, WITHOUT
+% WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the
+% License for the specific language governing permissions and limitations under
+% the License.
+
+-module(couch_compress_tests).
+
+-include_lib("couch/include/couch_eunit.hrl").
+-include_lib("couch/include/couch_db.hrl").
+
+-define(TERM, {[{a, 1}, {b, 2}, {c, 3}, {d, 4}, {e, 5}]}).
+-define(NONE, <<131,104,1,108,0,0,0,5,104,2,100,0,1,97,97,1,
+ 104,2,100,0,1,98,97,2,104,2,100,0,1,99,97,3,104,2,100,0,
+ 1,100,97,4,104,2,100,0,1,101,97,5,106>>).
+-define(DEFLATE, <<131,80,0,0,0,48,120,218,203,96,204,97,96,
+ 96,96,205,96,74,97,96,76,76,100,4,211,73,137,76,96,58,57,
+ 145,25,76,167,36,178,128,233,212,68,214,44,0,212,169,9,51>>).
+-define(SNAPPY, <<1,49,64,131,104,1,108,0,0,0,5,104,2,100,0,
+ 1,97,97,1,104,1,8,8,98,97,2,5,8,8,99,97,3,5,8,44,100,97,
+ 4,104,2,100,0,1,101,97,5,106>>).
+
+
+compress_test_() ->
+ [
+ ?_assertEqual(?NONE, couch_compress:compress(?TERM, none)),
+ ?_assertEqual(?DEFLATE, couch_compress:compress(?TERM, {deflate, 9})),
+ ?_assertEqual(?SNAPPY, couch_compress:compress(?TERM, snappy))
+ ].
+
+decompress_test_() ->
+ [
+ ?_assertEqual(?TERM, couch_compress:decompress(?NONE)),
+ ?_assertEqual(?TERM, couch_compress:decompress(?DEFLATE)),
+ ?_assertEqual(?TERM, couch_compress:decompress(?SNAPPY))
+ ].
+
+recompress_test_() ->
+ [
+ ?_assertEqual(?DEFLATE, couch_compress:compress(?NONE, {deflate, 9})),
+ ?_assertEqual(?SNAPPY, couch_compress:compress(?NONE, snappy)),
+ ?_assertEqual(?NONE, couch_compress:compress(?DEFLATE, none)),
+ ?_assertEqual(?SNAPPY, couch_compress:compress(?DEFLATE, snappy)),
+ ?_assertEqual(?NONE, couch_compress:compress(?SNAPPY, none)),
+ ?_assertEqual(?DEFLATE, couch_compress:compress(?SNAPPY, {deflate, 9}))
+ ].
+
+is_compressed_test_() ->
+ [
+ ?_assert(couch_compress:is_compressed(?NONE, none)),
+ ?_assert(couch_compress:is_compressed(?DEFLATE, {deflate, 9})),
+ ?_assert(couch_compress:is_compressed(?SNAPPY, snappy)),
+ ?_assertNot(couch_compress:is_compressed(?NONE, {deflate, 0})),
+ ?_assertNot(couch_compress:is_compressed(?NONE, {deflate, 9})),
+ ?_assertNot(couch_compress:is_compressed(?NONE, snappy)),
+ ?_assertNot(couch_compress:is_compressed(?DEFLATE, none)),
+ ?_assertNot(couch_compress:is_compressed(?DEFLATE, snappy)),
+ ?_assertNot(couch_compress:is_compressed(?SNAPPY, none)),
+ ?_assertNot(couch_compress:is_compressed(?SNAPPY, {deflate, 9}))
+ ].
[2/3] couch commit: updated refs/heads/master to aef448d
Posted by ei...@apache.org.
Raise an exception on corrupt binary decompression
Project: http://git-wip-us.apache.org/repos/asf/couchdb-couch/repo
Commit: http://git-wip-us.apache.org/repos/asf/couchdb-couch/commit/925c1deb
Tree: http://git-wip-us.apache.org/repos/asf/couchdb-couch/tree/925c1deb
Diff: http://git-wip-us.apache.org/repos/asf/couchdb-couch/diff/925c1deb
Branch: refs/heads/master
Commit: 925c1deb738617cd48bab40707b9b60e32a4dcdd
Parents: 21b2847
Author: Eric Avdey <ei...@eiri.ca>
Authored: Tue May 3 14:58:51 2016 -0300
Committer: Eric Avdey <ei...@eiri.ca>
Committed: Tue May 3 14:58:51 2016 -0300
----------------------------------------------------------------------
src/couch_compress.erl | 8 ++++++--
test/couch_compress_tests.erl | 12 ++++++++++--
2 files changed, 16 insertions(+), 4 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/couchdb-couch/blob/925c1deb/src/couch_compress.erl
----------------------------------------------------------------------
diff --git a/src/couch_compress.erl b/src/couch_compress.erl
index f9b5aa6..71588b2 100644
--- a/src/couch_compress.erl
+++ b/src/couch_compress.erl
@@ -65,7 +65,9 @@ decompress(<<?SNAPPY_PREFIX, Rest/binary>>) ->
{ok, TermBin} = snappy:decompress(Rest),
binary_to_term(TermBin);
decompress(<<?TERM_PREFIX, _/binary>> = Bin) ->
- binary_to_term(Bin).
+ binary_to_term(Bin);
+decompress(_) ->
+ error(invalid_compression).
is_compressed(<<?SNAPPY_PREFIX, _/binary>>, Method) ->
@@ -77,5 +79,7 @@ is_compressed(<<?COMPRESSED_TERM_PREFIX, _/binary>>, _Method) ->
is_compressed(<<?TERM_PREFIX, _/binary>>, Method) ->
Method =:= none;
is_compressed(Term, _Method) when not is_binary(Term) ->
- false.
+ false;
+is_compressed(_, _) ->
+ error(invalid_compression).
http://git-wip-us.apache.org/repos/asf/couchdb-couch/blob/925c1deb/test/couch_compress_tests.erl
----------------------------------------------------------------------
diff --git a/test/couch_compress_tests.erl b/test/couch_compress_tests.erl
index f47bf47..6d6e6a7 100644
--- a/test/couch_compress_tests.erl
+++ b/test/couch_compress_tests.erl
@@ -25,6 +25,7 @@
-define(SNAPPY, <<1,49,64,131,104,1,108,0,0,0,5,104,2,100,0,
1,97,97,1,104,1,8,8,98,97,2,5,8,8,99,97,3,5,8,44,100,97,
4,104,2,100,0,1,101,97,5,106>>).
+-define(CORRUPT, <<2,12,85,06>>).
compress_test_() ->
@@ -38,7 +39,8 @@ decompress_test_() ->
[
?_assertEqual(?TERM, couch_compress:decompress(?NONE)),
?_assertEqual(?TERM, couch_compress:decompress(?DEFLATE)),
- ?_assertEqual(?TERM, couch_compress:decompress(?SNAPPY))
+ ?_assertEqual(?TERM, couch_compress:decompress(?SNAPPY)),
+ ?_assertError(invalid_compression, couch_compress:decompress(?CORRUPT))
].
recompress_test_() ->
@@ -62,5 +64,11 @@ is_compressed_test_() ->
?_assertNot(couch_compress:is_compressed(?DEFLATE, none)),
?_assertNot(couch_compress:is_compressed(?DEFLATE, snappy)),
?_assertNot(couch_compress:is_compressed(?SNAPPY, none)),
- ?_assertNot(couch_compress:is_compressed(?SNAPPY, {deflate, 9}))
+ ?_assertNot(couch_compress:is_compressed(?SNAPPY, {deflate, 9})),
+ ?_assertError(invalid_compression,
+ couch_compress:is_compressed(?CORRUPT, none)),
+ ?_assertError(invalid_compression,
+ couch_compress:is_compressed(?CORRUPT, {deflate, 9})),
+ ?_assertError(invalid_compression,
+ couch_compress:is_compressed(?CORRUPT, snappy))
].
[3/3] couch commit: updated refs/heads/master to aef448d
Posted by ei...@apache.org.
Merge remote branch 'github/pr/169'
This closes #169
Signed-off-by: Eric Avdey <ei...@eiri.ca>
Project: http://git-wip-us.apache.org/repos/asf/couchdb-couch/repo
Commit: http://git-wip-us.apache.org/repos/asf/couchdb-couch/commit/aef448db
Tree: http://git-wip-us.apache.org/repos/asf/couchdb-couch/tree/aef448db
Diff: http://git-wip-us.apache.org/repos/asf/couchdb-couch/diff/aef448db
Branch: refs/heads/master
Commit: aef448db040eb41d0c7a8a75d2ac447893444194
Parents: 78f575e 925c1de
Author: Eric Avdey <ei...@eiri.ca>
Authored: Tue May 3 16:04:19 2016 -0300
Committer: Eric Avdey <ei...@eiri.ca>
Committed: Tue May 3 16:04:19 2016 -0300
----------------------------------------------------------------------
src/couch_compress.erl | 8 +++--
test/couch_compress_tests.erl | 74 ++++++++++++++++++++++++++++++++++++++
2 files changed, 80 insertions(+), 2 deletions(-)
----------------------------------------------------------------------