From 8b9dbb86133ff0fd6002a391268383d1593918ca Mon Sep 17 00:00:00 2001 From: Joram Wilander Date: Fri, 29 Sep 2017 11:45:59 -0400 Subject: PLT-7404 Return viewed at times in view channel API response (#7428) * Return viewed at times in view channel API response * Updated transaction to read and write once * Remove transaction and only update if new value greater than older --- api4/channel_test.go | 12 +++++++++--- 1 file changed, 9 insertions(+), 3 deletions(-) (limited to 'api4/channel_test.go') diff --git a/api4/channel_test.go b/api4/channel_test.go index fbe96728d..517d166dc 100644 --- a/api4/channel_test.go +++ b/api4/channel_test.go @@ -1375,13 +1375,19 @@ func TestViewChannel(t *testing.T) { ChannelId: th.BasicChannel.Id, } - pass, resp := Client.ViewChannel(th.BasicUser.Id, view) + viewResp, resp := Client.ViewChannel(th.BasicUser.Id, view) CheckNoError(t, resp) - if !pass { + if viewResp.Status != "OK" { t.Fatal("should have passed") } + channel, _ := th.App.GetChannel(th.BasicChannel.Id) + + if lastViewedAt := viewResp.LastViewedAtTimes[channel.Id]; lastViewedAt != channel.LastPostAt { + t.Fatal("LastPostAt does not match returned LastViewedAt time") + } + view.PrevChannelId = th.BasicChannel.Id _, resp = Client.ViewChannel(th.BasicUser.Id, view) CheckNoError(t, resp) @@ -1396,7 +1402,7 @@ func TestViewChannel(t *testing.T) { member, resp := Client.GetChannelMember(th.BasicChannel.Id, th.BasicUser.Id, "") CheckNoError(t, resp) - channel, resp := Client.GetChannel(th.BasicChannel.Id, "") + channel, resp = Client.GetChannel(th.BasicChannel.Id, "") CheckNoError(t, resp) if member.MsgCount != channel.TotalMsgCount { -- cgit v1.2.3-1-g7c22