Merge pull request #1 from iamelevich/validate-context

Add context nil validation

README.md

@@ -1,5 +1,5 @@
 
-![Coverage](https://img.shields.io/badge/Coverage-58.8%25-yellow)
+![Coverage](https://img.shields.io/badge/Coverage-21.1%25-red)
 <!-- TOC -->
   * [Overview](#overview)
     * [Requirements](#requirements)

plugin.go

@@ -36,6 +36,14 @@ type Plugin struct {
 }
 
 func (p *Plugin) Validate() error {
+	if p.options == nil {
+		return fmt.Errorf("options is required")
+	}
+
+	if p.options.Ctx == nil {
+		return fmt.Errorf("context is required")
+	}
+
 	if p.app == nil {
 		return fmt.Errorf("app is required")
 	}

plugin_test.go

@@ -1,6 +1,7 @@
 package pocketbase_plugin_ngrok
 
 import (
+	"context"
 	"github.com/pocketbase/pocketbase"
 	"github.com/pocketbase/pocketbase/core"
 	"testing"
@@ -21,6 +22,19 @@ func TestPlugin_Validate(t *testing.T) {
 			fields: fields{
 				app: nil,
 				options: &Options{
+					Ctx:       context.Background(),
+					Enabled:   true,
+					AuthToken: "",
+				},
+			},
+			wantErr: true,
+		},
+		{
+			name: "Context is nil",
+			fields: fields{
+				app: pocketbase.New(),
+				options: &Options{
+					Ctx:       nil,
 					Enabled:   true,
 					AuthToken: "",
 				},
@@ -32,6 +46,7 @@ func TestPlugin_Validate(t *testing.T) {
 			fields: fields{
 				app: pocketbase.New(),
 				options: &Options{
+					Ctx:       context.Background(),
 					Enabled:   true,
 					AuthToken: "",
 				},
@@ -43,6 +58,7 @@ func TestPlugin_Validate(t *testing.T) {
 			fields: fields{
 				app: pocketbase.New(),
 				options: &Options{
+					Ctx:       context.Background(),
 					Enabled:   false,
 					AuthToken: "",
 				},
@@ -54,6 +70,7 @@ func TestPlugin_Validate(t *testing.T) {
 			fields: fields{
 				app: pocketbase.New(),
 				options: &Options{
+					Ctx:       context.Background(),
 					Enabled:   true,
 					AuthToken: "TEST_AUTH_TOKEN",
 				},