Software Publico / Gitlab

10 Aug, 2012

1 commit

77541891 Fully embrace Ruby 1.9 hash syntax ... Browse File »
```
Didn't bother with files in db/, config/, or features/
```
Robert Speicher
2012-08-10 18:25:15 -0400

20 Jun, 2012

7 commits

65989141 Protect IssueObserver#send_reassigned_email method. Browse File »

Robb Kidd
2012-06-20 21:23:05 -0400
97ca4f5d Deliver issue mails. ... Browse File »
```
It helps to actually deliver messages.
```
Robb Kidd
2012-06-20 14:09:46 -0400

dfb5da9d Disable observers in specs. Enable only when observer is under test. ... Browse File »

Used the built-in observer enable/disable feature in ActiveModel[1].
ActiveRecord::Base includes ActiveModel::Observing which provides this
behavior.

Simple wraps to enable the observer under test were added to the specs
for: ActivityObserver, IssueObserver, Admin::Users and Issues.

The spec for Project.last_activity was refactored to separate the tests
for #last_activity and #last_activity_date. Each had doubles added to
isolate the spec from the hidden dependency on the ActivityObserver
action to create an Event for the project when an Issue is created. This
ActivityObserver behavior is already tested by its spec.

[1] http://api.rubyonrails.org/classes/ActiveModel/ObserverArray.html

2012-06-20 14:09:46 -0400

6617eaaf Make IssueObserver handle issus, not MailerObserver Browse File »

Robb Kidd
2012-06-20 14:09:46 -0400
356430c3 Add method for an issue to know whether it is being closed ... Browse File »
```
Update IssueObserver to create a Note on the issue its being closed.
```
Robb Kidd
2012-06-20 14:09:46 -0400
00ec81ea Update IssueObserver to send reassigned emails when an issue is reassigned. Browse File »

Robb Kidd
2012-06-20 14:09:46 -0400
f6035552 New IssueObserver class and spec. ... Browse File »
```
Handles emails for new issues and reassigned issues.
Need to add creating a Note on Issue close.
```
Robb Kidd
2012-06-20 14:09:46 -0400