#578 More extensive database seeding - bulk creating members, gardens and plantings

[oshiho3]

Made some improvement on the database seeding process, covering some items from issue #578. Just did this as I wanted to test performance on geo related processing.

This will let you...

• Create N number of members and
• Populate member's location
• Populate garden's location
• Creates a planting from various crops, sunniness and planted_from for each user.

The default number of members is still 3, but you can specify the number of members as...
rake db:seed member_size=100
or
rake db:reset member_size=100
Tested up to 1000 worked fine.

[coveralls]

Coverage remained the same at 83.65% when pulling 59d823f on oshiho3:578-database-seeding-improvement into 6505254 on Growstuff:dev.

[cesy]

What's the rationale for this change?

[oshiho3]

Thanks for looking at my code. Currently when a test user is loaded, content of the confirmation email for each user opens up in a browser. However, skip_confirmation automatically confirm the user and avoid generating thousands of confirmation emails/pages. My rationale is most of the time we ignore the confirmation pages, and if someone wants to test the confirmation email content specifically, it can be tested manually.

[maco]

I find those browser windows annoying; therefore, I love you right now.

[oshiho3]

Thanks for your endorsement :-) yes, it was annoying me too.

[Skud]

This is great, thanks! I suspect .skip_confirmation is new since we
started using Devise but it is a very nice thing to be able to use, and
avoid those popup windows.

On 26/03/2015 8:08 am, Shiho Takagi wrote:

In db/seeds.rb
#731 (comment):

     :login_name => "test#{i}",
     :email => "test#{i}@example.com",
     :password => "password#{i}",
     :tos_agreement => true
 )

• @user.confirm!
• @user.skip_confirmation!

Thanks for looking at my code. Currently when a test user is loaded,
content of the confirmation email for each user opens up in a browser.
However, |skip_confirmation| automatically confirm the user and avoid
generating thousands of confirmation emails/pages. My rationale is
most of the time we ignore the confirmation pages, and if someone
wants to test the confirmation email content specifically, it can be
tested manually.

—
Reply to this email directly or view it on GitHub
https://github.com/Growstuff/growstuff/pull/731/files#r27166170.

Alex "Skud" Bayley
skud@growstuff.org
http://growstuff.org/

[coveralls]

Coverage remained the same at 83.69% when pulling 1fe9a7d on oshiho3:578-database-seeding-improvement into d7ef598 on Growstuff:dev.

[coveralls]

Coverage remained the same at 83.69% when pulling 7e7b273 on oshiho3:578-database-seeding-improvement into d7ef598 on Growstuff:dev.

[coveralls]

Coverage remained the same at 83.69% when pulling 7e7b273 on oshiho3:578-database-seeding-improvement into d7ef598 on Growstuff:dev.

[oshiho3]

Added sunniness and planted_from field data population on plantings.
Also made all the user confirmation email skipped as some said they also don't want to see the page popping up.

[coveralls]

Coverage remained the same at 83.69% when pulling d95bd0e on oshiho3:578-database-seeding-improvement into d7ef598 on Growstuff:dev.

[oshiho3]

Added sunniness and planted_from field on planting.
Also got rid of all the user confirmation popup since some said they like the idea.

[Marlena]

This will help move work forward on the pull request for sunniness and planting charts. Hope we can merge this one in soon.

[maco]

As soon as release 8 is pushed live, I have about 10 things to merge right away.

[Marlena]

That's cool. I can see that there is a large queue. Working on pulling in the branch and just using that for now.

[Skud]

This looks great and I'm going to merge it :) I would support the idea of automatically creating 20 or so test users, as a default case, fwiw.

[oshiho3]

Welcome back, @Skud! Yup, increasing the number of default users sounds good. I think I'll implement it in the next iteration along with other improvements in database seeding.