1
0
Fork 0
mirror of https://github.com/moby/moby.git synced 2022-11-09 12:21:53 -05:00
moby--moby/pkg/tarsum
Josh Hawn 8d9e25dbdd Fix TarSum iteration test
I noticed that 3 of the tarsum test cases had expected a tarsum with
a sha256 hash of
  e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855
As I've been working with sha256 quite a bit lately, it struck me that
this is the initial digest value for sha256, which means that no data
was processed. However, these tests *do* process data. It turns out that
there was a bug in the test handling code which did not wait for tarsum
to end completely. This patch corrects these test cases.

I'm unaware of anywhere else in the code base where this would be an issue,
though we definitily need to look out in the future to ensure we are
completing tarsum reads (waiting for EOF).

Docker-DCO-1.1-Signed-off-by: Josh Hawn <josh.hawn@docker.com> (github: jlhawn)
2014-12-02 15:28:13 -08:00
..
testdata tarsum: name collision fix 2014-09-17 14:37:46 -04:00
fileinfosums.go tarsum: name collision fix 2014-09-17 14:37:46 -04:00
fileinfosums_test.go tarsum: name collision fix 2014-09-17 14:37:46 -04:00
MAINTAINERS pkg/tarsum: add maintainers & add missing s 2014-11-26 00:44:16 +02:00
tarsum.go pkg/tarsum: actually init the TarSum struct 2014-11-19 15:46:03 -05:00
tarsum_test.go Fix TarSum iteration test 2014-12-02 15:28:13 -08:00
versioning.go Finalize TarSum Version 1 w/ refactor 2014-11-04 21:38:52 -08:00
versioning_test.go Finalize TarSum Version 1 w/ refactor 2014-11-04 21:38:52 -08:00
writercloser.go TarSum: versioning 2014-09-10 15:41:52 -04:00