Fix functionals tests

Signed-off-by: Fabio Teixeira <fabio1079@gmail.com> Signed-off-by: Gabriela Navarro <navarro1703@gmail.com> Signed-off-by: Luciano Prestes <lucianopcbr@gmail.com>

Fix functionals tests
Signed-off-by: Fabio Teixeira <fabio1079@gmail.com> Signed-off-by: Gabriela Navarro <navarro1703@gmail.com> Signed-off-by: Luciano Prestes <lucianopcbr@gmail.com>
Fabio Teixeira · Luciano Prestes
1 parent 7b9b2e23
Showing 3 changed files with 21 additions and 16 deletions Show diff stats
lib/ext/profile_editor_controller.rb
test/functional/software_communities_plugin_myprofile_controller_test.rb
test/helpers/software_test_helper.rb
@@ -13,7 +13,6 @@ class ProfileEditorController
   protected
-
   def redirect_to_edit_software_community
     if profile.class == Community && profile.software?
       redirect_to :action => 'edit_software_community'
@@ -78,7 +78,7 @@ class SoftwareCommunitiesPluginMyprofileControllerTest &lt; ActionController::TestC
       :new_software,
       :profile => @person.identifier,
       :community => fields[1],
-      :license_info => fields[0],
+      :license => fields[0],
       :software_info => fields[2]
     )
     assert_equal SoftwareInfo.last.community.name, "Debian"
@@ -227,7 +227,7 @@ class SoftwareCommunitiesPluginMyprofileControllerTest &lt; ActionController::TestC
       :new_software,
       :community => {:name =>"New Software"},
       :software_info => {:finality => "", :repository_link => ""},
-      :license_info =>{:id => LicenseInfo.last.id},
+      :license =>{:license_infos_id => LicenseInfo.last.id},
       :profile => @person.identifier
     )
@@ -245,9 +245,10 @@ class SoftwareCommunitiesPluginMyprofileControllerTest &lt; ActionController::TestC
       :new_software,
       :community => { :name =>"New Software" },
       :software_info => { :finality => "", :repository_link => "" },
-      :license_info =>{ :id => license_another.id },
-      :license => { :version => another_license_version,
-                    :link => another_license_link },
+      :license => { :license_infos_id => license_another.id,
+        :version => another_license_version,
+        :link=> another_license_link
+      },
       :profile => @person.identifier
     )
@@ -80,9 +80,10 @@ module SoftwareTestHelper
     community.save!
     software.community = community
-    software.license_info_id = license_system_hash
+    software.license_info_id = license_system_hash[:license_infos_id]
+
+    software.save!
-    software.save
     software
   end
@@ -164,25 +165,29 @@ module SoftwareTestHelper
   end
   def software_fields
-    fields = Hash.new
-    fields_community = Hash.new
-    fields_license = Hash.new
     hash_list = []
     #Fields for license info
-    fields_license['version'] = LicenseInfo.last.version
-    fields_license['id'] = LicenseInfo.last.id
+    fields_license = {
+      license_infos_id: LicenseInfo.last.id
+    }
     hash_list << fields_license
     #Fields for community
-    fields_community['name'] = 'Debian'
-    fields_community['identifier'] = 'debian'
+    fields_community = {
+      name: 'Debian',
+      identifier: 'debian'
+    }
     hash_list << fields_community
     #Fields for basic information
-    fields['finality'] = 'This is the finality of the software'
+    fields = {
+      finality: 'This is the finality of the software'
+    }
     hash_list << fields
     hash_list
   end
 end
+#version: LicenseInfo.last.version,
+#id: LicenseInfo.last.id
 \ No newline at end of file
	@@ -13,7 +13,6 @@ class ProfileEditorController		@@ -13,7 +13,6 @@ class ProfileEditorController
13		13
14	protected	14	protected
15		15
16	-
17	def redirect_to_edit_software_community	16	def redirect_to_edit_software_community
18	if profile.class == Community && profile.software?	17	if profile.class == Community && profile.software?
19	redirect_to :action => 'edit_software_community'	18	redirect_to :action => 'edit_software_community'