rawlinp commented on a change in pull request #2928: TO Client: no ids
required in input json
URL: https://github.com/apache/trafficcontrol/pull/2928#discussion_r230401059
##########
File path: traffic_ops/testing/api/v14/user_test.go
##########
@@ -95,20 +100,46 @@ func UpdateTestUsers(t *testing.T) {
func GetTestUsers(t *testing.T) {
_, _, err := TOSession.GetUsers()
if err != nil {
- t.Fatalf("cannot GET users: %v\n", err)
+ t.Errorf("cannot GET users: %v\n", err)
}
}
func GetTestUserCurrent(t *testing.T) {
user, _, err := TOSession.GetUserCurrent()
if err != nil {
- t.Fatalf("cannot GET current user: %v\n", err)
+ t.Errorf("cannot GET current user: %v\n", err)
}
if user.UserName == nil {
- t.Fatalf("current user expected: %v actual: %v\n",
SessionUserName, nil)
+ t.Errorf("current user expected: %v actual: %v\n",
SessionUserName, nil)
}
if *user.UserName != SessionUserName {
- t.Fatalf("current user expected: %v actual: %v\n",
SessionUserName, *user.UserName)
+ t.Errorf("current user expected: %v actual: %v\n",
SessionUserName, *user.UserName)
+ }
+}
+
+// ForceDeleteTestUsers forcibly deletes the users from the db.
+func ForceDeleteTestUsers(t *testing.T) {
Review comment:
Do you foresee ever having to use this outside of this test file?
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
