| @@ -94,9 +94,9 class Member < ActiveRecord::Base | |||||
| 94 | end |
|
94 | end | |
| 95 |
|
95 | |||
| 96 | def set_issue_category_nil |
|
96 | def set_issue_category_nil | |
| 97 | if user |
|
97 | if user_id && project_id | |
| 98 | # remove category based auto assignments for this member |
|
98 | # remove category based auto assignments for this member | |
| 99 |
IssueCategory.where(["project_id = ? AND assigned_to_id = ?", project |
|
99 | IssueCategory.where(["project_id = ? AND assigned_to_id = ?", project_id, user_id]). | |
| 100 | update_all("assigned_to_id = NULL") |
|
100 | update_all("assigned_to_id = NULL") | |
| 101 | end |
|
101 | end | |
| 102 | end |
|
102 | end | |
| @@ -94,6 +94,16 class MemberTest < ActiveSupport::TestCase | |||||
| 94 | assert !member.save |
|
94 | assert !member.save | |
| 95 | end |
|
95 | end | |
| 96 |
|
96 | |||
|
|
97 | def test_set_issue_category_nil_should_handle_nil_values | |||
|
|
98 | m = Member.new | |||
|
|
99 | assert_nil m.user | |||
|
|
100 | assert_nil m.project | |||
|
|
101 | ||||
|
|
102 | assert_nothing_raised do | |||
|
|
103 | m.set_issue_category_nil | |||
|
|
104 | end | |||
|
|
105 | end | |||
|
|
106 | ||||
| 97 | def test_destroy |
|
107 | def test_destroy | |
| 98 | category1 = IssueCategory.find(1) |
|
108 | category1 = IssueCategory.find(1) | |
| 99 | assert_equal @jsmith.user.id, category1.assigned_to_id |
|
109 | assert_equal @jsmith.user.id, category1.assigned_to_id | |
General Comments 0
You need to be logged in to leave comments.
Login now