From fd75f33c8cefcf63d9d4c9f516cd57aee1dadfac Mon Sep 17 00:00:00 2001 From: Aaron Boodman Date: Thu, 9 Jul 2015 18:57:26 -0700 Subject: [PATCH] comment cleanup --- types/incremental_test.go | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/types/incremental_test.go b/types/incremental_test.go index b40cdbca3b..20c270835e 100644 --- a/types/incremental_test.go +++ b/types/incremental_test.go @@ -34,8 +34,6 @@ func TestIncrementalLoad(t *testing.T) { assert.NoError(err) actual := actualVar.(List) - // NOTE: No equals test here because Ref() ends up expanding entire tree, which would kill the point of the test. - prev := cs.count assert.Equal(1, prev) for i := uint64(0); i < expected.Len(); i++ { @@ -45,12 +43,13 @@ func TestIncrementalLoad(t *testing.T) { next := prev switch v.(type) { case Blob, Set, List, Map: + // These are the types that are out-of-line in our current encoding. So we expect them to cause a load. next += 1 } assert.Equal(next, cs.count) - // Do it again to make sure multiple derefs don't change anything. + // Do it again to make sure multiple derefs don't do multiple loads. v = actual.Get(i) assert.Equal(next, cs.count)