Skip to content

Commit 182b2f0

Browse files
has_site_administrator_explorer_privileges rename
1 parent 937973a commit 182b2f0

File tree

3 files changed

+3
-3
lines changed

3 files changed

+3
-3
lines changed

metadata-ingestion/src/datahub/ingestion/source/tableau/tableau.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -645,7 +645,7 @@ def report_user_role(report: TableauSourceReport, server: Server) -> None:
645645
# the site-role might be different on another site
646646
logged_in_user: UserInfo = UserInfo.from_server(server=server)
647647

648-
if not logged_in_user.is_site_administrator_explorer():
648+
if not logged_in_user.has_site_administrator_explorer_privileges():
649649
report.warning(
650650
title=title,
651651
message=message,

metadata-ingestion/src/datahub/ingestion/source/tableau/tableau_server_wrapper.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@ class UserInfo:
1111
site_role: str
1212
site_id: str
1313

14-
def is_site_administrator_explorer(self):
14+
def has_site_administrator_explorer_privileges(self):
1515
return self.site_role in [
1616
c.ROLE_SITE_ADMIN_EXPLORER,
1717
c.ROLE_SITE_ADMIN_CREATOR,

metadata-ingestion/src/datahub/ingestion/source/tableau/tableau_validation.py

+1-1
Original file line numberDiff line numberDiff line change
@@ -28,7 +28,7 @@ def check_user_role(
2828

2929
try:
3030
# TODO: Add check for `Enable Derived Permissions`
31-
if not logged_in_user.is_site_administrator_explorer():
31+
if not logged_in_user.has_site_administrator_explorer_privileges():
3232
capability_dict[c.SITE_PERMISSION] = CapabilityReport(
3333
capable=False,
3434
failure_reason=f"{failure_reason} Their current role is {logged_in_user.site_role}.",

0 commit comments

Comments
 (0)