fix: increase token size limit

fix: guide tag collisions
fix: tag bulk create
test: add tests for tags

db/schema/user_session.go

@@ -20,7 +20,7 @@ func (UserSession) Mixin() []ent.Mixin {
 
 func (UserSession) Fields() []ent.Field {
 	return []ent.Field{
-		field.String("token").MaxLen(256).Unique(),
+		field.String("token").MaxLen(512).Unique(),
 		field.String("user_agent").Optional(),
 	}
 }

gql/resolver_guides.go

@@ -102,7 +102,7 @@ func (r *mutationResolver) UpdateGuide(ctx context.Context, guideID string, g ge
 
 		return tx.GuideTag.MapCreateBulk(g.TagIDs, func(create *ent.GuideTagCreate, i int) {
 			create.SetGuideID(guideID).SetTagID(g.TagIDs[i])
-		}).Exec(ctx)
+		}).OnConflict(sql.DoNothing()).Exec(ctx)
 	}, nil); err != nil {
 		return nil, err
 	}

gql/resolver_tags.go

@@ -35,10 +35,17 @@ func (r *mutationResolver) CreateMultipleTags(ctx context.Context, tags []*gener
 		}
 	}
 
-	result, err := db.From(ctx).Tag.MapCreateBulk(tags, func(create *ent.TagCreate, i int) {
-		create.SetName(tags[i].Name).SetDescription(tags[i].Description)
-	}).Save(ctx)
-	if err != nil {
+	var result []*ent.Tag
+	if err := db.Tx(ctx, func(ctx context.Context, tx *ent.Tx) error {
+		for _, newTag := range tags {
+			t, err := tx.Tag.Create().SetName(newTag.Name).SetDescription(newTag.Description).Save(ctx)
+			if err != nil {
+				return err
+			}
+			result = append(result, t)
+		}
+		return nil
+	}, nil); err != nil {
 		return nil, err
 	}
 

migrations/sql/000028_increase_token_length_limit.down.sql

@@ -0,0 +1,2 @@
+ALTER TABLE user_sessions
+    ALTER COLUMN token TYPE varchar(256);

migrations/sql/000028_increase_token_length_limit.up.sql

@@ -0,0 +1,2 @@
+ALTER TABLE user_sessions
+    ALTER COLUMN token TYPE varchar(512);

tests/guides_test.go

@@ -18,7 +18,6 @@ func init() {
 	db.EnableDebug()
 }
 
-// TODO Add guide tag test
 // TODO Add rate limit test
 
 func TestGuides(t *testing.T) {
@@ -28,21 +27,25 @@ func TestGuides(t *testing.T) {
 	token, userID, err := makeUser(ctx)
 	testza.AssertNoError(t, err)
 
+	tags := seedTags(ctx, t, token, client)
+
 	// Run Twice to detect any cache issues
 	for i := range 2 {
 		t.Run("Loop"+strconv.Itoa(i), func(t *testing.T) {
 			var guideID string
 
 			t.Run("Create", func(t *testing.T) {
-				createGuide := authRequest(`mutation {
+				createGuide := authRequest(`mutation ($tags: [TagID!]) {
 					createGuide(guide: {
 						name: "Hello World",
 						short_description: "Short description about the guide",
-						guide: "The full guide text goes here."
+						guide: "The full guide text goes here.",
+						tagIDs: $tags
 					}) {
 						id
 					}
 				}`, token)
+				createGuide.Var("tags", tags)
 
 				var createGuideResponse struct {
 					CreateGuide generated.Guide
@@ -79,17 +82,19 @@ func TestGuides(t *testing.T) {
 			})
 
 			t.Run("Update", func(t *testing.T) {
-				updateGuide := authRequest(`mutation ($id: GuideID!) {
+				updateGuide := authRequest(`mutation ($id: GuideID!, $tags: [TagID!]) {
 					updateGuide(
 						guideId: $id,
 						guide: {
-							name: "Foo Bar"
+							name: "Foo Bar",
+							tagIDs: $tags
 						}
 					) {
 						id
 					}
 				}`, token)
 				updateGuide.Var("id", guideID)
+				updateGuide.Var("tags", tags)
 
 				var updateGuideResponse struct {
 					UpdateGuide generated.Guide
@@ -126,6 +131,39 @@ func TestGuides(t *testing.T) {
 				testza.AssertEqual(t, userID, queryGuidesResponse.GetGuides.Guides[0].User.ID)
 			})
 
+			t.Run("Search", func(t *testing.T) {
+				searchGuides := authRequest(`query ($tags: [TagID!]) {
+					getGuides (filter: {
+							search: "Bar",
+							tagIDs: $tags
+						}){
+						count
+						guides {
+							id
+							name
+							short_description
+							guide
+							user {
+								id
+							}				
+						}
+					}
+				}`, token)
+				searchGuides.Var("tags", tags[:1])
+
+				var queryGuidesResponse struct {
+					GetGuides generated.GetGuides
+				}
+				testza.AssertNoError(t, client.Run(ctx, searchGuides, &queryGuidesResponse))
+				testza.AssertEqual(t, 1, queryGuidesResponse.GetGuides.Count)
+				testza.AssertEqual(t, 1, len(queryGuidesResponse.GetGuides.Guides))
+				testza.AssertEqual(t, guideID, queryGuidesResponse.GetGuides.Guides[0].ID)
+				testza.AssertEqual(t, "Foo Bar", queryGuidesResponse.GetGuides.Guides[0].Name)
+				testza.AssertEqual(t, "Short description about the guide", queryGuidesResponse.GetGuides.Guides[0].ShortDescription)
+				testza.AssertEqual(t, "The full guide text goes here.", queryGuidesResponse.GetGuides.Guides[0].Guide)
+				testza.AssertEqual(t, userID, queryGuidesResponse.GetGuides.Guides[0].User.ID)
+			})
+
 			t.Run("Delete", func(t *testing.T) {
 				deleteGuide := authRequest(`mutation ($id: GuideID!) {
 					deleteGuide(guideId: $id)