@@ -130,7 +130,7 class Project < ActiveRecord::Base | |||||
130 | end |
|
130 | end | |
131 |
|
131 | |||
132 | def identifier_frozen? |
|
132 | def identifier_frozen? | |
133 |
errors[:identifier]. |
|
133 | errors[:identifier].blank? && !(new_record? || identifier.blank?) | |
134 | end |
|
134 | end | |
135 |
|
135 | |||
136 | # returns latest created projects |
|
136 | # returns latest created projects |
@@ -106,6 +106,20 class ProjectTest < ActiveSupport::TestCase | |||||
106 | end |
|
106 | end | |
107 | end |
|
107 | end | |
108 |
|
108 | |||
|
109 | def test_identifier_should_not_be_frozen_for_a_new_project | |||
|
110 | assert_equal false, Project.new.identifier_frozen? | |||
|
111 | end | |||
|
112 | ||||
|
113 | def test_identifier_should_not_be_frozen_for_a_saved_project_with_blank_identifier | |||
|
114 | Project.update_all(["identifier = ''"], "id = 1") | |||
|
115 | ||||
|
116 | assert_equal false, Project.find(1).identifier_frozen? | |||
|
117 | end | |||
|
118 | ||||
|
119 | def test_identifier_should_be_frozen_for_a_saved_project_with_valid_identifier | |||
|
120 | assert_equal true, Project.find(1).identifier_frozen? | |||
|
121 | end | |||
|
122 | ||||
109 | def test_members_should_be_active_users |
|
123 | def test_members_should_be_active_users | |
110 | Project.all.each do |project| |
|
124 | Project.all.each do |project| | |
111 | assert_nil project.members.detect {|m| !(m.user.is_a?(User) && m.user.active?) } |
|
125 | assert_nil project.members.detect {|m| !(m.user.is_a?(User) && m.user.active?) } |
General Comments 0
You need to be logged in to leave comments.
Login now