Merge pull request #12 from ngs/fix-response

Fix response
ngs · Feb 19, 2018 · f8889cd · f8889cd
2 parents ca124b5 + e91ed75
commit f8889cd
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 3 deletions.
diff --git a/app/slack_test.go b/app/slack_test.go
@@ -91,7 +91,7 @@ func testGetActionCallbackWithActionType(t *testing.T, actionType string, succes
 		test.Compare(t)
 	}
 
-	setupActionCallbackGocks(actionType, "OK")
+	setupActionCallbackGocks(actionType, `"OK"`)
 	msg, responseURL, err = ctx.getActionCallback(&slack.AttachmentActionCallback{
 		Actions:     []slack.AttachmentAction{{Name: actionType}},
 		Token:       app.SlackVerificationToken,

diff --git a/app/timetable.go b/app/timetable.go
@@ -182,10 +182,11 @@ func (client *timeTableClient) UpdateTimeTable(timeTable *timeTable) (bool, erro
 		return false, err
 	}
 	body, err := client.doRequest(http.MethodPost, bytes.NewBuffer(b))
+	fmt.Printf("%v %v %v\n", string(body), err, string(body) == `"OK"`)
 	if err != nil {
 		return false, err
 	}
-	return string(body) == "OK", nil
+	return string(body) == `"OK"`, nil
 }
 
 func (client *timeTableClient) SetAttendance(attendance bool) (bool, error) {
@@ -198,5 +199,5 @@ func (client *timeTableClient) SetAttendance(attendance bool) (bool, error) {
 	if err != nil {
 		return false, err
 	}
-	return string(body) == "OK", nil
+	return string(body) == `"OK"`, nil
 }