Made team delete operation an archive operation

Author: begedin

app/controllers/teams_controller.rb

@@ -1,3 +1,4 @@
+require 'team_playbook/scenario/archive_team'
 require 'team_playbook/scenario/create_team'
 require 'team_playbook/scenario/change_plan_for_team'
 require 'team_playbook/scenario/add_card_to_team'
@@ -24,7 +25,7 @@ def show
   def destroy
     if has_team_subdomain?
       authorize! :destroy, current_team
-      TeamPlaybook::Scenario::DeleteTeam.new.call(team: current_team)
+      TeamPlaybook::Scenario::ArchiveTeam.new.call(team: current_team)
       render nothing: true, status: 204
     else
       forbidden
@@ -47,4 +48,4 @@ def team_params
   def credit_card_required
     render json: {error: "A credit card is required for a paid plan."}, status: :unprocessable_entity
   end
-end
+end

app/models/team.rb

@@ -1,10 +1,12 @@
 class Team < ActiveRecord::Base
+  enum status: [:archived, :active]
+
   validates :subdomain, presence: true, exclusion: { in: Settings.reserved_subdomains,
     message: "%{value} is not a valid subdomain." }
   validates :name, presence: true
 
   belongs_to :owner, class_name: "User"
-  has_many :team_memberships, dependent: :delete_all
+  has_many :team_memberships
   has_many :members, source: :user, through: :team_memberships
 
   has_one :subscription
@@ -13,4 +15,4 @@ class Team < ActiveRecord::Base
   delegate :name, to: :plan, prefix: true
   delegate :slug, to: :plan, prefix: true
 
-end
+end

db/migrate/20150515071915_add_status_field_to_team.rb

@@ -0,0 +1,5 @@
+class AddStatusFieldToTeam < ActiveRecord::Migration
+  def change
+    add_column :teams, :status, :integer, default: 1
+  end
+end

db/schema.rb

@@ -11,7 +11,7 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema.define(version: 20150514124031) do
+ActiveRecord::Schema.define(version: 20150515071915) do
 
   # These are extensions that must be enabled in order to support this database
   enable_extension "plpgsql"
@@ -45,12 +45,13 @@
   end
 
   create_table "teams", force: :cascade do |t|
-    t.string   "name",               null: false
-    t.string   "subdomain",          null: false
-    t.datetime "created_at",         null: false
-    t.datetime "updated_at",         null: false
+    t.string   "name",                           null: false
+    t.string   "subdomain",                      null: false
+    t.datetime "created_at",                     null: false
+    t.datetime "updated_at",                     null: false
     t.integer  "owner_id"
     t.string   "stripe_customer_id"
+    t.integer  "status",             default: 1
   end
 
   create_table "users", force: :cascade do |t|

lib/team_playbook/scenario/delete_team.rb renamed to lib/team_playbook/scenario/archive_team.rb

@@ -1,8 +1,8 @@
 module TeamPlaybook
   module Scenario
-    class DeleteTeam
+    class ArchiveTeam
       def call(team:)
-        team.destroy
+        team.archived!
       end
     end
   end

spec/lib/team_playbook/scenario/delete_team_spec.rb

@@ -1,26 +1,15 @@
 require 'rails_helper'
-require 'team_playbook/scenario/delete_team'
+require 'team_playbook/scenario/archive_team'
 
 module TeamPlaybook
   module Scenario
-    describe DeleteTeam do
-      it "should delete team" do
+    describe ArchiveTeam do
+      it "should archive team" do
         team = create(:team)
 
-        DeleteTeam.new.call(team: team)
-        expect(team).not_to be_persisted
-        expect(Team.all.count).to be 0
-      end
-
-      it "should delete associated memberships" do
-        user = create(:user)
-        team = create(:team)
-        other_team = create(:team)
-        create_list(:team_membership, 10, team: team)
-        create_list(:team_membership, 5, team: other_team)
-
-        DeleteTeam.new.call(team: team)
-        expect(TeamMembership.all.count).to eq 5
+        ArchiveTeam.new.call(team: team)
+        expect(team.archived?).to be true
+        expect(Team.active.count).to be 0
       end
     end
   end