diff options
| author | Christopher Speller <crspeller@gmail.com> | 2016-11-04 17:20:21 -0400 |
|---|---|---|
| committer | Harrison Healey <harrisonmhealey@gmail.com> | 2016-11-04 17:20:21 -0400 |
| commit | 18745b2d5c1899bc279696548cb3ebecf7c6b90f (patch) | |
| tree | 23cf9a3f8a75a89970792dd6ce0ff5b26df82850 /model/channel_test.go | |
| parent | 20254073cb9976e783875f997a291829e6a0d78d (diff) | |
| download | chat-18745b2d5c1899bc279696548cb3ebecf7c6b90f.tar.gz chat-18745b2d5c1899bc279696548cb3ebecf7c6b90f.tar.bz2 chat-18745b2d5c1899bc279696548cb3ebecf7c6b90f.zip | |
Increase Channel Purpose length to 250, and add channel field length handling code to the slack importer (#4458)
* Increase Channel Purpose length to 250.
This commit increases the maxmimum length of the channel purpose field
to 250, including the database migration necessary.
It also adds a method to the Slack importer to check the lengths of
channel properties before importing, and truncate them if necessary.
Fixes #4168
* Fix database migration.
Diffstat (limited to 'model/channel_test.go')
| -rw-r--r-- | model/channel_test.go | 7 |
1 files changed, 6 insertions, 1 deletions
diff --git a/model/channel_test.go b/model/channel_test.go index 590417cfe..af4e3c16d 100644 --- a/model/channel_test.go +++ b/model/channel_test.go @@ -78,7 +78,7 @@ func TestChannelIsValid(t *testing.T) { t.Fatal(err) } - o.Purpose = strings.Repeat("01234567890", 20) + o.Purpose = strings.Repeat("01234567890", 30) if err := o.IsValid(); err == nil { t.Fatal("should be invalid") } @@ -87,6 +87,11 @@ func TestChannelIsValid(t *testing.T) { if err := o.IsValid(); err != nil { t.Fatal(err) } + + o.Purpose = strings.Repeat("0123456789", 25) + if err := o.IsValid(); err != nil { + t.Fatal(err) + } } func TestChannelPreSave(t *testing.T) { |