diff --git a/pkg/util/slack/client.go b/pkg/util/slack/client.go
index 912f7f90..6675018d 100644
--- a/pkg/util/slack/client.go
+++ b/pkg/util/slack/client.go
@@ -126,7 +126,25 @@ func (c *threadedClient) SendMessage(ctx context.Context, recipient string, grou
 		options = append(options, sl.MsgOptionTS(ts))
 	}
 
-	if ts == "" || policy == Post || policy == PostAndUpdate {
+	// Updating an existing message
+	if ts != "" && (policy == Update || policy == PostAndUpdate) {
+		_, _, err := SendMessageRateLimited(
+			c.Client,
+			ctx,
+			c.Limiter,
+			c.getChannelID(recipient),
+			sl.MsgOptionUpdate(ts),
+			sl.MsgOptionAsUser(true),
+			sl.MsgOptionCompose(options...),
+		)
+		if err != nil {
+			return err
+		}
+		return nil
+	}
+
+	// Posting a new message
+	if policy == Post || policy == PostAndUpdate {
 		newTs, channelID, err := SendMessageRateLimited(
 			c.Client,
 			ctx,
@@ -147,21 +165,6 @@ func (c *threadedClient) SendMessage(ctx context.Context, recipient string, grou
 		c.lock.Unlock()
 	}
 
-	if ts != "" && (policy == Update || policy == PostAndUpdate) {
-		_, _, err := SendMessageRateLimited(
-			c.Client,
-			ctx,
-			c.Limiter,
-			c.getChannelID(recipient),
-			sl.MsgOptionUpdate(ts),
-			sl.MsgOptionAsUser(true),
-			sl.MsgOptionCompose(options...),
-		)
-		if err != nil {
-			return err
-		}
-	}
-
 	return nil
 }
 
diff --git a/pkg/util/slack/client_test.go b/pkg/util/slack/client_test.go
index cb03d527..d7ac8718 100644
--- a/pkg/util/slack/client_test.go
+++ b/pkg/util/slack/client_test.go
@@ -103,58 +103,56 @@ func TestThreadedClient(t *testing.T) {
 		groupingKey   string
 		policy        DeliveryPolicy
 		wantPostType1 gomock.Matcher
-		wantPostType2 gomock.Matcher
-		wantthreadTSs timestampMap
+		wantThreadTSs timestampMap
 	}{
 		"Post, basic": {
 			threadTSs:     timestampMap{},
 			groupingKey:   "",
 			policy:        Post,
 			wantPostType1: EqChatPost(),
-			wantthreadTSs: timestampMap{},
+			wantThreadTSs: timestampMap{},
 		},
 		"Post, no parent, with grouping": {
 			threadTSs:     timestampMap{},
 			groupingKey:   groupingKey,
 			policy:        Post,
 			wantPostType1: EqChatPost(),
-			wantthreadTSs: timestampMap{channel: {groupingKey: ts1}},
+			wantThreadTSs: timestampMap{channel: {groupingKey: ts1}},
 		},
 		"Post, with parent, with grouping": {
 			threadTSs:     timestampMap{channel: {groupingKey: ts2}},
 			groupingKey:   groupingKey,
 			policy:        Post,
 			wantPostType1: EqChatPost(),
-			wantthreadTSs: timestampMap{channel: {groupingKey: ts2}},
+			wantThreadTSs: timestampMap{channel: {groupingKey: ts2}},
 		},
 		"PostAndUpdate, no parent. First post should not be updated": {
 			threadTSs:     timestampMap{},
 			groupingKey:   groupingKey,
 			policy:        PostAndUpdate,
 			wantPostType1: EqChatPost(),
-			wantthreadTSs: timestampMap{channel: {groupingKey: ts1}},
+			wantThreadTSs: timestampMap{channel: {groupingKey: ts1}},
 		},
 		"PostAndUpdate, with parent. First post should be updated": {
 			threadTSs:     timestampMap{channel: {groupingKey: ts2}},
 			groupingKey:   groupingKey,
 			policy:        PostAndUpdate,
-			wantPostType1: EqChatPost(),
-			wantPostType2: EqChatUpdate(),
-			wantthreadTSs: timestampMap{channel: {groupingKey: ts2}},
+			wantPostType1: EqChatUpdate(),
+			wantThreadTSs: timestampMap{channel: {groupingKey: ts2}},
 		},
-		"Update, no parent. Only call should be post": {
+		"Update, no parent. There should be no call, no new thread": {
 			threadTSs:     timestampMap{},
 			groupingKey:   groupingKey,
 			policy:        Update,
-			wantPostType1: EqChatPost(),
-			wantthreadTSs: timestampMap{channel: {groupingKey: ts1}},
+			wantPostType1: nil,
+			wantThreadTSs: timestampMap{},
 		},
 		"Update, with parent. Only call should be update": {
 			threadTSs:     timestampMap{channel: {groupingKey: ts2}},
 			groupingKey:   groupingKey,
 			policy:        Update,
 			wantPostType1: EqChatUpdate(),
-			wantthreadTSs: timestampMap{channel: {groupingKey: ts2}},
+			wantThreadTSs: timestampMap{channel: {groupingKey: ts2}},
 		},
 	}
 	for name, tc := range tests {
@@ -163,13 +161,13 @@ func TestThreadedClient(t *testing.T) {
 			defer ctrl.Finish()
 			m := mocks.NewMockSlackClient(ctrl)
 
-			m.EXPECT().
-				SendMessageContext(gomock.Any(), gomock.Eq(channel), tc.wantPostType1).
-				Return(channelID, ts1, "", nil)
+			expectedFunctionCall := m.EXPECT().
+				SendMessageContext(gomock.Any(), gomock.Eq(channel), tc.wantPostType1)
 
-			if tc.wantPostType2 != nil {
-				m.EXPECT().
-					SendMessageContext(gomock.Any(), gomock.Eq(channelID), tc.wantPostType2)
+			if tc.wantPostType1 != nil {
+				expectedFunctionCall.Return(channelID, ts1, "", nil)
+			} else {
+				expectedFunctionCall.MaxTimes(0)
 			}
 
 			client := NewThreadedClient(
@@ -182,7 +180,7 @@ func TestThreadedClient(t *testing.T) {
 			)
 			err := client.SendMessage(context.TODO(), channel, tc.groupingKey, false, tc.policy, []slack.MsgOption{})
 			assert.NoError(t, err)
-			assert.Equal(t, tc.wantthreadTSs, client.ThreadTSs)
+			assert.Equal(t, tc.wantThreadTSs, client.ThreadTSs)
 		})
 	}
 }