[ARVADOS] updated: 5f47ebc3f5a136cd8932d7cf3aa4931274b6c0a2

[Git user]

Summary of changes:
 services/keepstore/s3_volume_test.go | 15 +++++++++++++++
 1 file changed, 15 insertions(+)

       via  5f47ebc3f5a136cd8932d7cf3aa4931274b6c0a2 (commit)
      from  c8c2068cdb980ef8a1c6a791f6caf6a4bd4b4415 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.


commit 5f47ebc3f5a136cd8932d7cf3aa4931274b6c0a2
Author: Tom Clegg <tom at curoverse.com>
Date:   Tue Jul 26 12:25:35 2016 -0400

    8555: Test Get() after successful Untrash. Test Put+Mtime in all scenarios.

diff --git a/services/keepstore/s3_volume_test.go b/services/keepstore/s3_volume_test.go
index e41e04b..65ce2be 100644
--- a/services/keepstore/s3_volume_test.go
+++ b/services/keepstore/s3_volume_test.go
@@ -270,6 +270,14 @@ func (s *StubbedS3Suite) TestBackendStates(c *check.C) {
 		loc, blk = setupScenario()
 		err = v.Untrash(loc)
 		c.Check(err == nil, check.Equals, scenario.canUntrash)
+		if scenario.dataT != none || scenario.trashT != none {
+			// In all scenarios where the data exists, we
+			// should be able to Get after Untrash --
+			// regardless of timestamps, errors, race
+			// conditions, etc.
+			_, err = v.Get(loc, buf)
+			c.Check(err, check.IsNil)
+		}
 
 		loc, blk = setupScenario()
 		v.EmptyTrash()
@@ -282,6 +290,13 @@ func (s *StubbedS3Suite) TestBackendStates(c *check.C) {
 			// allowance for 1s timestamp precision)
 			c.Check(t.After(t0.Add(-time.Second)), check.Equals, true)
 		}
+
+		loc, blk = setupScenario()
+		err = v.Put(loc, blk)
+		c.Check(err, check.IsNil)
+		t, err := v.Mtime(loc)
+		c.Check(err, check.IsNil)
+		c.Check(t.After(t0.Add(-time.Second)), check.Equals, true)
 	}
 }
 

-----------------------------------------------------------------------


hooks/post-receive
--