Skip to content
Snippets Groups Projects
Unverified Commit a4cab90e authored by Cam Saul's avatar Cam Saul Committed by GitHub
Browse files

Merge pull request #8883 from metabase/add-test-for-updating-perms-graph-from-api

Add test for updating perms graph from API
parents f95321bf fdb241a7
No related branches found
No related tags found
No related merge requests found
(ns metabase.api.permissions-test
"Tests for `/api/permissions` endpoints."
(:require [expectations :refer :all]
[metabase.models.permissions-group :as group :refer [PermissionsGroup]]
[metabase.models
[permissions :as perms]
[permissions-group :as group :refer [PermissionsGroup]]]
[metabase.test.data :as data]
[metabase.test.data.users :as test-users]
[metabase.util :as u]
[toucan.util.test :as tt]))
......@@ -36,3 +39,17 @@
(test-users/delete-temp-users!)
(set (for [member (:members ((test-users/user->client :crowberto) :get 200 (str "permissions/group/" (u/get-id (group/all-users)))))]
(update member :membership_id (complement nil?))))))
;; make sure we can update the perms graph from the API
(expect
{(data/id :categories) :none
(data/id :checkins) :none
(data/id :users) :none
(data/id :venues) :all}
(tt/with-temp PermissionsGroup [group]
((test-users/user->client :crowberto) :put 200 "permissions/graph"
(assoc-in (perms/graph)
[:groups (u/get-id group) (data/id) :schemas]
{"PUBLIC" {(data/id :venues) :all}}))
(get-in (perms/graph) [:groups (u/get-id group) (data/id) :schemas "PUBLIC"])))
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment