15003: Update doc comment on Map, and use it in lib/service.

[arvados.git] / sdk / go / arvados / fs_project_test.go

diff --git a/sdk/go/arvados/fs_project_test.go b/sdk/go/arvados/fs_project_test.go
index 69bae8942fe587672b11595a6953773825819981..49e7d675f8b5c6729b61d94d0f271f615c68e924 100644 (file)
--- a/sdk/go/arvados/fs_project_test.go
+++ b/sdk/go/arvados/fs_project_test.go
@@ -10,8 +10,8 @@ import (
        "io"
        "os"
        "path/filepath"
+       "strings"
 
-       "git.curoverse.com/arvados.git/sdk/go/arvadostest"
        check "gopkg.in/check.v1"
 )
 
@@ -66,15 +66,15 @@ func (s *SiteFSSuite) testHomeProject(c *check.C, path string) {
        f, err = s.fs.Open(path + "/A Project/..")
        c.Assert(err, check.IsNil)
        fi, err := f.Stat()
-       c.Check(err, check.IsNil)
+       c.Assert(err, check.IsNil)
        c.Check(fi.IsDir(), check.Equals, true)
        _, basename := filepath.Split(path)
        c.Check(fi.Name(), check.Equals, basename)
 
        f, err = s.fs.Open(path + "/A Project/A Subproject")
-       c.Check(err, check.IsNil)
+       c.Assert(err, check.IsNil)
        fi, err = f.Stat()
-       c.Check(err, check.IsNil)
+       c.Assert(err, check.IsNil)
        c.Check(fi.IsDir(), check.Equals, true)
 
        for _, nx := range []string{
@@ -89,18 +89,74 @@ func (s *SiteFSSuite) testHomeProject(c *check.C, path string) {
        }
 }
 
+func (s *SiteFSSuite) TestProjectReaddirAfterLoadOne(c *check.C) {
+       f, err := s.fs.Open("/users/active/A Project/A Subproject")
+       c.Assert(err, check.IsNil)
+       defer f.Close()
+       f, err = s.fs.Open("/users/active/A Project/Project does not exist")
+       c.Assert(err, check.NotNil)
+       f, err = s.fs.Open("/users/active/A Project/A Subproject")
+       c.Assert(err, check.IsNil)
+       defer f.Close()
+       f, err = s.fs.Open("/users/active/A Project")
+       c.Assert(err, check.IsNil)
+       defer f.Close()
+       fis, err := f.Readdir(-1)
+       c.Assert(err, check.IsNil)
+       c.Logf("%#v", fis)
+       var foundSubproject, foundCollection bool
+       for _, fi := range fis {
+               switch fi.Name() {
+               case "A Subproject":
+                       foundSubproject = true
+               case "collection_to_move_around":
+                       foundCollection = true
+               }
+       }
+       c.Check(foundSubproject, check.Equals, true)
+       c.Check(foundCollection, check.Equals, true)
+}
+
+func (s *SiteFSSuite) TestSlashInName(c *check.C) {
+       badCollection := Collection{
+               Name:      "bad/collection",
+               OwnerUUID: fixtureAProjectUUID,
+       }
+       err := s.client.RequestAndDecode(&badCollection, "POST", "arvados/v1/collections", s.client.UpdateBody(&badCollection), nil)
+       c.Assert(err, check.IsNil)
+       defer s.client.RequestAndDecode(nil, "DELETE", "arvados/v1/collections/"+badCollection.UUID, nil, nil)
+
+       badProject := Group{
+               Name:       "bad/project",
+               GroupClass: "project",
+               OwnerUUID:  fixtureAProjectUUID,
+       }
+       err = s.client.RequestAndDecode(&badProject, "POST", "arvados/v1/groups", s.client.UpdateBody(&badProject), nil)
+       c.Assert(err, check.IsNil)
+       defer s.client.RequestAndDecode(nil, "DELETE", "arvados/v1/groups/"+badProject.UUID, nil, nil)
+
+       dir, err := s.fs.Open("/users/active/A Project")
+       c.Assert(err, check.IsNil)
+       fis, err := dir.Readdir(-1)
+       c.Check(err, check.IsNil)
+       for _, fi := range fis {
+               c.Logf("fi.Name() == %q", fi.Name())
+               c.Check(strings.Contains(fi.Name(), "/"), check.Equals, false)
+       }
+}
+
 func (s *SiteFSSuite) TestProjectUpdatedByOther(c *check.C) {
        s.fs.MountProject("home", "")
 
        project, err := s.fs.OpenFile("/home/A Project", 0, 0)
-       c.Check(err, check.IsNil)
+       c.Assert(err, check.IsNil)
 
        _, err = s.fs.Open("/home/A Project/oob")
        c.Check(err, check.NotNil)
 
        oob := Collection{
                Name:      "oob",
-               OwnerUUID: arvadostest.AProjectUUID,
+               OwnerUUID: fixtureAProjectUUID,
        }
        err = s.client.RequestAndDecode(&oob, "POST", "arvados/v1/collections", s.client.UpdateBody(&oob), nil)
        c.Assert(err, check.IsNil)
@@ -111,6 +167,7 @@ func (s *SiteFSSuite) TestProjectUpdatedByOther(c *check.C) {
        f, err := s.fs.Open("/home/A Project/oob")
        c.Assert(err, check.IsNil)
        fi, err := f.Stat()
+       c.Assert(err, check.IsNil)
        c.Check(fi.IsDir(), check.Equals, true)
        f.Close()