yandex: fix fs.Name()

Put in tests for fs.Root() and fs.Name() for all remotes

amazonclouddrive/amazonclouddrive_test.go

@@ -20,6 +20,8 @@ func TestSetup(t *testing.T) {
 // Generic tests for the Fs
 func TestInit(t *testing.T)                { fstests.TestInit(t) }
 func TestFsString(t *testing.T)            { fstests.TestFsString(t) }
+func TestFsName(t *testing.T)              { fstests.TestFsName(t) }
+func TestFsRoot(t *testing.T)              { fstests.TestFsRoot(t) }
 func TestFsRmdirEmpty(t *testing.T)        { fstests.TestFsRmdirEmpty(t) }
 func TestFsRmdirNotFound(t *testing.T)     { fstests.TestFsRmdirNotFound(t) }
 func TestFsMkdir(t *testing.T)             { fstests.TestFsMkdir(t) }

b2/b2_test.go

@@ -20,6 +20,8 @@ func TestSetup(t *testing.T) {
 // Generic tests for the Fs
 func TestInit(t *testing.T)                { fstests.TestInit(t) }
 func TestFsString(t *testing.T)            { fstests.TestFsString(t) }
+func TestFsName(t *testing.T)              { fstests.TestFsName(t) }
+func TestFsRoot(t *testing.T)              { fstests.TestFsRoot(t) }
 func TestFsRmdirEmpty(t *testing.T)        { fstests.TestFsRmdirEmpty(t) }
 func TestFsRmdirNotFound(t *testing.T)     { fstests.TestFsRmdirNotFound(t) }
 func TestFsMkdir(t *testing.T)             { fstests.TestFsMkdir(t) }

crypt/crypt2_test.go

@@ -21,6 +21,8 @@ func TestSetup2(t *testing.T) {
 // Generic tests for the Fs
 func TestInit2(t *testing.T)                { fstests.TestInit(t) }
 func TestFsString2(t *testing.T)            { fstests.TestFsString(t) }
+func TestFsName2(t *testing.T)              { fstests.TestFsName(t) }
+func TestFsRoot2(t *testing.T)              { fstests.TestFsRoot(t) }
 func TestFsRmdirEmpty2(t *testing.T)        { fstests.TestFsRmdirEmpty(t) }
 func TestFsRmdirNotFound2(t *testing.T)     { fstests.TestFsRmdirNotFound(t) }
 func TestFsMkdir2(t *testing.T)             { fstests.TestFsMkdir(t) }

crypt/crypt3_test.go

@@ -21,6 +21,8 @@ func TestSetup3(t *testing.T) {
 // Generic tests for the Fs
 func TestInit3(t *testing.T)                { fstests.TestInit(t) }
 func TestFsString3(t *testing.T)            { fstests.TestFsString(t) }
+func TestFsName3(t *testing.T)              { fstests.TestFsName(t) }
+func TestFsRoot3(t *testing.T)              { fstests.TestFsRoot(t) }
 func TestFsRmdirEmpty3(t *testing.T)        { fstests.TestFsRmdirEmpty(t) }
 func TestFsRmdirNotFound3(t *testing.T)     { fstests.TestFsRmdirNotFound(t) }
 func TestFsMkdir3(t *testing.T)             { fstests.TestFsMkdir(t) }

crypt/crypt_test.go

@@ -21,6 +21,8 @@ func TestSetup(t *testing.T) {
 // Generic tests for the Fs
 func TestInit(t *testing.T)                { fstests.TestInit(t) }
 func TestFsString(t *testing.T)            { fstests.TestFsString(t) }
+func TestFsName(t *testing.T)              { fstests.TestFsName(t) }
+func TestFsRoot(t *testing.T)              { fstests.TestFsRoot(t) }
 func TestFsRmdirEmpty(t *testing.T)        { fstests.TestFsRmdirEmpty(t) }
 func TestFsRmdirNotFound(t *testing.T)     { fstests.TestFsRmdirNotFound(t) }
 func TestFsMkdir(t *testing.T)             { fstests.TestFsMkdir(t) }

drive/drive_test.go

@@ -20,6 +20,8 @@ func TestSetup(t *testing.T) {
 // Generic tests for the Fs
 func TestInit(t *testing.T)                { fstests.TestInit(t) }
 func TestFsString(t *testing.T)            { fstests.TestFsString(t) }
+func TestFsName(t *testing.T)              { fstests.TestFsName(t) }
+func TestFsRoot(t *testing.T)              { fstests.TestFsRoot(t) }
 func TestFsRmdirEmpty(t *testing.T)        { fstests.TestFsRmdirEmpty(t) }
 func TestFsRmdirNotFound(t *testing.T)     { fstests.TestFsRmdirNotFound(t) }
 func TestFsMkdir(t *testing.T)             { fstests.TestFsMkdir(t) }

dropbox/dropbox_test.go

@@ -20,6 +20,8 @@ func TestSetup(t *testing.T) {
 // Generic tests for the Fs
 func TestInit(t *testing.T)                { fstests.TestInit(t) }
 func TestFsString(t *testing.T)            { fstests.TestFsString(t) }
+func TestFsName(t *testing.T)              { fstests.TestFsName(t) }
+func TestFsRoot(t *testing.T)              { fstests.TestFsRoot(t) }
 func TestFsRmdirEmpty(t *testing.T)        { fstests.TestFsRmdirEmpty(t) }
 func TestFsRmdirNotFound(t *testing.T)     { fstests.TestFsRmdirNotFound(t) }
 func TestFsMkdir(t *testing.T)             { fstests.TestFsMkdir(t) }

fstest/fstests/fstests.go

@@ -52,6 +52,7 @@ var (
 	dumpHeaders    = flag.Bool("dump-headers", false, "Dump HTTP headers - may contain sensitive info")
 	dumpBodies     = flag.Bool("dump-bodies", false, "Dump HTTP headers and bodies - may contain sensitive info")
 	overrideRemote = flag.String("remote", "", "Set this to override the default remote name (eg s3:)")
+	isLocalRemote  bool
 )
 
 // ExtraConfigItem describes a config item added on the fly while testing
@@ -88,6 +89,7 @@ func TestInit(t *testing.T) {
 	if RemoteName == "" {
 		RemoteName, err = fstest.LocalRemote()
 		require.NoError(t, err)
+		isLocalRemote = true
 	}
 	subRemoteName, subRemoteLeaf, err = fstest.RandomRemoteName(RemoteName)
 	require.NoError(t, err)
@@ -125,7 +127,29 @@ func skipIfNotListR(t *testing.T) func() {
 func TestFsString(t *testing.T) {
 	skipIfNotOk(t)
 	str := remote.String()
-	require.NotEqual(t, str, "")
+	require.NotEqual(t, "", str)
+}
+
+// TestFsName tests the Name method
+func TestFsName(t *testing.T) {
+	skipIfNotOk(t)
+	got := remote.Name()
+	want := RemoteName
+	if isLocalRemote {
+		want = "local:"
+	}
+	require.Equal(t, want, got+":")
+}
+
+// TestFsRoot tests the Root method
+func TestFsRoot(t *testing.T) {
+	skipIfNotOk(t)
+	name := remote.Name() + ":"
+	root := remote.Root()
+	if isLocalRemote {
+		name = ""
+	}
+	require.Equal(t, subRemoteName, name+root)
 }
 
 // TestFsRmdirEmpty tests deleting an empty directory

ftp/ftp_test.go

@@ -20,6 +20,8 @@ func TestSetup(t *testing.T) {
 // Generic tests for the Fs
 func TestInit(t *testing.T)                { fstests.TestInit(t) }
 func TestFsString(t *testing.T)            { fstests.TestFsString(t) }
+func TestFsName(t *testing.T)              { fstests.TestFsName(t) }
+func TestFsRoot(t *testing.T)              { fstests.TestFsRoot(t) }
 func TestFsRmdirEmpty(t *testing.T)        { fstests.TestFsRmdirEmpty(t) }
 func TestFsRmdirNotFound(t *testing.T)     { fstests.TestFsRmdirNotFound(t) }
 func TestFsMkdir(t *testing.T)             { fstests.TestFsMkdir(t) }

googlecloudstorage/googlecloudstorage_test.go

@@ -20,6 +20,8 @@ func TestSetup(t *testing.T) {
 // Generic tests for the Fs
 func TestInit(t *testing.T)                { fstests.TestInit(t) }
 func TestFsString(t *testing.T)            { fstests.TestFsString(t) }
+func TestFsName(t *testing.T)              { fstests.TestFsName(t) }
+func TestFsRoot(t *testing.T)              { fstests.TestFsRoot(t) }
 func TestFsRmdirEmpty(t *testing.T)        { fstests.TestFsRmdirEmpty(t) }
 func TestFsRmdirNotFound(t *testing.T)     { fstests.TestFsRmdirNotFound(t) }
 func TestFsMkdir(t *testing.T)             { fstests.TestFsMkdir(t) }

hubic/hubic_test.go

@@ -20,6 +20,8 @@ func TestSetup(t *testing.T) {
 // Generic tests for the Fs
 func TestInit(t *testing.T)                { fstests.TestInit(t) }
 func TestFsString(t *testing.T)            { fstests.TestFsString(t) }
+func TestFsName(t *testing.T)              { fstests.TestFsName(t) }
+func TestFsRoot(t *testing.T)              { fstests.TestFsRoot(t) }
 func TestFsRmdirEmpty(t *testing.T)        { fstests.TestFsRmdirEmpty(t) }
 func TestFsRmdirNotFound(t *testing.T)     { fstests.TestFsRmdirNotFound(t) }
 func TestFsMkdir(t *testing.T)             { fstests.TestFsMkdir(t) }

local/local_test.go

@@ -20,6 +20,8 @@ func TestSetup(t *testing.T) {
 // Generic tests for the Fs
 func TestInit(t *testing.T)                { fstests.TestInit(t) }
 func TestFsString(t *testing.T)            { fstests.TestFsString(t) }
+func TestFsName(t *testing.T)              { fstests.TestFsName(t) }
+func TestFsRoot(t *testing.T)              { fstests.TestFsRoot(t) }
 func TestFsRmdirEmpty(t *testing.T)        { fstests.TestFsRmdirEmpty(t) }
 func TestFsRmdirNotFound(t *testing.T)     { fstests.TestFsRmdirNotFound(t) }
 func TestFsMkdir(t *testing.T)             { fstests.TestFsMkdir(t) }

onedrive/onedrive_test.go

@@ -20,6 +20,8 @@ func TestSetup(t *testing.T) {
 // Generic tests for the Fs
 func TestInit(t *testing.T)                { fstests.TestInit(t) }
 func TestFsString(t *testing.T)            { fstests.TestFsString(t) }
+func TestFsName(t *testing.T)              { fstests.TestFsName(t) }
+func TestFsRoot(t *testing.T)              { fstests.TestFsRoot(t) }
 func TestFsRmdirEmpty(t *testing.T)        { fstests.TestFsRmdirEmpty(t) }
 func TestFsRmdirNotFound(t *testing.T)     { fstests.TestFsRmdirNotFound(t) }
 func TestFsMkdir(t *testing.T)             { fstests.TestFsMkdir(t) }

s3/s3_test.go

@@ -20,6 +20,8 @@ func TestSetup(t *testing.T) {
 // Generic tests for the Fs
 func TestInit(t *testing.T)                { fstests.TestInit(t) }
 func TestFsString(t *testing.T)            { fstests.TestFsString(t) }
+func TestFsName(t *testing.T)              { fstests.TestFsName(t) }
+func TestFsRoot(t *testing.T)              { fstests.TestFsRoot(t) }
 func TestFsRmdirEmpty(t *testing.T)        { fstests.TestFsRmdirEmpty(t) }
 func TestFsRmdirNotFound(t *testing.T)     { fstests.TestFsRmdirNotFound(t) }
 func TestFsMkdir(t *testing.T)             { fstests.TestFsMkdir(t) }

sftp/sftp_test.go

@@ -20,6 +20,8 @@ func TestSetup(t *testing.T) {
 // Generic tests for the Fs
 func TestInit(t *testing.T)                { fstests.TestInit(t) }
 func TestFsString(t *testing.T)            { fstests.TestFsString(t) }
+func TestFsName(t *testing.T)              { fstests.TestFsName(t) }
+func TestFsRoot(t *testing.T)              { fstests.TestFsRoot(t) }
 func TestFsRmdirEmpty(t *testing.T)        { fstests.TestFsRmdirEmpty(t) }
 func TestFsRmdirNotFound(t *testing.T)     { fstests.TestFsRmdirNotFound(t) }
 func TestFsMkdir(t *testing.T)             { fstests.TestFsMkdir(t) }

swift/swift_test.go

@@ -20,6 +20,8 @@ func TestSetup(t *testing.T) {
 // Generic tests for the Fs
 func TestInit(t *testing.T)                { fstests.TestInit(t) }
 func TestFsString(t *testing.T)            { fstests.TestFsString(t) }
+func TestFsName(t *testing.T)              { fstests.TestFsName(t) }
+func TestFsRoot(t *testing.T)              { fstests.TestFsRoot(t) }
 func TestFsRmdirEmpty(t *testing.T)        { fstests.TestFsRmdirEmpty(t) }
 func TestFsRmdirNotFound(t *testing.T)     { fstests.TestFsRmdirNotFound(t) }
 func TestFsMkdir(t *testing.T)             { fstests.TestFsMkdir(t) }

yandex/yandex.go

@@ -130,7 +130,8 @@ func NewFs(name, root string) (fs.Fs, error) {
 	yandexDisk := yandex.NewClient(token.AccessToken, fs.Config.Client())
 
 	f := &Fs{
-		yd: yandexDisk,
+		name: name,
+		yd:   yandexDisk,
 	}
 	f.features = (&fs.Features{ReadMimeType: true, WriteMimeType: true}).Fill(f)
 	f.setRoot(root)

yandex/yandex_test.go

@@ -20,6 +20,8 @@ func TestSetup(t *testing.T) {
 // Generic tests for the Fs
 func TestInit(t *testing.T)                { fstests.TestInit(t) }
 func TestFsString(t *testing.T)            { fstests.TestFsString(t) }
+func TestFsName(t *testing.T)              { fstests.TestFsName(t) }
+func TestFsRoot(t *testing.T)              { fstests.TestFsRoot(t) }
 func TestFsRmdirEmpty(t *testing.T)        { fstests.TestFsRmdirEmpty(t) }
 func TestFsRmdirNotFound(t *testing.T)     { fstests.TestFsRmdirNotFound(t) }
 func TestFsMkdir(t *testing.T)             { fstests.TestFsMkdir(t) }