From 8b432927283aafca165f5b7f38c7c4eff952eefb Mon Sep 17 00:00:00 2001
From: Luciano Prestes Cavalcanti <lucianopcbr@gmail.com>
Date: Wed, 29 Oct 2014 17:33:11 -0200
Subject: [PATCH] Tests for software language helper

---
 lib/software_language_helper.rb            | 14 ++++++++++++--
 test/unit/software_language_helper_test.rb | 76 ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++
 2 files changed, 88 insertions(+), 2 deletions(-)
 create mode 100644 test/unit/software_language_helper_test.rb

diff --git a/lib/software_language_helper.rb b/lib/software_language_helper.rb
index 55e7904..3751c2a 100644
--- a/lib/software_language_helper.rb
+++ b/lib/software_language_helper.rb
@@ -1,11 +1,21 @@
 module SoftwareLanguageHelper
 
+  def self.valid_language? language
+    return false if SoftwareHelper.all_table_is_empty?(language)
+
+    programming_language_id_list = ProgrammingLanguage.select(:id).collect {|dd| dd.id }
+
+    return false unless programming_language_id_list.include?(language[:programming_language_id].to_i)
+
+    true
+  end
+
   def self.list_language new_languages
     return [] if new_languages.nil? or new_languages.length == 0
     list_languages = []
 
     new_languages.each do |new_language|
-      unless SoftwareHelper.all_table_is_empty? new_language, ["programming_language_id"]
+      if valid_language? new_language
         language = SoftwareLanguage.new
         language.programming_language = ProgrammingLanguage.find(new_language[:programming_language_id])
         language.version = new_language[:version]
@@ -17,7 +27,7 @@ module SoftwareLanguageHelper
     list_languages
   end
 
-    def self.valid_list_language? list_languages
+  def self.valid_list_language? list_languages
     return false if list_languages.nil? or list_languages.length == 0
 
     list_languages.each do |language|
diff --git a/test/unit/software_language_helper_test.rb b/test/unit/software_language_helper_test.rb
new file mode 100644
index 0000000..b828f5e
--- /dev/null
+++ b/test/unit/software_language_helper_test.rb
@@ -0,0 +1,76 @@
+require File.dirname(__FILE__) + '/../../../../test/test_helper'
+
+class SoftwareLanguageHelperTest < ActiveSupport::TestCase
+
+  include SoftwareLanguageHelper
+
+  def setup
+    pl1 = ProgrammingLanguage.create(:name => "Python")
+    pl2 = ProgrammingLanguage.create(:name => "Java")
+
+    @software_language_objects = [
+      {:programming_language_id => pl1.id.to_s ,:version => "2.0", :operating_system => "debian"},
+      {:programming_language_id => pl2.id.to_s ,:version => "2.1", :operating_system => "debian"},
+      {:programming_language_id => pl1.id.to_s ,:version => "2.2", :operating_system => "debian"}]
+    @software_language_objects
+  end
+
+  def teardown
+    @software_language_objects = nil
+    ProgrammingLanguage.destroy_all
+  end
+
+  should "return an empty list" do
+    empty_list = []
+    assert_equal [], SoftwareLanguageHelper.list_language(empty_list)
+  end
+
+  should "return a list with current software language objects" do
+    list_compare = []
+    assert_equal list_compare.class, SoftwareLanguageHelper.list_language(@software_language_objects).class
+  end
+
+  should "have same information from the list passed as parameter" do
+    list_compare = SoftwareLanguageHelper.list_language(@software_language_objects)
+    assert_equal @software_language_objects.first[:programming_language_id].to_i, list_compare.first.programming_language_id
+  end
+
+  should "return a list with the same size of the parameter" do
+    list_compare = SoftwareLanguageHelper.list_language(@software_language_objects)
+    assert_equal @software_language_objects.count, list_compare.count
+  end
+
+  should "return false if list_language are empty or null" do
+    list_compare = []
+    assert_equal false,SoftwareLanguageHelper.valid_list_language?(list_compare)
+  end
+
+  should "return a html text with operating system equals to linux" do
+    softwares_languages = []
+
+    programming_language = ProgrammingLanguage.new
+    programming_language.name = "teste"
+
+    software_language = SoftwareLanguage.new
+    software_language.version = 2
+    software_language.operating_system = "linux"
+    software_language.programming_language = programming_language
+
+    softwares_languages << software_language
+
+    assert_not_nil SoftwareLanguageHelper.language_as_tables(softwares_languages).first.call.index("linux")
+  end
+
+  should "remove invalid tables from the list" do
+    @software_language_objects.push({
+      :programming_language_id => "I'm not a valid id",
+      :version => "2.0",
+      :operating_system => "debian"
+    })
+
+    software_language_objects_length = @software_language_objects.count
+    list_compare = SoftwareLanguageHelper.list_language(@software_language_objects)
+
+    assert_equal software_language_objects_length-1, list_compare.count
+  end
+end
--
libgit2 0.21.2