Removed migration code from legacy SLACK_API_TOKEN.

Author: dblock

CHANGELOG.md

@@ -2,6 +2,7 @@
 
 #### 0.10.1 (Next)
 
+* [#101](https://github.com/slack-ruby/slack-ruby-bot-server/pull/101): Removed support for migrating from a legacy `SLACK_API_TOKEN` - [@dblock](https://github.com/dblock).
 * [#98](https://github.com/slack-ruby/slack-ruby-bot-server/pull/98): Removed `unicorn` from gem dependencies - [@dblock](https://github.com/dblock).
 * [#90](https://github.com/slack-ruby/slack-ruby-bot-server/pull/90): Update ActiveRecord sample app to support ENV variables in `postgresql.yml` - [@ashkan18](https://github.com/ashkan18).
 * Your contribution here.

UPGRADING.md

@@ -3,6 +3,12 @@ Upgrading Slack-Ruby-Bot-Server
 
 ### Upgrading to >= 0.10.1
 
+#### Migration from SLACK_API_TOKEN
+
+The code to automatically migrate from a legacy SLACK_API_TOKEN has been removed.
+
+See [#101](https://github.com/slack-ruby/slack-ruby-bot-server/pull/101) for more information.
+
 #### Unicorn Dependency
 
 The dependency on `unicorn` has been removed from gemspec. Use `unicorn` or `puma` or another application server as you see fit by explicitly adding a dependency in your Gemfile.

lib/slack-ruby-bot-server/app.rb

@@ -4,7 +4,6 @@ def prepare!
       check_database!
       init_database!
       mark_teams_active!
-      migrate_from_single_team!
       update_team_name_and_id!
       purge_inactive_teams!
       configure_global_aliases!
@@ -47,14 +46,6 @@ def update_team_name_and_id!
       end
     end
 
-    def migrate_from_single_team!
-      return unless ENV.key?('SLACK_API_TOKEN')
-      logger.info 'Migrating from env SLACK_API_TOKEN ...'
-      team = Team.find_or_create_from_env!
-      logger.info "Automatically migrated team: #{team}."
-      logger.warn "You should unset ENV['SLACK_API_TOKEN']."
-    end
-
     def purge_inactive_teams!
       Team.purge!
     end

lib/slack-ruby-bot-server/models/team/methods.rb

@@ -38,18 +38,5 @@ def ping!
         presence: client.users_getPresence(user: auth['user_id'])
       }
     end
-
-    def self.find_or_create_from_env!
-      token = ENV['SLACK_API_TOKEN']
-      return unless token
-      team = Team.where(token: token).first
-      team ||= Team.new(token: token)
-      info = Slack::Web::Client.new(token: token).team_info
-      team.team_id = info['team']['id']
-      team.name = info['team']['name']
-      team.domain = info['team']['domain']
-      team.save!
-      team
-    end
   end
 end

spec/models/team_spec.rb

@@ -1,24 +1,6 @@
 require 'spec_helper'
 
 describe Team do
-  context '#find_or_create_from_env!' do
-    before do
-      ENV['SLACK_API_TOKEN'] = 'token'
-    end
-    context 'team', vcr: { cassette_name: 'team_info' } do
-      it 'creates a team' do
-        expect { Team.find_or_create_from_env! }.to change(Team, :count).by(1)
-        team = Team.first
-        expect(team.team_id).to eq 'T04KB5WQH'
-        expect(team.name).to eq 'dblock'
-        expect(team.domain).to eq 'dblockdotorg'
-        expect(team.token).to eq 'token'
-      end
-    end
-    after do
-      ENV.delete 'SLACK_API_TOKEN'
-    end
-  end
   context '#purge!' do
     let!(:active_team) { Fabricate(:team) }
     let!(:inactive_team) { Fabricate(:team, active: false) }