Fix for orendon#22 and orendon#26

akonan · Dec 3, 2012 · 50ce079 · 50ce079
1 parent 5b08d37
commit 50ce079
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 1 deletion.
diff --git a/app/models/repo.rb b/app/models/repo.rb
@@ -55,7 +55,7 @@ def create_or_update(user, params)
 
     def get_languages
       languages_list = select(:language).uniq
-      languages_list.collect(&:language)
+      languages_list.collect(&:language).reject! { |l| l.nil? || l.empty? }
     end
   end
 

diff --git a/spec/factories/repos.rb b/spec/factories/repos.rb
@@ -6,5 +6,6 @@
     name { Faker::Name.first_name }
     need_help true
     user_description { Faker::Lorem.paragraph }
+    language { Faker::Lorem.word }
   end
 end
diff --git a/spec/models/repo_spec.rb b/spec/models/repo_spec.rb
@@ -29,5 +29,13 @@
     end
 
   end
+
+  describe "get_languages" do
+    @repo1 = FactoryGirl.create(:repo, language: "Ruby")
+    @repo2 = FactoryGirl.create(:repo, language: "")
+    @repo3 = FactoryGirl.create(:repo, language: nil)
+
+    Repo.get_languages.should == ["Ruby"]
+  end
 
 end