Skip to content

Commit

Permalink
Don't establish database connection without models
Browse files Browse the repository at this point in the history
  • Loading branch information
AlexWayfer committed Nov 19, 2020
1 parent c9eaf59 commit d29f8b8
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 2 deletions.
1 change: 1 addition & 0 deletions flame-cli.gemspec
Original file line number Diff line number Diff line change
Expand Up @@ -38,6 +38,7 @@ Gem::Specification.new do |spec|
spec.add_development_dependency 'rspec', '~> 3.9'
spec.add_development_dependency 'simplecov', '~> 0.18.0'

spec.add_development_dependency 'example_file', '~> 0.2.0'
spec.add_development_dependency 'rubocop', '~> 0.90.0'
spec.add_development_dependency 'rubocop-performance', '~> 1.0'
spec.add_development_dependency 'rubocop-rspec', '~> 1.0'
Expand Down
6 changes: 5 additions & 1 deletion spec/flame/cli/new/app_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,8 @@
require 'pathname'
require 'net/http'

require 'example_file'

describe 'Flame::CLI::New::App' do
subject(:execute_command) do
`#{FLAME_CLI} new app #{options} #{app_name}`
Expand Down Expand Up @@ -883,7 +885,9 @@
)

Dir['config/**/*.example.{yaml,conf}'].each do |config_example_file_name|
FileUtils.cp config_example_file_name, config_example_file_name.sub('.example', '')
FileUtils.cp(
config_example_file_name, config_example_file_name.sub(ExampleFile::SUFFIX, '')
)
end

## HACK for testing while some server is running
Expand Down
3 changes: 2 additions & 1 deletion template/application.rb.erb
Original file line number Diff line number Diff line change
Expand Up @@ -61,7 +61,8 @@ module <%= @module_name %>

def setup
## Initialize Sequel connection before models requiring
db_connection
## https://cirrus-ci.com/task/4975995981660160
db_connection if Dir["#{__dir__}/models/**/*"].any?

## Require dirs
require_dirs APP_DIRS, ignore: [%r{config/puma.rb}, %r{lib/\w+/spec/}]
Expand Down

0 comments on commit d29f8b8

Please sign in to comment.