From 64f9b7ec089f56303b5eca696fff741f6618952b Mon Sep 17 00:00:00 2001 From: =Corey Hulen Date: Wed, 16 Mar 2016 09:38:35 -0700 Subject: PLT-2344 support 3 versions back --- api/command_loadtest_test.go | 32 +++++++++++++++++--------------- model/version.go | 5 +++++ 2 files changed, 22 insertions(+), 15 deletions(-) diff --git a/api/command_loadtest_test.go b/api/command_loadtest_test.go index 1debc766a..ef370cf19 100644 --- a/api/command_loadtest_test.go +++ b/api/command_loadtest_test.go @@ -201,21 +201,23 @@ func TestLoadTestUrlCommands(t *testing.T) { t.Fatal("/loadtest url for README.md should've executed") } - command = "/loadtest url test-emoticons1.md" - if r := Client.Must(Client.Command(channel.Id, command, false)).Data.(*model.CommandResponse); r.Text != "Loading data..." { - t.Fatal("/loadtest url for test-emoticons.md should've executed") - } - - command = "/loadtest url test-emoticons1" - if r := Client.Must(Client.Command(channel.Id, command, false)).Data.(*model.CommandResponse); r.Text != "Loading data..." { - t.Fatal("/loadtest url for test-emoticons should've executed") - } - - posts := Client.Must(Client.GetPosts(channel.Id, 0, 5, "")).Data.(*model.PostList) - // note that this may make more than 3 posts if files are too long to fit in an individual post - if len(posts.Order) < 3 { - t.Fatal("/loadtest url made too few posts, perhaps there needs to be a delay before GetPosts in the test?") - } + // Removing these tests since they break compatibilty with previous release branches because the url pulls from github master + + // command = "/loadtest url test-emoticons1.md" + // if r := Client.Must(Client.Command(channel.Id, command, false)).Data.(*model.CommandResponse); r.Text != "Loading data..." { + // t.Fatal("/loadtest url for test-emoticons.md should've executed") + // } + + // command = "/loadtest url test-emoticons1" + // if r := Client.Must(Client.Command(channel.Id, command, false)).Data.(*model.CommandResponse); r.Text != "Loading data..." { + // t.Fatal("/loadtest url for test-emoticons should've executed") + // } + + // posts := Client.Must(Client.GetPosts(channel.Id, 0, 5, "")).Data.(*model.PostList) + // // note that this may make more than 3 posts if files are too long to fit in an individual post + // if len(posts.Order) < 3 { + // t.Fatal("/loadtest url made too few posts, perhaps there needs to be a delay before GetPosts in the test?") + // } time.Sleep(2 * time.Second) } diff --git a/model/version.go b/model/version.go index 6011a9245..737071934 100644 --- a/model/version.go +++ b/model/version.go @@ -118,5 +118,10 @@ func IsPreviousVersionsSupported(versionToCheck string) bool { return true } + // Current - 3 Supported + if versionsWithoutHotFixes[3] == versionToCheckStr { + return true + } + return false } -- cgit v1.2.3-1-g7c22