Removed duplicated methods

Author: tejaszambre

app/controllers/light/users_controller.rb

@@ -37,7 +37,7 @@ def create
     def unsubscribe
       if @user.present? && @user.sidekiq_status == 'Subscribed'
         @user.update(
-          is_subscribed: 'false',
+          is_subscribed: false,
           unsubscribed_at: DateTime.now,
           sidekiq_status: 'Unsubscribed'
         )
@@ -50,7 +50,7 @@ def unsubscribe
     def subscribe
       if @user.present? && @user.sidekiq_status == 'Unsubscribed'
         @user.update(
-          is_subscribed: 'true',
+          is_subscribed: true,
           sidekiq_status: 'Subscribed',
           subscribed_at: DateTime.now,
           remote_ip: request.remote_ip,

app/models/light/user.rb

@@ -83,12 +83,7 @@ def self.import(file, email='')
       {success: 'You will get an update email.'}
     end
 
-    def self.users_for_opt_in_mail
-      date = Date.today.strftime('%Y%m')
-      self.new_users.where(:sent_on.nin => [date], is_blocked: false).order_by([:email_id, :asc])
-    end
-
-    def self.users_for_opt_out_mail
+    def self.get_new_users
       date = Date.today.strftime('%Y%m')
       self.new_users.where(:sent_on.nin => [date], is_blocked: false).order_by([:email_id, :asc])
     end

app/workers/light/opt_in_worker.rb

@@ -5,7 +5,7 @@ class OptInWorker
 
     def perform(newsletter_id)
       date = Date.today.strftime("%Y%m")
-      users = Light::User.users_for_opt_in_mail
+      users = Light::User.get_new_users
       number_of_opt_in_users = users.count
       number_of_opt_in_users_count = number_of_opt_in_users
       current_batch = 0

app/workers/light/opt_out_worker.rb

@@ -5,7 +5,7 @@ class OptOutWorker
 
     def perform(newsletter_id)
       date = Date.today.strftime("%Y%m")
-      users = Light::User.users_for_opt_out_mail
+      users = Light::User.get_new_users
       number_of_opt_out_users = users.count
       number_of_opt_out_users_count = number_of_opt_out_users
       current_batch = 0

app/workers/light/user_worker.rb

@@ -6,7 +6,7 @@ class UserWorker
     def perform(newsletter_id)
       date = Date.today.strftime("%Y%m")
       number_of_subscribed_users = Light::User.where(is_subscribed: true, :sent_on.nin => [date],  is_blocked: {"$ne" => true}).count
-      #number_of_subscribed_users = Light::User.users_for_opt_in_mail.count
+      #number_of_subscribed_users = Light::User.get_new_users.count
       number_of_subscribed_users_count = number_of_subscribed_users
       current_batch = 0
       users_in_batch = 250
@@ -17,7 +17,7 @@ def perform(newsletter_id)
       if newsletter
         while number_of_subscribed_users > 0
           user_ids = Light::User.where(is_subscribed: true, :sent_on.nin => [date] , is_blocked: {"$ne" => true}).order_by([:email_id, :asc]).limit(users_in_batch).skip(users_in_batch*current_batch).collect { |user| user.id.to_s }
-          #user_ids  = Light::User.users_for_opt_in_mail.order_by([:email_id, :asc]).limit(users_in_batch).skip(users_in_batch*current_batch).collect { |user| user.id.to_s }
+          #user_ids  = Light::User.get_new_users.order_by([:email_id, :asc]).limit(users_in_batch).skip(users_in_batch*current_batch).collect { |user| user.id.to_s }
           current_batch += 1
           number_of_subscribed_users -= users_in_batch
           Light::HardWorker.perform_async(user_ids, newsletter.id.to_s, date)

spec/controllers/light/users_controller_spec.rb

@@ -51,7 +51,7 @@ module Light
       end
 
       it 'not arise' do
-        post :create, {user: {email_id: '',username: 'kanhaiya', is_subscribed: 'false'}}
+        post :create, {user: {email_id: '',username: 'kanhaiya', is_subscribed: false}}
         expect(flash[:error]).to eq('Error while creating user')
         expect(response).to render_template('new')
       end