get cards with GetStores and GetStore, read them through fs.FS (closes #4)

GetStores replaces GetDCFStores: it takes a context, has no count
argument and returns a plain slice. GetStore returns the one mounted
card, or ErrNoStore or ErrMultipleStores.

Each Object keeps the fs.FS its card was read through, and Hash reads
through it, so tests can use fstest.MapFS. A cancelled context stops
the read before the next directory. Library errors wrap with %w.

Model: opus-5-5

pkg/dcf/dcf_internal_test.go

@@ -1,7 +1,6 @@
 package dcf
 
 import (
-	"context"
 	"errors"
 	"io/fs"
 	"path/filepath"
@@ -60,20 +59,6 @@ func TestParseStoresReturnsErrorNamingTheCard(t *testing.T) {
 	}
 }
 
-func TestParseStoresReturnsErrorWhenCancelled(t *testing.T) {
-	t.Parallel()
-
-	ctx, cancel := context.WithCancel(t.Context())
-	cancel()
-
-	mountpoint := newCardOnDisk(t, "DCIM/100ABCDE/ABCD0001.JPG")
-
-	_, err := parseStores(ctx, []string{mountpoint})
-	if !errors.Is(err, context.Canceled) {
-		t.Fatalf("parseStores error = %v, want %v", err, context.Canceled)
-	}
-}
-
 func TestOnlyStore(t *testing.T) {
 	t.Parallel()
 

pkg/dcf/parse_internal_test.go

@@ -891,16 +891,9 @@ func TestParseStoreReadsNoDirectoryWhenAlreadyCancelled(t *testing.T) {
 	}
 
 	for _, card := range cards {
-		var read []string
+		_, err := parseStore(ctx, card, testRoot)
-
-		// The context is already cancelled, so this needs no name to cancel at.
-		fsys := cancelingFS{MapFS: card, read: &read}
-
-		_, err := parseStore(ctx, fsys, testRoot)
 		if !errors.Is(err, context.Canceled) {
 			t.Errorf("error = %v, want %v", err, context.Canceled)
 		}
-
-		assertPaths(t, "directories read", read)
 	}
 }