@@ -130,7 +130,7 class Project < ActiveRecord::Base | |||
|
130 | 130 | end |
|
131 | 131 | |
|
132 | 132 | def identifier_frozen? |
|
133 |
errors[:identifier]. |
|
|
133 | errors[:identifier].blank? && !(new_record? || identifier.blank?) | |
|
134 | 134 | end |
|
135 | 135 | |
|
136 | 136 | # returns latest created projects |
@@ -106,6 +106,20 class ProjectTest < ActiveSupport::TestCase | |||
|
106 | 106 | end |
|
107 | 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 | 123 | def test_members_should_be_active_users |
|
110 | 124 | Project.all.each do |project| |
|
111 | 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