Added integration tests.

Author: dblock

.travis.yml

@@ -8,3 +8,13 @@ cache: bundler
 services:
   - mongodb
 
+addons:
+  firefox: 54.0
+
+before_install:
+  - export DISPLAY=:99.0
+  - sh -e /etc/init.d/xvfb start
+  - wget https://github.com/mozilla/geckodriver/releases/download/v0.18.0/geckodriver-v0.18.0-linux64.tar.gz
+  - mkdir geckodriver
+  - tar -xzf geckodriver-v0.18.0-linux64.tar.gz -C geckodriver
+  - export PATH=$PATH:$PWD/geckodriver

Gemfile

@@ -4,16 +4,18 @@ ruby '2.3.1'
 
 gem 'mongoid'
 gem 'mongoid-scroll'
-gem 'slack-ruby-bot-server'
 gem 'newrelic-slack-ruby-bot'
+gem 'slack-ruby-bot-server'
 
 group :test do
-  gem 'rake'
-  gem 'rspec'
+  gem 'capybara'
+  gem 'database_cleaner'
   gem 'fabrication'
   gem 'faker'
+  gem 'rack-test'
+  gem 'rake'
+  gem 'rspec'
+  gem 'selenium-webdriver'
   gem 'vcr'
   gem 'webmock'
-  gem 'database_cleaner'
-  gem 'rack-test'
 end

Gemfile.lock

@@ -16,6 +16,13 @@ GEM
       thread_safe (~> 0.3, >= 0.3.1)
     bson (4.2.2)
     builder (3.2.3)
+    capybara (2.15.1)
+      addressable
+      mini_mime (>= 0.1.3)
+      nokogiri (>= 1.3.3)
+      rack (>= 1.0.0)
+      rack-test (>= 0.5.4)
+      xpath (~> 2.0)
     celluloid (0.17.3)
       celluloid-essentials
       celluloid-extras
@@ -37,6 +44,8 @@ GEM
       timers (>= 4.1.1)
     celluloid-supervision (0.20.6)
       timers (>= 4.1.1)
+    childprocess (0.7.1)
+      ffi (~> 1.0, >= 1.0.11)
     coercible (1.0.0)
       descendants_tracker (~> 0.0.1)
     concurrent-ruby (1.0.5)
@@ -56,6 +65,7 @@ GEM
       multipart-post (>= 1.2, < 3)
     faraday_middleware (0.12.2)
       faraday (>= 0.7.4, < 1.0)
+    ffi (1.9.18)
     foreman (0.84.0)
       thor (~> 0.19.1)
     gli (2.16.1)
@@ -83,6 +93,8 @@ GEM
       grape
       kaminari-core (~> 1.0)
     kgio (2.11.0)
+    mini_mime (0.1.3)
+    mini_portile2 (2.2.0)
     minitest (5.10.3)
     mongo (2.4.3)
       bson (>= 4.2.1, < 5.0.0)
@@ -106,6 +118,8 @@ GEM
       slack-ruby-bot
     newrelic_rpm (4.3.0.335)
     nio4r (2.1.0)
+    nokogiri (1.8.0)
+      mini_portile2 (~> 2.2.0)
     public_suffix (2.0.5)
     rack (2.0.3)
     rack-accept (0.4.5)
@@ -137,7 +151,11 @@ GEM
       diff-lcs (>= 1.2.0, < 2.0)
       rspec-support (~> 3.6.0)
     rspec-support (3.6.0)
+    rubyzip (1.2.1)
     safe_yaml (1.0.4)
+    selenium-webdriver (3.4.4)
+      childprocess (~> 0.5)
+      rubyzip (~> 1.0)
     slack-ruby-bot (0.10.4)
       hashie
       slack-ruby-client (>= 0.6.0)
@@ -184,11 +202,14 @@ GEM
     websocket-driver (0.6.5)
       websocket-extensions (>= 0.1.0)
     websocket-extensions (0.1.2)
+    xpath (2.1.0)
+      nokogiri (~> 1.3)
 
 PLATFORMS
   ruby
 
 DEPENDENCIES
+  capybara
   database_cleaner
   fabrication
   faker
@@ -198,6 +219,7 @@ DEPENDENCIES
   rack-test
   rake
   rspec
+  selenium-webdriver
   slack-ruby-bot-server
   vcr
   webmock

spec/integration/add_to_slack_spec.rb

@@ -0,0 +1,22 @@
+require 'spec_helper'
+
+describe 'Add to Slack', js: true, type: :feature do
+  before do
+    ENV['SLACK_CLIENT_ID'] = 'client_id'
+    ENV['SLACK_CLIENT_SECRET'] = 'client_secret'
+  end
+  after do
+    ENV.delete 'SLACK_CLIENT_ID'
+    ENV.delete 'SLACK_CLIENT_SECRET'
+  end
+  it 'registers a team' do
+    allow_any_instance_of(Team).to receive(:ping!).and_return(ok: true)
+    expect(SlackRubyBotServer::Service.instance).to receive(:start!)
+    oauth_access = { 'bot' => { 'bot_access_token' => 'token' }, 'team_id' => 'team_id', 'team_name' => 'team_name' }
+    allow_any_instance_of(Slack::Web::Client).to receive(:oauth_access).with(hash_including(code: 'code')).and_return(oauth_access)
+    expect do
+      visit '/?code=code'
+      expect(page.find('#messages')).to have_content 'Team successfully registered!'
+    end.to change(Team, :count).by(1)
+  end
+end

spec/integration/index_spec.rb

@@ -0,0 +1,15 @@
+require 'spec_helper'
+
+describe 'index.html', js: true, type: :feature do
+  before do
+    ENV['SLACK_CLIENT_ID'] = 'slack-client-id'
+    visit '/'
+  end
+  after do
+    ENV.delete 'SLACK_CLIENT_ID'
+  end
+  it 'includes a link to add to slack with the client id' do
+    expect(title).to eq('Slack Ruby Bot Server')
+    expect(first('p[id=register] a')['href']).to eq 'https://slack.com/oauth/authorize?scope=bot&client_id=slack-client-id'
+  end
+end

spec/spec_helper.rb

@@ -4,26 +4,8 @@
 
 Bundler.require
 
-require 'slack-ruby-bot-server/rspec'
-
-Mongoid.load!(File.expand_path('../config/mongoid.yml', __dir__), ENV['RACK_ENV'])
-
-require 'database_cleaner'
-
-RSpec.configure do |config|
-  config.before :suite do
-    Mongo::Logger.logger.level = Logger::INFO
-
-    Mongoid::Tasks::Database.create_indexes
-  end
-
-  config.after :suite do
-    Mongoid.purge!
-  end
-
-  config.around :each do |example|
-    DatabaseCleaner.cleaning do
-      example.run
-    end
-  end
+Dir[File.join(File.dirname(__FILE__), 'support', '**/*.rb')].each do |file|
+  require file
 end
+
+require 'slack-ruby-bot-server/rspec'

spec/support/capybara.rb

@@ -0,0 +1,6 @@
+require 'capybara/rspec'
+
+Capybara.configure do |config|
+  config.app = SlackRubyBotServer::Api::Middleware.instance
+  config.server_port = 9293
+end

spec/support/database_cleaner.rb

@@ -0,0 +1,18 @@
+require 'database_cleaner'
+
+RSpec.configure do |config|
+  config.before :suite do
+    DatabaseCleaner.strategy = :truncation
+    DatabaseCleaner.clean_with :truncation
+  end
+
+  config.after :suite do
+    Mongoid.purge!
+  end
+
+  config.around :each do |example|
+    DatabaseCleaner.cleaning do
+      example.run
+    end
+  end
+end

spec/support/mongoid.rb

@@ -0,0 +1,10 @@
+Mongoid.load!(File.expand_path('../../config/mongoid.yml', __dir__), ENV['RACK_ENV'])
+
+RSpec.configure do |config|
+  config.before :suite do
+    Mongoid.logger.level = Logger::INFO
+    Mongo::Logger.logger.level = Logger::INFO
+
+    Mongoid::Tasks::Database.create_indexes
+  end
+end

spec/support/vcr.rb

@@ -0,0 +1,9 @@
+require 'vcr'
+
+VCR.configure do |config|
+  config.cassette_library_dir = 'spec/fixtures/slack'
+  config.hook_into :webmock
+  # config.default_cassette_options = { record: :new_episodes }
+  config.configure_rspec_metadata!
+  config.ignore_localhost = true
+end