migrated to rails 5.2.2

.rubocop.yml

@@ -2,6 +2,7 @@ AllCops:
   TargetRubyVersion: 2.3
   Include:
     - '**/config.ru'
+    - 'spec/**/*'
   Exclude:
     - '**/Rakefile'
     - 'db/**/*'

Gemfile

@@ -2,7 +2,7 @@ source 'https://rubygems.org'
 
 
 # Bundle edge Rails instead: gem 'rails', github: 'rails/rails'
-gem 'rails', '5.2.0'
+gem 'rails', '5.2.2'
 
 # Use SCSS for stylesheets
 gem 'sass-rails', '~> 5.0'
@@ -38,6 +38,8 @@ gem 'devise_invitable'
 gem 'omniauth'
 gem 'oj'  # fast json
 gem 'mini_magick'
+gem 'bootsnap'
+gem 'rubocop-rspec'
 
 group :development, :test do
   # Call 'byebug' anywhere in the code to stop execution and get a debugger console

bin/bundle

 #!/usr/bin/env ruby
-ENV['BUNDLE_GEMFILE'] ||= File.expand_path('../../Gemfile', __FILE__)
+ENV['BUNDLE_GEMFILE'] ||= File.expand_path('../Gemfile', __dir__)
 load Gem.bin_path('bundler', 'bundle')

bin/setup

 #!/usr/bin/env ruby
-require 'pathname'
 require 'fileutils'
 include FileUtils
 
 # path to your application root.
-APP_ROOT = Pathname.new File.expand_path('../../', __FILE__)
+APP_ROOT = File.expand_path('..', __dir__)
 
 def system!(*args)
   system(*args) || abort("\n== Command #{args} failed ==")
@@ -18,7 +17,6 @@ chdir APP_ROOT do
   system! 'gem install bundler --conservative'
   system('bundle check') || system!('bundle install')
 
-
   # puts "\n== Copying sample files =="
   # unless File.exist?('config/database.yml')
   #   cp 'config/database.yml.sample', 'config/database.yml'

bin/update

 #!/usr/bin/env ruby
-require 'pathname'
 require 'fileutils'
 include FileUtils
 
 # path to your application root.
-APP_ROOT = Pathname.new File.expand_path('../../', __FILE__)
+APP_ROOT = File.expand_path('..', __dir__)
 
 def system!(*args)
   system(*args) || abort("\n== Command #{args} failed ==")

config/boot.rb

 ENV['BUNDLE_GEMFILE'] ||= File.expand_path('../Gemfile', __dir__)
 
 require 'bundler/setup' # Set up gems listed in the Gemfile.
+require 'bootsnap/setup' # Speed up boot time by caching expensive operations.

config/environments/development.rb

@@ -13,12 +13,13 @@ Rails.application.configure do
   config.consider_all_requests_local = true
 
   # Enable/disable caching. By default caching is disabled.
-  if Rails.root.join('tmp/caching-dev.txt').exist?
+  # Run rails dev:cache to toggle caching.
+  if Rails.root.join('tmp', 'caching-dev.txt').exist?
     config.action_controller.perform_caching = true
 
     config.cache_store = :memory_store
     config.public_file_server.headers = {
-      'Cache-Control' => "public, max-age=#{2.days.seconds.to_i}"
+      'Cache-Control' => "public, max-age=#{2.days.to_i}"
     }
   else
     config.action_controller.perform_caching = false
@@ -26,16 +27,13 @@ Rails.application.configure do
     config.cache_store = :null_store
   end
 
-  # render json on error since this is an API app
-  config.debug_exception_response_format = :api
+  # Store uploaded files on the local file system (see config/storage.yml for options)
+  config.active_storage.service = :local
 
   # Don't care if the mailer can't send.
   config.action_mailer.raise_delivery_errors = false
+
   config.action_mailer.perform_caching = false
-  config.action_mailer.default_url_options = { :host => 'localhost:3000' }
-  config.action_mailer.delivery_method = :smtp
-  config.action_mailer.smtp_settings =
-    { :address => "localhost", :port => 1025 }
 
   # Print deprecation notices to the Rails logger.
   config.active_support.deprecation = :log
@@ -43,32 +41,14 @@ Rails.application.configure do
   # Raise an error on page load if there are pending migrations.
   config.active_record.migration_error = :page_load
 
-  # Debug mode disables concatenation and preprocessing of assets.
-  # This option may cause significant delays in view rendering with a large
-  # number of complex assets.
-  config.assets.debug = true
+  # Highlight code that triggered database queries in logs.
+  config.active_record.verbose_query_logs = true
 
-  # Suppress logger output for asset requests.
-  config.assets.quiet = true
 
   # Raises error for missing translations
   # config.action_view.raise_on_missing_translations = true
 
   # Use an evented file watcher to asynchronously detect changes in source code,
   # routes, locales, etc. This feature depends on the listen gem.
-  # config.file_watcher = ActiveSupport::EventedFileUpdateChecker
-  config.middleware.insert_before 0, Rack::Cors do
-    allow do
-      origins '*'
-      resource '*',
-        :headers => :any,
-        :expose => ['access-token', 'expiry', 'token-type', 'uid', 'client'],
-        :methods =>  [:get, :post, :options, :delete, :put]
-    end
-  end
-  config.interface_url_template = lambda do |id|
-    #return "http://#{id}.interfaces.wattsworth.local"
-    return "http://wattsworth.local/api/interfaces/#{id}/"
-  end
-
+  config.file_watcher = ActiveSupport::EventedFileUpdateChecker
 end

config/environments/production.rb

@@ -4,6 +4,10 @@ Rails.application.configure do
   # Code is not reloaded between requests.
   config.cache_classes = true
 
+  # Ensures that a master key has been made available in either ENV["RAILS_MASTER_KEY"]
+  # or in config/master.key. This key is used to decrypt credentials (and other encrypted files).
+  # config.require_master_key = true
+
   # Eager load code on boot. This eager loads most of Rails and
   # your application in memory, allowing both threaded web servers
   # and those relying on copy on write to perform better.

config/environments/test.rb

@@ -15,7 +15,7 @@ Rails.application.configure do
   # Configure public file server for tests with Cache-Control for performance.
   config.public_file_server.enabled = true
   config.public_file_server.headers = {
-    'Cache-Control' => "public, max-age=#{1.hour.seconds.to_i}"
+    'Cache-Control' => "public, max-age=#{1.hour.to_i}"
   }
 
   # Show full error reports and disable caching.

config/initializers/application_controller_renderer.rb

 # Be sure to restart your server when you modify this file.
 
-# ApplicationController.renderer.defaults.merge!(
-#   http_host: 'example.org',
-#   https: false
-# )
+# ActiveSupport::Reloader.to_prepare do
+#   ApplicationController.renderer.defaults.merge!(
+#     http_host: 'example.org',
+#     https: false
+#   )
+# end

config/initializers/new_framework_defaults_5_2.rb

+# Be sure to restart your server when you modify this file.
+#
+# This file contains migration options to ease your Rails 5.2 upgrade.
+#
+# Once upgraded flip defaults one by one to migrate to the new default.
+#
+# Read the Guide for Upgrading Ruby on Rails for more info on each option.
+
+# Make Active Record use stable #cache_key alongside new #cache_version method.
+# This is needed for recyclable cache keys.
+Rails.application.config.active_record.cache_versioning = true
+
+# Use AES-256-GCM authenticated encryption for encrypted cookies.
+# Also, embed cookie expiry in signed or encrypted cookies for increased security.
+#
+# This option is not backwards compatible with earlier Rails versions.
+# It's best enabled when your entire app is migrated and stable on 5.2.
+#
+# Existing cookies will be converted on read then written with the new scheme.
+Rails.application.config.action_dispatch.use_authenticated_cookie_encryption = true
+
+# Use AES-256-GCM authenticated encryption as default cipher for encrypting messages
+# instead of AES-256-CBC, when use_authenticated_message_encryption is set to true.
+Rails.application.config.active_support.use_authenticated_message_encryption = true
+
+# Add default protection from forgery to ActionController::Base instead of in
+# ApplicationController.
+Rails.application.config.action_controller.default_protect_from_forgery = true
+
+# Store boolean values are in sqlite3 databases as 1 and 0 instead of 't' and
+# 'f' after migrating old data.
+Rails.application.config.active_record.sqlite3.represent_boolean_as_integer = true
+
+# Use SHA-1 instead of MD5 to generate non-sensitive digests, such as the ETag header.
+Rails.application.config.active_support.use_sha1_digests = true
+
+# Make `form_with` generate id attributes for any generated HTML tags.
+Rails.application.config.action_view.form_with_generates_ids = true

config/storage.yml

+test:
+  service: Disk
+  root: <%= Rails.root.join("tmp/storage") %>
+
+local:
+  service: Disk
+  root: <%= Rails.root.join("storage") %>
+
+# Use rails credentials:edit to set the AWS secrets (as aws:access_key_id|secret_access_key)
+# amazon:
+#   service: S3
+#   access_key_id: <%= Rails.application.credentials.dig(:aws, :access_key_id) %>
+#   secret_access_key: <%= Rails.application.credentials.dig(:aws, :secret_access_key) %>
+#   region: us-east-1
+#   bucket: your_own_bucket
+
+# Remember not to checkin your GCS keyfile to a repository
+# google:
+#   service: GCS
+#   project: your_project
+#   credentials: <%= Rails.root.join("path/to/gcs.keyfile") %>
+#   bucket: your_own_bucket
+
+# Use rails credentials:edit to set the Azure Storage secret (as azure_storage:storage_access_key)
+# microsoft:
+#   service: AzureStorage
+#   storage_account_name: your_account_name
+#   storage_access_key: <%= Rails.application.credentials.dig(:azure_storage, :storage_access_key) %>
+#   container: your_container_name
+
+# mirror:
+#   service: Mirror
+#   primary: local
+#   mirrors: [ amazon, google, microsoft ]

spec/cassettes/Joule_Backend/retrieves_module_schema.yml

@@ -2,7 +2,7 @@
 http_interactions:
 - request:
     method: get
-    uri: http://nuc:8088/modules.json
+    uri: http://nuc:8088/modules.json?statistics=1
     body:
       encoding: US-ASCII
       string: ''

spec/factories/data_views.rb

@@ -2,7 +2,7 @@ FactoryBot.define do
   factory :data_view do
     name { Faker::Lorem.words(3).join(' ') }
     description { Faker::Lorem.sentence }
-    visibility "public"
-    redux_json "auto generated from factory"
+    visibility { "public" }
+    redux_json { "auto generated from factory" }
   end
 end

spec/factories/db_element.rb

@@ -4,13 +4,13 @@
 FactoryBot.define do
   factory :db_element do
     name { Faker::Lorem.unique.words(3).join(' ') }
-    units 'volts'
+    units { 'volts' }
     sequence(:column)
-    default_max 100
-    default_min 0
-    scale_factor 1.0
-    offset 0.0
-    plottable true
-    display_type 'continuous'
+    default_max { 100 }
+    default_min { 0 }
+    scale_factor { 1.0 }
+    offset { 0.0 }
+    plottable { true }
+    display_type { 'continuous' }
   end
 end

spec/factories/db_folder.rb

@@ -8,7 +8,7 @@ FactoryBot.define do
     start_time { Faker::Number.number(6)}
     end_time { start_time + Faker::Number.number(5) }
     size_on_disk { Faker::Number.number(6) }
-    hidden false
+    hidden { false }
     path { "/root/#{Faker::Lorem.word}/#{Faker::Number.unique.number(4)}" }
   end
 end

spec/factories/db_stream.rb

@@ -6,18 +6,18 @@ FactoryBot.define do
     name { Faker::Lorem.words(3).join(' ') }
     name_abbrev { Faker::Lorem.word }
     description { Faker::Lorem.sentence }
-    delete_locked false
+    delete_locked { false }
     start_time { Faker::Number.number(6).to_i}
     end_time { start_time + Faker::Number.number(5).to_i }
     total_time {end_time - start_time}
     size_on_disk { Faker::Number.number(6).to_i }
-    hidden false
+    hidden { false }
     path { "/root/streams/#{Faker::Lorem.unique.word}" }
     data_type { "float32_#{elements_count}" }
 
     transient do
-      elements_count 4
-      decimations_count 0
+      elements_count { 4 }
+      decimations_count { 0 }
     end
 
     after(:create) do |stream, evaluator|

spec/factories/dbs.rb

@@ -2,12 +2,12 @@
 
 FactoryBot.define do
   factory :db do
-    url ""
+    url { "" }
     max_points_per_plot { Faker::Number.number(3) }
     size_db { Faker::Number.number(5) }
     size_other { Faker::Number.number(5) }
     size_total { size_db + size_other }
-    available true
+    available { true }
     root_folder
   end
 end

spec/factories/joule_modules.rb

@@ -2,9 +2,9 @@ FactoryBot.define do
   factory :joule_module do
     name { Faker::Lorem.words(3).join(' ') }
     description { Faker::Lorem.sentence }
-    exec_cmd '/path/to/cmd'
-    web_interface false
-    status 'running'
-    joule_id Faker::Number.number(3).to_i
+    exec_cmd { '/path/to/cmd' }
+    web_interface { false }
+    status { 'running' }
+    joule_id { Faker::Number.number(3).to_i }
   end
 end

spec/factories/nilms.rb

@@ -8,11 +8,11 @@ FactoryBot.define do
     name {Faker::Lorem.unique.words(3).join(' ')}
     description { Faker::Lorem.sentence }
     url {Faker::Internet.unique.url}
-    node_type 'nilmdb'
+    node_type { 'nilmdb' }
     transient do
-      admins []
-      owners []
-      viewers []
+      admins { [] }
+      owners { [] }
+      viewers { [] }
     end
 
     after(:create) do |nilm, evaluator|

spec/factories/permissions.rb

 FactoryBot.define do
   factory :permission do
     nilm
-    role "admin"
+    role { "admin" }
     after(:build) do |permission|
       # if no user or group specified, create a user
       if(permission.user.blank? &&

spec/factories/user_groups.rb

 FactoryBot.define do
   factory :user_group do
     transient do
-      members []
+      members { [] }
     end
 
     sequence :name do |n| "group#{n}" end
@@ -11,7 +11,7 @@ FactoryBot.define do
 
     factory :test_user_group do
       transient do
-        size []
+        size { [] }
       end
       name { Faker::Company.unique.name }
       description { Faker::ChuckNorris.fact }