internal/pkg: rename archive checksum helpers

These names are more consistent with other helper names, so rename them while the API is still internal.

Signed-off-by: Ophestra <cat@gensokyo.uk>

internal/pkg/archive.go

@@ -262,8 +262,8 @@ func Write(fsys fs.FS, root string, w io.Writer) error {
 	return aw.Close()
 }
 
-// HashFS returns a checksum produced by hashing the result of [Flatten].
-func HashFS(buf *Checksum, fsys fs.FS, root string) error {
+// SumFS saves checksum of the archive of fsys to the value pointed to by buf.
+func SumFS(buf *Checksum, fsys fs.FS, root string) error {
 	h := sha512.New384()
 	if err := Write(fsys, root, h); err != nil {
 		return err
@@ -272,7 +272,8 @@ func HashFS(buf *Checksum, fsys fs.FS, root string) error {
 	return nil
 }
 
-// HashDir returns a checksum produced by hashing the result of [Flatten].
-func HashDir(buf *Checksum, pathname *check.Absolute) error {
-	return HashFS(buf, os.DirFS(pathname.String()), ".")
+// SumDir saves checksum of the archive of directory at pathname to the value
+// pointed to by buf.
+func SumDir(buf *Checksum, pathname *check.Absolute) error {
+	return SumFS(buf, os.DirFS(pathname.String()), ".")
 }

internal/pkg/archive_test.go

@@ -141,10 +141,10 @@ func TestArchive(t *testing.T) {
 				t.Parallel()
 
 				var got pkg.Checksum
-				if err := pkg.HashFS(&got, tc.fsys, "."); err != nil {
-					t.Fatalf("HashFS: error = %v", err)
+				if err := pkg.SumFS(&got, tc.fsys, "."); err != nil {
+					t.Fatalf("SumFS: error = %v", err)
 				} else if got != tc.sum {
-					t.Fatalf("HashFS: %v", &pkg.ChecksumMismatchError{
+					t.Fatalf("SumFS: %v", &pkg.ChecksumMismatchError{
 						Got:  got,
 						Want: tc.sum,
 					})

internal/pkg/clean_test.go

@@ -283,8 +283,8 @@ func TestClean(t *testing.T) {
 
 			want := tc.want.hash()
 			var checksum pkg.Checksum
-			if err = pkg.HashDir(&checksum, base); err != nil {
-				t.Fatalf("HashDir: error = %v", err)
+			if err = pkg.SumDir(&checksum, base); err != nil {
+				t.Fatalf("SumDir: error = %v", err)
 			} else if checksum != want {
 				t.Error(expectsFrom(base.String()))
 			}

internal/pkg/pkg.go

@@ -1007,7 +1007,7 @@ func (c *Cache) Scrub(checks int) error {
 
 				pathname := dir.Append(ent.Name())
 				if ent.IsDir() {
-					if err := HashDir(got, pathname); err != nil {
+					if err := SumDir(got, pathname); err != nil {
 						addErr(pathname, err)
 						return true
 					}
@@ -2132,7 +2132,7 @@ func (c *Cache) cure(a Artifact, curesExempt bool) (
 	}
 
 	var gotChecksum Checksum
-	if err = HashFS(
+	if err = SumFS(
 		&gotChecksum,
 		dotOverrideFS{os.DirFS(t.work.String()).(dirFS)},
 		".",

internal/pkg/pkg_test.go

@@ -357,7 +357,7 @@ type expectsFS fstest.MapFS
 
 // hash computes the checksum of e.
 func (e expectsFS) hash() (checksum pkg.Checksum) {
-	if err := pkg.HashFS(&checksum, fstest.MapFS(e), "."); err != nil {
+	if err := pkg.SumFS(&checksum, fstest.MapFS(e), "."); err != nil {
 		panic(err)
 	}
 	return
@@ -561,8 +561,8 @@ func checkWithCache(t *testing.T, testCases []cacheTestCase) {
 			want := tc.want.hash()
 
 			var checksum pkg.Checksum
-			if err := pkg.HashDir(&checksum, base); err != nil {
-				t.Fatalf("HashDir: error = %v", err)
+			if err := pkg.SumDir(&checksum, base); err != nil {
+				t.Fatalf("SumDir: error = %v", err)
 			} else if checksum != want {
 				t.Fatal(expectsFrom(base.String()))
 			}
@@ -581,8 +581,8 @@ func checkWithCache(t *testing.T, testCases []cacheTestCase) {
 			}
 
 			// validate again to make sure scrub did not condemn anything
-			if err := pkg.HashDir(&checksum, base); err != nil {
-				t.Fatalf("HashDir: error = %v", err)
+			if err := pkg.SumDir(&checksum, base); err != nil {
+				t.Fatalf("SumDir: error = %v", err)
 			} else if checksum != want {
 				t.Fatalf("(scrubbed) %s", expectsFrom(base.String()))
 			}