Move issue #95 tests into their own file

diff --git a/storage/file_test.go b/storage/file_test.go
index e7776132035efb802cef5e10593f9fffcfc199de..6e1d055e7d65c13a6e9fa32078b8a2f1c6d839b4 100644 (file)
--- a/storage/file_test.go
+++ b/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{}))
-}

diff --git a/storage/issue95_test.go b/storage/issue95_test.go
new file mode 100644 (file)
index 0000000..8641649
--- /dev/null
+++ b/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{}))
+}