reorganize

Author: zcesur

lib/algora/shared/util.ex

@@ -122,4 +122,11 @@ defmodule Algora.Util do
   # TODO: Implement this for all countries
   def locale_from_country_code("gr"), do: "el"
   def locale_from_country_code(country_code), do: country_code
+
+  def parse_github_url(url) do
+    case Regex.run(~r{(?:github\.com/)?([^/\s]+)/([^/\s]+)}, url) do
+      [_, owner, repo] -> {:ok, {owner, repo}}
+      _ -> {:error, "Must be a valid GitHub repository URL (e.g. github.com/owner/repo) or owner/repo format"}
+    end
+  end
 end

lib/algora_web/live/home_live.ex

@@ -815,13 +815,6 @@ defmodule AlgoraWeb.HomeLive do
     end
   end
 
-  def parse_github_url(url) do
-    case Regex.run(~r{(?:github\.com/)?([^/\s]+)/([^/\s]+)}, url) do
-      [_, owner, repo] -> {:ok, {owner, repo}}
-      _ -> {:error, "Must be a valid GitHub repository URL (e.g. github.com/owner/repo) or owner/repo format"}
-    end
-  end
-
   @impl true
   def handle_event("submit_repo", %{"repo_form" => params}, socket) do
     changeset =
@@ -832,7 +825,7 @@ defmodule AlgoraWeb.HomeLive do
     if changeset.valid? do
       url = get_field(changeset, :url)
 
-      case url |> parse_github_url() |> dbg() do
+      case url |> Algora.Util.parse_github_url() |> dbg() do
         {:ok, {repo_owner, repo_name}} ->
           token = Github.TokenPool.get_token()
 

test/algora/shared/util_test.exs

@@ -18,4 +18,40 @@ defmodule Algora.UtilTest do
       assert Util.format_pct(Decimal.new("0.1050")) == "10.5%"
     end
   end
+
+  describe "parse_github_url/1" do
+    test "parses full GitHub URLs" do
+      assert Util.parse_github_url("https://github.com/owner/repo") == {:ok, {"owner", "repo"}}
+      assert Util.parse_github_url("http://github.com/owner/repo") == {:ok, {"owner", "repo"}}
+      assert Util.parse_github_url("github.com/owner/repo") == {:ok, {"owner", "repo"}}
+    end
+
+    test "parses owner/repo format" do
+      assert Util.parse_github_url("owner/repo") == {:ok, {"owner", "repo"}}
+    end
+
+    test "handles URLs with dashes and underscores" do
+      assert Util.parse_github_url("my-org/my_repo") == {:ok, {"my-org", "my_repo"}}
+      assert Util.parse_github_url("github.com/my-org/my_repo") == {:ok, {"my-org", "my_repo"}}
+    end
+
+    test "handles numeric characters" do
+      assert Util.parse_github_url("owner123/repo456") == {:ok, {"owner123", "repo456"}}
+    end
+
+    test "rejects invalid formats" do
+      error_msg = "Must be a valid GitHub repository URL (e.g. github.com/owner/repo) or owner/repo format"
+
+      assert Util.parse_github_url("") == {:error, error_msg}
+      assert Util.parse_github_url("invalid") == {:error, error_msg}
+      assert Util.parse_github_url("owner") == {:error, error_msg}
+      assert Util.parse_github_url("owner/") == {:error, error_msg}
+      assert Util.parse_github_url("/repo") == {:error, error_msg}
+    end
+
+    test "handles whitespace" do
+      assert Util.parse_github_url("  owner/repo  ") == {:ok, {"owner", "repo"}}
+      assert Util.parse_github_url("  github.com/owner/repo  ") == {:ok, {"owner", "repo"}}
+    end
+  end
 end