From 39e319b24d1b3c6842e430073cd21f6544bde4d9 Mon Sep 17 00:00:00 2001
From: ajermaky <ara.jermakyan@gmail.com>
Date: Wed, 1 May 2024 12:43:45 -0700
Subject: [PATCH] pr feedback

---
 lib/go/edgecontext/edgecontext_test.go | 56 +++++++++++++-------------
 lib/go/edgecontext/service.go          |  4 +-
 2 files changed, 30 insertions(+), 30 deletions(-)

diff --git a/lib/go/edgecontext/edgecontext_test.go b/lib/go/edgecontext/edgecontext_test.go
index 4049147..cb7e1d7 100644
--- a/lib/go/edgecontext/edgecontext_test.go
+++ b/lib/go/edgecontext/edgecontext_test.go
@@ -796,14 +796,14 @@ func TestFromHeader(t *testing.T) {
 
 				svc, ok := e.Service()
 				if !ok {
-					t.Errorf("Expected service to be true, got false")
+					t.Fatal("Expected service to be true, got false")
 				}
-				if name, ok := svc.Name(); !ok {
-					t.Error("Failed to get service name")
-				} else {
-					if name != expectedServiceName {
-						t.Errorf("Expected service name %q, got %q", expectedServiceName, name)
-					}
+				name, ok := svc.Name()
+				if !ok {
+					t.Fatal("Failed to get service name")
+				}
+				if name != expectedServiceName {
+					t.Errorf("Expected service name %q, got %q", expectedServiceName, name)
 				}
 
 				if id, ok := svc.OnBehalfOfID(); ok {
@@ -814,7 +814,7 @@ func TestFromHeader(t *testing.T) {
 					t.Errorf("expected no roles, got %q", roles)
 				}
 
-				if svc.IsElevatedAccess() {
+				if svc.RequestsElevatedAccess() {
 					t.Errorf("expected no elevated access, got true")
 				}
 			})
@@ -827,33 +827,33 @@ func TestFromHeader(t *testing.T) {
 
 				svc, ok := e.Service()
 				if !ok {
-					t.Errorf("Expected service to be true, got false")
+					t.Fatal("Expected service to be true, got false")
 				}
-				if name, ok := svc.Name(); !ok {
-					t.Error("Failed to get service name")
-				} else {
-					if name != expectedServiceName {
-						t.Errorf("Expected service name %q, got %q", expectedServiceName, name)
-					}
+				name, ok := svc.Name()
+				if !ok {
+					t.Fatal("Failed to get service name")
+				}
+				if name != expectedServiceName {
+					t.Errorf("Expected service name %q, got %q", expectedServiceName, name)
 				}
 
-				if id, ok := svc.OnBehalfOfID(); !ok {
-					t.Error("Failed to get on behalf of id")
-				} else {
-					if id != expectedLoID {
-						t.Errorf("Expected on behalf of id %q, got %q", expectedLoID, id)
-					}
+				id, ok := svc.OnBehalfOfID()
+				if !ok {
+					t.Fatal("Failed to get on behalf of id")
+				}
+				if id != expectedLoID {
+					t.Errorf("Expected on behalf of id %q, got %q", expectedLoID, id)
 				}
 
-				if roles, ok := svc.OnBehalfOfRoles(); !ok {
-					t.Error("Failed to get on behalf of roles")
-				} else {
-					if diff := cmp.Diff([]string{"admin"}, roles); diff != "" {
-						t.Errorf("mismatch (-want +got)\n%s\n", diff)
-					}
+				roles, ok := svc.OnBehalfOfRoles()
+				if !ok {
+					t.Fatal("Failed to get on behalf of roles")
+				}
+				if diff := cmp.Diff([]string{"admin"}, roles); diff != "" {
+					t.Errorf("mismatch (-want +got)\n%s\n", diff)
 				}
 
-				if !svc.IsElevatedAccess() {
+				if !svc.RequestsElevatedAccess() {
 					t.Errorf("expected elevated access, got false")
 				}
 			})
diff --git a/lib/go/edgecontext/service.go b/lib/go/edgecontext/service.go
index 14b369d..9b8cc81 100644
--- a/lib/go/edgecontext/service.go
+++ b/lib/go/edgecontext/service.go
@@ -60,8 +60,8 @@ func (s Service) OnBehalfOfRoles() (roles []string, ok bool) {
 	return nil, false
 }
 
-// IsElevatedAccess returns whether the service requested elevated access.
-func (s Service) IsElevatedAccess() bool {
+// RequestsElevatedAccess returns whether the service requested elevated access.
+func (s Service) RequestsElevatedAccess() bool {
 	if s.isService() {
 		return AuthenticationToken(s).ServiceRequestedElevatedAccess
 	}