From 751b757c1d9a91a7ed911212bdb19afdf2e4c85d Mon Sep 17 00:00:00 2001 From: Robin Ward Date: Thu, 18 Jul 2013 12:26:38 -0400 Subject: [PATCH] FIX: Some failing specs --- app/assets/javascripts/discourse/components/utilities.js | 2 +- spec/components/pretty_text_spec.rb | 6 +++--- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/app/assets/javascripts/discourse/components/utilities.js b/app/assets/javascripts/discourse/components/utilities.js index f13533865c7..43a630d4a9e 100644 --- a/app/assets/javascripts/discourse/components/utilities.js +++ b/app/assets/javascripts/discourse/components/utilities.js @@ -61,7 +61,7 @@ Discourse.Utilities = { var url = Discourse.Utilities.avatarUrl(options.username, options.size, options.avatarTemplate); // We won't render an invalid url - if (Em.isEmpty(url)) { return ""; } + if (!url || url.length === 0) { return ""; } var classes = "avatar" + (options.extraClasses ? " " + options.extraClasses : ""); var title = (options.title) ? " title='" + Handlebars.Utils.escapeExpression(options.title || "") + "'" : ""; diff --git a/spec/components/pretty_text_spec.rb b/spec/components/pretty_text_spec.rb index 2235f0cb273..3ce512cb7eb 100644 --- a/spec/components/pretty_text_spec.rb +++ b/spec/components/pretty_text_spec.rb @@ -15,15 +15,15 @@ test end it "produces a quote even with new lines in it" do - PrettyText.cook("[quote=\"EvilTrout, post:123, topic:456, full:true\"]ddd\n[/quote]").should match_html "

" + PrettyText.cook("[quote=\"EvilTrout, post:123, topic:456, full:true\"]ddd\n[/quote]").should match_html "

" end it "should produce a quote" do - PrettyText.cook("[quote=\"EvilTrout, post:123, topic:456, full:true\"]ddd[/quote]").should match_html "

" + PrettyText.cook("[quote=\"EvilTrout, post:123, topic:456, full:true\"]ddd[/quote]").should match_html "

" end it "trims spaces on quote params" do - PrettyText.cook("[quote=\"EvilTrout, post:555, topic: 666\"]ddd[/quote]").should match_html "

" + PrettyText.cook("[quote=\"EvilTrout, post:555, topic: 666\"]ddd[/quote]").should match_html "

" end