From f7fc6a650132d409215ba72776fc72d116b1bb5e Mon Sep 17 00:00:00 2001
From: Mark Rushakoff <mark@influxdb.com>
Date: Tue, 13 Feb 2018 16:47:48 -0800
Subject: [PATCH] Remove IsAdmin() method from meta.User interface

The method was only called in tests.
---
 services/meta/client_test.go | 13 +++++++++----
 services/meta/data.go        |  5 -----
 2 files changed, 9 insertions(+), 9 deletions(-)

diff --git a/services/meta/client_test.go b/services/meta/client_test.go
index 46530a7bc9..c18868225d 100644
--- a/services/meta/client_test.go
+++ b/services/meta/client_test.go
@@ -599,7 +599,7 @@ func TestMetaClient_CreateUser(t *testing.T) {
 	if exp, got := "fred", u.ID(); exp != got {
 		t.Fatalf("unexpected user name: exp: %s got: %s", exp, got)
 	}
-	if !u.IsAdmin() {
+	if !isAdmin(u) {
 		t.Fatalf("expected user to be admin")
 	}
 
@@ -650,7 +650,7 @@ func TestMetaClient_CreateUser(t *testing.T) {
 	if exp, got := "wilma", u.ID(); exp != got {
 		t.Fatalf("unexpected user name: exp: %s got: %s", exp, got)
 	}
-	if u.IsAdmin() {
+	if isAdmin(u) {
 		t.Fatalf("expected user not to be an admin")
 	}
 
@@ -670,7 +670,7 @@ func TestMetaClient_CreateUser(t *testing.T) {
 	if exp, got := "wilma", u.ID(); exp != got {
 		t.Fatalf("unexpected user name: exp: %s got: %s", exp, got)
 	}
-	if !u.IsAdmin() {
+	if !isAdmin(u) {
 		t.Fatalf("expected user to be an admin")
 	}
 
@@ -686,7 +686,7 @@ func TestMetaClient_CreateUser(t *testing.T) {
 	if exp, got := "wilma", u.ID(); exp != got {
 		t.Fatalf("unexpected user name: exp: %s got: %s", exp, got)
 	}
-	if u.IsAdmin() {
+	if isAdmin(u) {
 		t.Fatalf("expected user not to be an admin")
 	}
 
@@ -1163,3 +1163,8 @@ func testTempDir(skip int) string {
 	}
 	return dir
 }
+
+func isAdmin(u meta.User) bool {
+	ui := u.(*meta.UserInfo)
+	return ui.Admin
+}
diff --git a/services/meta/data.go b/services/meta/data.go
index b3abd0e6cd..0e1ec7ec03 100644
--- a/services/meta/data.go
+++ b/services/meta/data.go
@@ -1579,17 +1579,12 @@ type UserInfo struct {
 type User interface {
 	query.Authorizer
 	ID() string
-	IsAdmin() bool
 }
 
 func (u *UserInfo) ID() string {
 	return u.Name
 }
 
-func (u *UserInfo) IsAdmin() bool {
-	return u.Admin
-}
-
 // AuthorizeDatabase returns true if the user is authorized for the given privilege on the given database.
 func (ui *UserInfo) AuthorizeDatabase(privilege influxql.Privilege, database string) bool {
 	if ui.Admin || privilege == influxql.NoPrivileges {