add test to confirm that has_role/2 works with integers too! e.g. has…

…_role(conn, 3) > true #1
dwyl · Sep 14, 2020 · 0aa6124 · 0aa6124
1 parent ffebff5
commit 0aa6124
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 0 deletions.
diff --git a/lib/rbac.ex b/lib/rbac.ex
@@ -110,6 +110,9 @@ defmodule RBAC do
 
   @doc """
   `has_role/2 confirms if the person has the given role
+   e.g: 
+   has_role(conn, "home_admin") > true
+   has_role(conn, "potus") > false
   """
   def has_role(conn, role_name) do
     role = get_role_from_cache(role_name)

diff --git a/test/rbac_test.exs b/test/rbac_test.exs
@@ -154,4 +154,19 @@ defmodule RBACTest do
 
     assert not RBAC.has_role(fake_conn, "non_existent_role")
   end
+
+
+  test "RBAC.has_role/1 works with integers too!" do
+    init()
+
+    fake_conn = %{
+      assigns: %{
+        person: %{
+          roles: "1,2,3"
+        }
+      }
+    }
+
+    assert RBAC.has_role(fake_conn, 3)
+  end
 end