Merge pull request #11 from austin1237/regular-expression

regular expression used instead of string.Contains and added tests.

scraper/interest/interest.go

@@ -2,17 +2,20 @@ package interest
 
 import (
 	"fmt"
+	"regexp"
 	"scraper/job"
 	"strings"
 	"sync"
 )
 
 func CheckIfInterested(description string) bool {
-	keywords := []string{"typescript", "node", "nodejs", "node.js", "go ", "golang", "go,", "go)", "(go"}
+	keywords := []string{"typescript", "node", "nodejs", "node.js", "go", "golang"}
 	// Check if keywords are present in the job's text
 	descriptionToLower := strings.ToLower(description)
 	for _, keyword := range keywords {
-		if strings.Contains(descriptionToLower, keyword) {
+		pattern := "\\b" + keyword + "\\b"
+		match, _ := regexp.MatchString(pattern, descriptionToLower)
+		if match {
 			return true
 		}
 	}

scraper/interest/interest_test.go

@@ -0,0 +1,52 @@
+package interest
+
+import (
+	"testing"
+
+	"github.com/stretchr/testify/assert"
+)
+
+func TestCheckIfInterested(t *testing.T) {
+	tests := []struct {
+		name        string
+		description string
+		want        bool
+	}{
+		{
+			name:        "contains typescript",
+			description: "This is a job for a TypeScript developer",
+			want:        true,
+		},
+		{
+			name:        "contains node",
+			description: "This is a job for a Node.js developer",
+			want:        true,
+		},
+		{
+			name:        "contains go",
+			description: "This is a job for a Go developer",
+			want:        true,
+		},
+		{
+			name:        "contains go",
+			description: "This is a job for go.",
+			want:        true,
+		},
+		{
+			name:        "contains django",
+			description: "This is a job for a django developer in Chicago",
+			want:        false,
+		},
+		{
+			name:        "does not contain keyword",
+			description: "This is a job for a Python developer",
+			want:        false,
+		},
+	}
+
+	for _, tt := range tests {
+		t.Run(tt.name, func(t *testing.T) {
+			assert.Equal(t, tt.want, CheckIfInterested(tt.description))
+		})
+	}
+}