jro_apps/redmine Commit - r1301:80a5ad7ff344

Fixed: Boards are not deleted when project is deleted (closes #963)....

Jean-Philippe Lang -

r1301:80a5ad7ff344

parent child

Context file:

r1301:80a5ad7ff344

app/models/project.rb +1 -1

               has_many :documents, :dependent => :destroy
               has_many :news, :dependent => :delete_all, :include => :author
               has_many :issue_categories, :dependent => :delete_all, :order => "#{IssueCategory.table_name}.name"
-              has_many :boards, :order => "position ASC"
+              has_many :boards, :dependent => :destroy, :order => "position ASC"
               has_one :repository, :dependent => :destroy
               has_many :changesets, :through => :repository
               has_one :wiki, :dependent => :destroy

test/unit/project_test.rb +6 -3

             require File.dirname(__FILE__) + '/../test_helper'
             class ProjectTest < Test::Unit::TestCase
-              fixtures :projects, :issues, :issue_statuses, :journals, :journal_details, :users, :members, :roles, :projects_trackers, :trackers
+              fixtures :projects, :issues, :issue_statuses, :journals, :journal_details, :users, :members, :roles, :projects_trackers, :trackers, :boards
               def setup
                 @ecookbook = Project.find(1)
                 assert_equal 2, @ecookbook.members.size
                 # and 1 is locked
                 assert_equal 3, Member.find(:all, :conditions => ['project_id = ?', @ecookbook.id]).size
+                # some boards
+                assert @ecookbook.boards.any?
                 @ecookbook.destroy
                 # make sure that the project non longer exists
                 assert_raise(ActiveRecord::RecordNotFound) { Project.find(@ecookbook.id) }
-                # make sure all members have been removed
+                # make sure related data was removed
-                assert_equal 0, Member.find(:all, :conditions => ['project_id = ?', @ecookbook.id]).size
+                assert Member.find(:all, :conditions => ['project_id = ?', @ecookbook.id]).empty?
+                assert Board.find(:all, :conditions => ['project_id = ?', @ecookbook.id]).empty?
               end
               def test_subproject_ok

General Comments 0

You need to be logged in to leave comments. Login now

No TODOs yet

	Repositories
g s	Goto summary page
g c	Goto changelog page
g f	Goto files page
g F	Goto files page with file search activated
g p	Goto pull requests page
g o	Goto repository settings
g O	Goto repository access permissions settings
t s	Toggle sidebar on some pages