FIX: Bulk badge awards should work even if the CSV has nil values

app/controllers/admin/badges_controller.rb

@@ -55,8 +55,12 @@ class Admin::BadgesController < Admin::AdminController
       csv.rewind
 
       csv.each_line do |email_line|
-        batch.concat CSV.parse_line(email_line)
-        line_number += 1
+        line = CSV.parse_line(email_line).first
+
+        if line.present?
+          batch << line
+          line_number += 1
+        end
 
         # Split the emails in batches of 200 elements.
         full_batch = csv.lineno % (BadgeGranter::MAX_ITEMS_FOR_DELTA * batch_number) == 0

spec/fixtures/csv/usernames_with_nil_values.csv

@@ -0,0 +1,4 @@
+username1,
+username2,
+username3,
+,,

spec/requests/admin/badges_controller_spec.rb

@@ -179,6 +179,8 @@ describe Admin::BadgesController do
     end
 
     describe '#mass_award' do
+      before { @user = Fabricate(:user, email: 'user1@test.com', username: 'username1') }
+
       it 'does nothing when there is no file' do
         post "/admin/badges/award/#{badge.id}.json", params: { file: '' }
 
@@ -202,23 +204,31 @@ describe Admin::BadgesController do
       it 'awards the badge using a list of user emails' do
         Jobs.run_immediately!
 
-        user = Fabricate(:user, email: 'user1@test.com')
         file = file_from_fixtures('user_emails.csv', 'csv')
 
         post "/admin/badges/award/#{badge.id}.json", params: { file: fixture_file_upload(file) }
 
-        expect(UserBadge.exists?(user: user, badge: badge)).to eq(true)
+        expect(UserBadge.exists?(user: @user, badge: badge)).to eq(true)
       end
 
       it 'awards the badge using a list of usernames' do
         Jobs.run_immediately!
 
-        user = Fabricate(:user, username: 'username1')
         file = file_from_fixtures('usernames.csv', 'csv')
 
         post "/admin/badges/award/#{badge.id}.json", params: { file: fixture_file_upload(file) }
 
-        expect(UserBadge.exists?(user: user, badge: badge)).to eq(true)
+        expect(UserBadge.exists?(user: @user, badge: badge)).to eq(true)
+      end
+
+      it 'works with a CSV containing nil values' do
+        Jobs.run_immediately!
+
+        file = file_from_fixtures('usernames_with_nil_values.csv', 'csv')
+
+        post "/admin/badges/award/#{badge.id}.json", params: { file: fixture_file_upload(file) }
+
+        expect(UserBadge.exists?(user: @user, badge: badge)).to eq(true)
       end
     end
   end