Move issue #95 tests into their own file

storage/file_test.go

@@ -9,7 +9,6 @@ import (
 	"testing"
 
 	"github.com/anacrolix/missinggo"
-	"github.com/anacrolix/missinggo/resource"
 	"github.com/stretchr/testify/assert"
 	"github.com/stretchr/testify/require"
 
@@ -39,50 +38,3 @@ func TestShortFile(t *testing.T) {
 	assert.EqualValues(t, 1, n)
 	assert.Equal(t, io.ErrUnexpectedEOF, err)
 }
-
-// Two different torrents opened from the same storage. Closing one should not
-// break the piece completion on the other.
-func testIssue95(t *testing.T, c Client) {
-	i1 := &metainfo.InfoEx{
-		Bytes: []byte("a"),
-		Info: metainfo.Info{
-			Files:  []metainfo.FileInfo{{Path: []string{"a"}}},
-			Pieces: make([]byte, 20),
-		},
-	}
-	t1, err := c.OpenTorrent(i1)
-	require.NoError(t, err)
-	i2 := &metainfo.InfoEx{
-		Bytes: []byte("b"),
-		Info: metainfo.Info{
-			Files:  []metainfo.FileInfo{{Path: []string{"a"}}},
-			Pieces: make([]byte, 20),
-		},
-	}
-	t2, err := c.OpenTorrent(i2)
-	require.NoError(t, err)
-	t2p := t2.Piece(i2.Piece(0))
-	assert.NoError(t, t1.Close())
-	assert.NotPanics(t, func() { t2p.GetIsComplete() })
-}
-
-func TestIssue95File(t *testing.T) {
-	td, err := ioutil.TempDir("", "")
-	require.NoError(t, err)
-	defer os.RemoveAll(td)
-	testIssue95(t, NewFile(td))
-}
-
-func TestIssue95MMap(t *testing.T) {
-	td, err := ioutil.TempDir("", "")
-	require.NoError(t, err)
-	defer os.RemoveAll(td)
-	testIssue95(t, NewMMap(td))
-}
-
-func TestIssue95ResourcePieces(t *testing.T) {
-	td, err := ioutil.TempDir("", "")
-	require.NoError(t, err)
-	defer os.RemoveAll(td)
-	testIssue95(t, NewResourcePieces(resource.OSFileProvider{}))
-}

storage/issue95_test.go

@@ -0,0 +1,59 @@
+package storage
+
+import (
+	"io/ioutil"
+	"os"
+	"testing"
+
+	"github.com/anacrolix/missinggo/resource"
+	"github.com/anacrolix/torrent/metainfo"
+	"github.com/stretchr/testify/assert"
+	"github.com/stretchr/testify/require"
+)
+
+// Two different torrents opened from the same storage. Closing one should not
+// break the piece completion on the other.
+func testIssue95(t *testing.T, c Client) {
+	i1 := &metainfo.InfoEx{
+		Bytes: []byte("a"),
+		Info: metainfo.Info{
+			Files:  []metainfo.FileInfo{{Path: []string{"a"}}},
+			Pieces: make([]byte, 20),
+		},
+	}
+	t1, err := c.OpenTorrent(i1)
+	require.NoError(t, err)
+	i2 := &metainfo.InfoEx{
+		Bytes: []byte("b"),
+		Info: metainfo.Info{
+			Files:  []metainfo.FileInfo{{Path: []string{"a"}}},
+			Pieces: make([]byte, 20),
+		},
+	}
+	t2, err := c.OpenTorrent(i2)
+	require.NoError(t, err)
+	t2p := t2.Piece(i2.Piece(0))
+	assert.NoError(t, t1.Close())
+	assert.NotPanics(t, func() { t2p.GetIsComplete() })
+}
+
+func TestIssue95File(t *testing.T) {
+	td, err := ioutil.TempDir("", "")
+	require.NoError(t, err)
+	defer os.RemoveAll(td)
+	testIssue95(t, NewFile(td))
+}
+
+func TestIssue95MMap(t *testing.T) {
+	td, err := ioutil.TempDir("", "")
+	require.NoError(t, err)
+	defer os.RemoveAll(td)
+	testIssue95(t, NewMMap(td))
+}
+
+func TestIssue95ResourcePieces(t *testing.T) {
+	td, err := ioutil.TempDir("", "")
+	require.NoError(t, err)
+	defer os.RemoveAll(td)
+	testIssue95(t, NewResourcePieces(resource.OSFileProvider{}))
+}