ActionItem1210: fixing tests

Antonio Terceiro
1 parent 0402708d
Showing 2 changed files with 1 additions and 19 deletions Show diff stats
app/models/mail_conf.rb
test/unit/user_mailer_test.rb
@@ -9,7 +9,7 @@ class MailConf
       if File.exists?(config_file)
         YAML.load_file(config_file)
       else
-        {}
+        { 'webmail_url' =>  'http://webmail.example.com' }
       end
     end
@@ -20,24 +20,6 @@ class UserMailerTest &lt; Test::Unit::TestCase
     end
   end
-  should 'deliver notify when activate email' do
-    u = create_user('testuser')
-    assert_difference ActionMailer::Base.deliveries, :size do
-      u.enable_email = true
-      u.save!
-    end
-  end
-
-  should 'not deliver notify when disactivate email' do
-    u = create_user('testuser')
-    u.enable_email = true
-    u.save!
-    assert_no_difference ActionMailer::Base.deliveries, :size do
-      u.enable_email = false
-      u.save!
-    end
-  end
-
   private
     def read_fixture(action)
	@@ -9,7 +9,7 @@ class MailConf		@@ -9,7 +9,7 @@ class MailConf
9	if File.exists?(config_file)	9	if File.exists?(config_file)
10	YAML.load_file(config_file)	10	YAML.load_file(config_file)
11	else	11	else
12	- {}	12	+ { 'webmail_url' => 'http://webmail.example.com' }
13	end	13	end
14	end	14	end
15		15
	@@ -20,24 +20,6 @@ class UserMailerTest < Test::Unit::TestCase		@@ -20,24 +20,6 @@ class UserMailerTest < Test::Unit::TestCase
20	end	20	end
21	end	21	end
22		22
23	- should 'deliver notify when activate email' do
24	- u = create_user('testuser')
25	- assert_difference ActionMailer::Base.deliveries, :size do
26	- u.enable_email = true
27	- u.save!
28	- end
29	- end
30	-
31	- should 'not deliver notify when disactivate email' do
32	- u = create_user('testuser')
33	- u.enable_email = true
34	- u.save!
35	- assert_no_difference ActionMailer::Base.deliveries, :size do
36	- u.enable_email = false
37	- u.save!
38	- end
39	- end
40	-
41	private	23	private
42		24
43	def read_fixture(action)	25	def read_fixture(action)