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

Tom Hughes notifications at github.com
Wed Sep 7 20:16:12 UTC 2016


> @@ -1,10 +1,8 @@
>  require "test_helper"
>  
>  class DiaryCommentTest < ActiveSupport::TestCase
> -  api_fixtures
> -  fixtures :diary_comments
> -
>    def test_diary_comment_count

This test name doesn't really reflect what the test does any more... Then again I'm not really sure what it is testing now - is it actually just testing that FactoryGirl can create records? or does that itself funnel through the rails code so that we're testing rails can create records?

-- 
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#r77895286
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20160907/144c988b/attachment.html>


More information about the rails-dev mailing list