Fix twitter message length

randomecho · Oct 4, 2013 · 677da43 · 677da43
1 parent f6aa538
commit 677da43
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/app/models/note.rb b/app/models/note.rb
@@ -53,7 +53,7 @@ def initialize(*args)
   end
 
   def twitter_message_max_length
-    140 - (5 + self.redirects.first.to_url.length)
+    140 - (7 + self.redirects.first.to_url.length)
   end
 
   def twitter_message

diff --git a/spec/models/note_spec.rb b/spec/models/note_spec.rb
@@ -129,6 +129,7 @@
         let(:tweet) { "\"JSFuck is an esoteric and educational programming style based on the atomic parts of JavaScript. It uses only six different characters to write and execute code.\" http://www.jsfuck.com/ " }
 
         it { expect(note.twitter_message).to start_with(tweet[0..note.twitter_message_max_length]) }
+        it { expect(note.twitter_message.length).to be <= 140 }
         it { expect(note.twitter_message).to end_with(" (#{note.redirects.first.to_url})") }
       end
     end