Merge pull request #531 from okriuchykhin/ok_SCI_1127

Remove logs table and old unused logging code [SCI-1127]

app/controllers/application_controller.rb

@@ -32,18 +32,6 @@ class ApplicationController < ActionController::Base
 
   protected
 
-  def log(message)
-    if @my_module
-      @my_module.log(message)
-    elsif @project
-      @project.log(message)
-    elsif @team
-      @team.log(message)
-    else
-      logger.error(message)
-    end
-  end
-
   def render_403(style = 'danger')
     respond_to do |format|
       format.html do

app/models/log.rb

@@ -1,6 +0,0 @@
-class Log < ActiveRecord::Base
-  validates :message, presence: true
-  validates :team, presence: true
-
-  belongs_to :team, inverse_of: :logs
-end

app/models/my_module.rb

@@ -348,12 +348,6 @@ class MyModule < ActiveRecord::Base
     return clone
   end
 
-  # Writes to user log.
-  def log(message)
-    final = "[%s] %s" % [name, message]
-    experiment.project.log(final)
-  end
-
   # Find an empty position for the restored module. It's
   # basically a first empty row with empty space inside x=[0, 32).
   def get_new_position

app/models/project.rb

@@ -160,12 +160,6 @@ class Project < ActiveRecord::Base
     st
   end
 
-  # Writes to user log.
-  def log(message)
-    final = "[%s] %s" % [name, message]
-    team.log(final)
-  end
-
   def assigned_samples
     Sample.joins(:my_modules).where(my_modules: {
                                       id: my_modules_ids.split(',')

app/models/team.rb

@@ -20,7 +20,6 @@ class Team < ActiveRecord::Base
   has_many :samples_tables, inverse_of: :team, dependent: :destroy
   has_many :sample_groups, inverse_of: :team
   has_many :sample_types, inverse_of: :team
-  has_many :logs, inverse_of: :team
   has_many :projects, inverse_of: :team
   has_many :custom_fields, inverse_of: :team
   has_many :protocols, inverse_of: :team, dependent: :destroy
@@ -71,12 +70,6 @@ class Team < ActiveRecord::Base
       .where_attributes_like(attributes, a_query)
   end
 
-  # Writes to user log
-  def log(message)
-    final = "[%s] %s" % [Time.current.to_s, message]
-    logs.create(message: final)
-  end
-
   # Imports samples into db
   # -1 == sample_name,
   # -2 == sample_type,

db/migrate/20170330092353_remove_logs_table.rb

@@ -0,0 +1,5 @@
+class RemoveLogsTable < ActiveRecord::Migration
+  def change
+    drop_table :logs
+  end
+end

test/models/log_test.rb

@@ -1,27 +0,0 @@
-require 'test_helper'
-
-class LogTest < ActiveSupport::TestCase
-  def setup
-    @log =  logs(:one)
-  end
-
-  test 'should validate log with valid data' do
-    assert @log.valid?
-  end
-
-  test 'should have non-blank message' do
-    @log.message = ''
-    assert @log.invalid?, 'Log with blank message returns valid? = true'
-    @log.message = nil
-    assert @log.invalid?, 'Log with nil message returns valid? = true'
-  end
-
-  test 'should have team' do
-    @log.team_id = 12321321
-    assert @log.invalid?, 'Log without team returns valid? = true'
-    @log.team = nil
-    assert @log.invalid?, 'Log without team returns valid? = true'
-    @log.team = Team.new
-    assert @log.valid?, 'Log with team returns valid? = false'
-  end
-end

test/models/my_module_test.rb

@@ -141,11 +141,4 @@ class MyModuleTest < ActiveSupport::TestCase
   test "should deep clone module" do
     skip
   end
-
-  test "should save log message" do
-    message = "This is test message for my module"
-    @my_module.log(message)
-    log_message = Log.last.message
-    assert_equal log_message[57..-1], message
-  end
 end

test/models/team_test.rb

@@ -32,13 +32,6 @@ class TeamTest < ActiveSupport::TestCase
     assert_equal Constants::MINIMAL_TEAM_SPACE_TAKEN, team.space_taken
   end
 
-  test 'should save log message' do
-    message = 'This is test message'
-    @team.log(message)
-    log_message = Log.last.message[26..-1]
-    assert_equal log_message, message
-  end
-
   test 'should open spreadsheet file' do
     skip
   end