chore(backend): delete op to config (#16)

internal/plugin/path_config.go

@@ -19,8 +19,9 @@ const (
 )
 
 var (
-	errMissingAPIKey       = errors.New("missing API key from configuration")
-	errTypeAssertionFailed = errors.New("type assertion failed")
+	errBackendNotConfigured = errors.New("backend not configured")
+	errMissingAPIKey        = errors.New("missing API key from configuration")
+	errTypeAssertionFailed  = errors.New("type assertion failed")
 )
 
 type backendConfig struct {
@@ -57,6 +58,9 @@ func (b *backend) pathConfig() []*framework.Path {
 				logical.CreateOperation: &framework.PathOperation{
 					Callback: b.pathConfigWrite,
 				},
+				logical.DeleteOperation: &framework.PathOperation{
+					Callback: b.pathConfigDelete,
+				},
 			},
 		},
 	}
@@ -71,7 +75,7 @@ func (b *backend) getConfig(ctx context.Context, storage logical.Storage) (*back
 	}
 
 	if e == nil || len(e.Value) == 0 {
-		return &backendConfig{}, nil
+		return nil, nil
 	}
 
 	if err = e.DecodeJSON(&config); err != nil {
@@ -141,3 +145,21 @@ func (b *backend) pathConfigWrite(ctx context.Context, req *logical.Request,
 
 	return &logical.Response{}, nil
 }
+
+func (b *backend) pathConfigDelete(ctx context.Context, req *logical.Request,
+	data *framework.FieldData) (*logical.Response, error) {
+	cfg, err := b.getConfig(ctx, req.Storage)
+	if err != nil {
+		return nil, err
+	}
+
+	if cfg == nil {
+		return nil, errBackendNotConfigured
+	}
+
+	if err = req.Storage.Delete(ctx, pathPatternConfig); err != nil {
+		return nil, err
+	}
+
+	return &logical.Response{}, nil
+}

internal/plugin/path_config_test.go

@@ -67,3 +67,47 @@ func TestBackend_PathConfigWrite(t *testing.T) {
 		require.Equal(t, cfg.APIKey, "foo")
 	})
 }
+
+func TestBackend_PathConfigDelete(t *testing.T) {
+	t.Parallel()
+
+	t.Run("DeleteWithoutInit", func(t *testing.T) {
+		t.Parallel()
+
+		ctx := context.Background()
+		b, storage := newTestBackend(t)
+
+		res, err := b.HandleRequest(ctx, &logical.Request{
+			Storage:   storage,
+			Operation: logical.DeleteOperation,
+			Path:      pathPatternConfig,
+		})
+		require.Error(t, err)
+		require.Nil(t, res)
+	})
+
+	t.Run("DeleteSuccess", func(t *testing.T) {
+		t.Parallel()
+
+		ctx := context.Background()
+		b, storage := newTestBackend(t)
+
+		_, err := b.HandleRequest(ctx, &logical.Request{
+			Storage:   storage,
+			Operation: logical.CreateOperation,
+			Path:      pathPatternConfig,
+			Data: map[string]any{
+				"api_key": "foo",
+			},
+		})
+		require.NoError(t, err)
+
+		res, err := b.HandleRequest(ctx, &logical.Request{
+			Storage:   storage,
+			Operation: logical.DeleteOperation,
+			Path:      pathPatternConfig,
+		})
+		require.NoError(t, err)
+		require.NotNil(t, res)
+	})
+}

internal/plugin/path_token.go

@@ -58,8 +58,8 @@ func (b *backend) pathTokenWrite(ctx context.Context, req *logical.Request,
 		return nil, err
 	}
 
-	if cfg.APIKey == "" {
-		return nil, errMissingAPIKey
+	if cfg == nil {
+		return nil, errBackendNotConfigured
 	}
 
 	ttl := int64(0)

internal/plugin/path_token_test.go

@@ -16,7 +16,7 @@ import (
 func TestToken_Create(t *testing.T) {
 	t.Parallel()
 
-	t.Run("CreateTokenWithEmptyBackend", func(t *testing.T) {
+	t.Run("CreateTokenWithoutBackend", func(t *testing.T) {
 		t.Parallel()
 
 		b, storage := newTestBackend(t)
@@ -27,7 +27,7 @@ func TestToken_Create(t *testing.T) {
 			Path:      pathPatternToken,
 			Data:      map[string]any{},
 		})
-		require.Equal(t, err, errMissingAPIKey)
+		require.Equal(t, err, errBackendNotConfigured)
 		require.Nil(t, r)
 	})