Make sure sent_on is properly set for read status

Author: norkans7

Diff for: backends/rapidpro/backend_test.go

@@ -518,6 +518,15 @@ func (ts *BackendTestSuite) TestMsgStatus() {
 	ts.True(m.SentOn_.Equal(sentOn)) // no change
 	ts.Equal(pq.StringArray([]string{string(clog1.UUID()), string(clog2.UUID()), string(clog3.UUID())}), m.LogUUIDs)
 
+	// update to READ using id
+	clog4 := updateStatusByID(10001, courier.MsgStatusRead, "")
+
+	m = readMsgFromDB(ts.b, 10001)
+	ts.Equal(m.Status_, courier.MsgStatusRead)
+	ts.True(m.ModifiedOn_.After(now))
+	ts.True(m.SentOn_.Equal(sentOn)) // no change
+	ts.Equal(pq.StringArray([]string{string(clog1.UUID()), string(clog2.UUID()), string(clog3.UUID()), string(clog4.UUID())}), m.LogUUIDs)
+
 	// no change for incoming messages
 	updateStatusByID(10002, courier.MsgStatusSent, "")
 

Diff for: backends/rapidpro/status.go

@@ -90,7 +90,7 @@ UPDATE msgs_msg SET
 	    END,
 	sent_on = CASE 
 		WHEN
-			s.status IN ('W', 'S', 'D')
+			s.status IN ('W', 'S', 'D', 'R')
 		THEN
 			COALESCE(sent_on, NOW())
 		ELSE