improve tagging logic

test/memex_web/live/context_live_test.exs

@@ -18,7 +18,7 @@ defmodule MemexWeb.ContextLiveTest do
   }
   @invalid_attrs %{
     "content" => nil,
-    "tags_string" => " ",
+    "tags_string" => "",
     "slug" => nil,
     "visibility" => nil
   }
@@ -114,13 +114,9 @@ defmodule MemexWeb.ContextLiveTest do
 
       assert_patch(show_live, Routes.context_show_path(conn, :edit, context.slug))
 
-      html =
+      assert show_live
-        show_live
+             |> form("#context-form", context: @invalid_attrs)
-        |> form("#context-form", context: @invalid_attrs)
+             |> render_change() =~ "can't be blank"
-        |> render_change()
-
-      assert html =~ "can't be blank"
-      assert html =~ "tags must be comma-delimited"
 
       {:ok, _, html} =
         show_live

test/memex_web/live/note_live_test.exs

@@ -19,7 +19,7 @@ defmodule MemexWeb.NoteLiveTest do
   }
   @invalid_attrs %{
     "content" => nil,
-    "tags_string" => " ",
+    "tags_string" => "",
     "slug" => nil,
     "visibility" => nil
   }
@@ -54,13 +54,9 @@ defmodule MemexWeb.NoteLiveTest do
 
       assert_patch(index_live, Routes.note_index_path(conn, :new))
 
-      html =
+      assert index_live
-        index_live
+             |> form("#note-form", note: @invalid_attrs)
-        |> form("#note-form", note: @invalid_attrs)
+             |> render_change() =~ "can't be blank"
-        |> render_change()
-
-      assert html =~ "can't be blank"
-      assert html =~ "tags must be comma-delimited"
 
       {:ok, _, html} =
         index_live

test/memex_web/live/pipeline_live_test.exs

@@ -17,7 +17,7 @@ defmodule MemexWeb.PipelineLiveTest do
   }
   @invalid_attrs %{
     "description" => nil,
-    "tags_string" => " ",
+    "tags_string" => "",
     "slug" => nil,
     "visibility" => nil
   }
@@ -128,13 +128,9 @@ defmodule MemexWeb.PipelineLiveTest do
 
       assert_patch(show_live, Routes.pipeline_show_path(conn, :edit, pipeline.slug))
 
-      html =
+      assert show_live
-        show_live
+             |> form("#pipeline-form", pipeline: @invalid_attrs)
-        |> form("#pipeline-form", pipeline: @invalid_attrs)
+             |> render_change() =~ "can't be blank"
-        |> render_change()
-
-      assert html =~ "can't be blank"
-      assert html =~ "tags must be comma-delimited"
 
       {:ok, _, html} =
         show_live