[openstreetmap/openstreetmap-website] Add Factorygirl as an alternative to fixtures (#1279)

Tom Hughes notifications at github.com
Wed Sep 7 20:25:39 UTC 2016


>    end
>  
>    private
>  
>    def diary_entry_valid(attrs, result = true)
> -    entry = DiaryEntry.new(diary_entries(:normal_user_entry_1).attributes)
> +    entry = DiaryEntry.new(attributes_for(:diary_entry))

Indeed can we not merge it with the next line as:

```
entry = build(:diary_entry, attrs)
```

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/pull/1279/files/d5013ad0d2640693af534db47f891ca4fbbf5a4c..3b3f328ae4427ba0d1e3700ceabf7d0870f78dbd#r77896748
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20160907/a8a4b592/attachment.html>


More information about the rails-dev mailing list