Make blkdiscard a DiskWiper

Should have been a DiskWiper from the beginning and its going to be
useful soon.

examples/blkdiscard/main.go

@@ -5,6 +5,7 @@ import (
 	"flag"
 	"time"
 
+	"github.com/metal-toolbox/ironlib/actions"
 	"github.com/metal-toolbox/ironlib/utils"
 	"github.com/sirupsen/logrus"
 )
@@ -27,13 +28,13 @@ func main() {
 		logger.WithError(err).Fatal("failed to parse timeout duration")
 	}
 
-	blkdiscard := utils.NewBlkdiscardCmd()
+	var blkdiscard actions.DiskWiper = utils.NewBlkdiscardCmd()
 
 	ctx, cancel := context.WithTimeout(context.Background(), timeout)
 	defer cancel()
 
 	logger.Info("running blkdiscard on ", *device)
-	err = blkdiscard.Discard(ctx, *device)
+	err = blkdiscard.WipeDisk(ctx, logger, *device)
 	if err != nil {
 		logger.WithError(err).Fatal("exiting")
 	}

utils/blkdiscard.go

@@ -4,6 +4,8 @@ import (
 	"cmp"
 	"context"
 	"os"
+
+	"github.com/sirupsen/logrus"
 )
 
 const (
@@ -37,6 +39,12 @@ func (b *Blkdiscard) Discard(ctx context.Context, device string) error {
 	return nil
 }
 
+// WipeDisk implements DiskWipe by calling Discard
+func (b *Blkdiscard) WipeDisk(ctx context.Context, logger *logrus.Logger, device string) error {
+	logger.WithField("device", device).WithField("method", "blkdiscard").Info("wiping")
+	return b.Discard(ctx, device)
+}
+
 // NewFakeBlkdiscard returns a mock implementation of the Blkdiscard interface for use in tests.
 func NewFakeBlkdiscard() *Blkdiscard {
 	return &Blkdiscard{

utils/blkdiscard_test.go

@@ -4,10 +4,15 @@ import (
 	"context"
 	"testing"
 
+	"github.com/sirupsen/logrus/hooks/test"
 	"github.com/stretchr/testify/assert"
 )
 
 func Test_blkdiscard(t *testing.T) {
-	err := NewFakeBlkdiscard().Discard(context.TODO(), "/dev/sdZZZ")
+	// Test Fill function
+	logger, hook := test.NewNullLogger()
+	defer hook.Reset()
+
+	err := NewFakeBlkdiscard().WipeDisk(context.TODO(), logger, "/dev/sdZZZ")
 	assert.NoError(t, err)
 }