Merge pull request #11 from gofullstack/actually-verify-cookbook-versions

Actually verify cookbook versions as part of the migration

Author: bcobb

lib/tasks/supermarket/import.rake

@@ -9,7 +9,8 @@ namespace :supermarket do
   cullings = [:users, :cookbooks, :cookbook_following, :cookbook_collaboration].
     map { |c| "cull:#{c}" }
 
-  task :migrate => imports + cullings + ['amend:cookbooks', 'amend:users']
+  task :migrate => imports + cullings + ['amend:cookbooks', 'amend:users',
+                                         'verify:cookbook_versions']
 
   namespace :import do
     #

lib/tasks/supermarket/verify.rake

@@ -9,10 +9,11 @@ namespace :supermarket do
         where(dependencies_imported: true).
         where('legacy_id IS NOT ?', nil)
 
-      verification_frequency = 8 # every 3 hours: 24 / 3 = 8
+      # each batch should be ~1000 cookbook versions
+      batch_divisor = base_scope.count / 1000
       batch_size = [
-        base_scope.count / verification_frequency,
-        verification_frequency
+        base_scope.count / batch_divisor,
+        batch_divisor
       ].max
 
       batch_scope = base_scope.