jro_apps/redmine Commit - r12225:8a6f4c3880a5

1

# encoding: utf-8

1

# encoding: utf-8

2

#

2

#

3

# Redmine - project management software

3

# Redmine - project management software

4

5

#

5

#

6

# This program is free software; you can redistribute it and/or

6

# This program is free software; you can redistribute it and/or

7

# modify it under the terms of the GNU General Public License

7

# modify it under the terms of the GNU General Public License

8

# as published by the Free Software Foundation; either version 2

8

# as published by the Free Software Foundation; either version 2

9

# of the License, or (at your option) any later version.

9

# of the License, or (at your option) any later version.

10

#

10

#

11

# This program is distributed in the hope that it will be useful,

11

# This program is distributed in the hope that it will be useful,

12

# but WITHOUT ANY WARRANTY; without even the implied warranty of

12

# but WITHOUT ANY WARRANTY; without even the implied warranty of

13

# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the

13

# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the

14

# GNU General Public License for more details.

14

# GNU General Public License for more details.

15

#

15

#

16

# You should have received a copy of the GNU General Public License

16

# You should have received a copy of the GNU General Public License

17

# along with this program; if not, write to the Free Software

17

# along with this program; if not, write to the Free Software

18

# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.

18

# Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.

19

20

require File.expand_path('../../test_helper', __FILE__)

20

require File.expand_path('../../test_helper', __FILE__)

21

22

class AttachmentTest < ActiveSupport::TestCase

22

class AttachmentTest < ActiveSupport::TestCase

23

fixtures :users, :projects, :roles, :members, :member_roles,

23

fixtures :users, :projects, :roles, :members, :member_roles,

24

:enabled_modules, :issues, :trackers, :attachments

24

:enabled_modules, :issues, :trackers, :attachments

25

26

class MockFile

26

class MockFile

27

attr_reader :original_filename, :content_type, :content, :size

27

attr_reader :original_filename, :content_type, :content, :size

28

29

def initialize(attributes)

29

def initialize(attributes)

30

@original_filename = attributes[:original_filename]

30

@original_filename = attributes[:original_filename]

31

@content_type = attributes[:content_type]

31

@content_type = attributes[:content_type]

32

@content = attributes[:content] || "Content"

32

@content = attributes[:content] || "Content"

33

@size = content.size

33

@size = content.size

34

end

34

end

35

end

35

end

36

37

def setup

37

def setup

38

set_tmp_attachments_directory

38

set_tmp_attachments_directory

39

end

39

end

40

41

def test_container_for_new_attachment_should_be_nil

41

def test_container_for_new_attachment_should_be_nil

42

assert_nil Attachment.new.container

42

assert_nil Attachment.new.container

43

end

43

end

44

45

def test_filename_should_remove_eols

45

def test_filename_should_remove_eols

46

assert_equal "line_feed", Attachment.new(:filename => "line\nfeed").filename

46

assert_equal "line_feed", Attachment.new(:filename => "line\nfeed").filename

47

assert_equal "line_feed", Attachment.new(:filename => "some\npath/line\nfeed").filename

47

assert_equal "line_feed", Attachment.new(:filename => "some\npath/line\nfeed").filename

48

assert_equal "carriage_return", Attachment.new(:filename => "carriage\rreturn").filename

48

assert_equal "carriage_return", Attachment.new(:filename => "carriage\rreturn").filename

49

assert_equal "carriage_return", Attachment.new(:filename => "some\rpath/carriage\rreturn").filename

49

assert_equal "carriage_return", Attachment.new(:filename => "some\rpath/carriage\rreturn").filename

50

end

50

end

51

52

def test_create

52

def test_create

53

a = Attachment.new(:container => Issue.find(1),

53

a = Attachment.new(:container => Issue.find(1),

54

:file => uploaded_test_file("testfile.txt", "text/plain"),

54

:file => uploaded_test_file("testfile.txt", "text/plain"),

55

:author => User.find(1))

55

:author => User.find(1))

56

assert a.save

56

assert a.save

57

assert_equal 'testfile.txt', a.filename

57

assert_equal 'testfile.txt', a.filename

58

assert_equal 59, a.filesize

58

assert_equal 59, a.filesize

59

assert_equal 'text/plain', a.content_type

59

assert_equal 'text/plain', a.content_type

60

assert_equal 0, a.downloads

60

assert_equal 0, a.downloads

61

assert_equal '1478adae0d4eb06d35897518540e25d6', a.digest

61

assert_equal '1478adae0d4eb06d35897518540e25d6', a.digest

62

63

assert a.disk_directory

63

assert a.disk_directory

64

assert_match %r{\A\d{4}/\d{2}\z}, a.disk_directory

64

assert_match %r{\A\d{4}/\d{2}\z}, a.disk_directory

65

66

assert File.exist?(a.diskfile)

66

assert File.exist?(a.diskfile)

67

assert_equal 59, File.size(a.diskfile)

67

assert_equal 59, File.size(a.diskfile)

68

end

68

end

69

70

def test_copy_should_preserve_attributes

70

def test_copy_should_preserve_attributes

71

a = Attachment.find(1)

71

a = Attachment.find(1)

72

copy = a.copy

72

copy = a.copy

73

74

assert_save copy

74

assert_save copy

75

copy = Attachment.order('id DESC').first

75

copy = Attachment.order('id DESC').first

76

%w(filename filesize content_type author_id created_on description digest disk_filename disk_directory diskfile).each do |attribute|

76

%w(filename filesize content_type author_id created_on description digest disk_filename disk_directory diskfile).each do |attribute|

77

assert_equal a.send(attribute), copy.send(attribute), "#{attribute} was different"

77

assert_equal a.send(attribute), copy.send(attribute), "#{attribute} was different"

78

end

78

end

79

end

79

end

80

81

def test_size_should_be_validated_for_new_file

81

def test_size_should_be_validated_for_new_file

82

with_settings :attachment_max_size => 0 do

82

with_settings :attachment_max_size => 0 do

83

a = Attachment.new(:container => Issue.find(1),

83

a = Attachment.new(:container => Issue.find(1),

84

:file => uploaded_test_file("testfile.txt", "text/plain"),

84

:file => uploaded_test_file("testfile.txt", "text/plain"),

85

:author => User.find(1))

85

:author => User.find(1))

86

assert !a.save

86

assert !a.save

87

end

87

end

88

end

88

end

89

90

def test_size_should_not_be_validated_when_copying

90

def test_size_should_not_be_validated_when_copying

91

a = Attachment.create!(:container => Issue.find(1),

91

a = Attachment.create!(:container => Issue.find(1),

92

:file => uploaded_test_file("testfile.txt", "text/plain"),

92

:file => uploaded_test_file("testfile.txt", "text/plain"),

93

:author => User.find(1))

93

:author => User.find(1))

94

with_settings :attachment_max_size => 0 do

94

with_settings :attachment_max_size => 0 do

95

copy = a.copy

95

copy = a.copy

96

assert copy.save

96

assert copy.save

97

end

97

end

98

end

98

end

99

100

def test_description_length_should_be_validated

100

def test_description_length_should_be_validated

101

a = Attachment.new(:description => 'a' * 300)

101

a = Attachment.new(:description => 'a' * 300)

102

assert !a.save

102

assert !a.save

103

assert_not_equal [], a.errors[:description]

103

assert_not_equal [], a.errors[:description]

104

end

104

end

105

106

def test_destroy

106

def test_destroy

107

a = Attachment.new(:container => Issue.find(1),

107

a = Attachment.new(:container => Issue.find(1),

108

:file => uploaded_test_file("testfile.txt", "text/plain"),

108

:file => uploaded_test_file("testfile.txt", "text/plain"),

109

:author => User.find(1))

109

:author => User.find(1))

110

assert a.save

110

assert a.save

111

assert_equal 'testfile.txt', a.filename

111

assert_equal 'testfile.txt', a.filename

112

assert_equal 59, a.filesize

112

assert_equal 59, a.filesize

113

assert_equal 'text/plain', a.content_type

113

assert_equal 'text/plain', a.content_type

114

assert_equal 0, a.downloads

114

assert_equal 0, a.downloads

115

assert_equal '1478adae0d4eb06d35897518540e25d6', a.digest

115

assert_equal '1478adae0d4eb06d35897518540e25d6', a.digest

116

diskfile = a.diskfile

116

diskfile = a.diskfile

117

assert File.exist?(diskfile)

117

assert File.exist?(diskfile)

118

assert_equal 59, File.size(a.diskfile)

118

assert_equal 59, File.size(a.diskfile)

119

assert a.destroy

119

assert a.destroy

120

assert !File.exist?(diskfile)

120

assert !File.exist?(diskfile)

121

end

121

end

122

123

def test_destroy_should_not_delete_file_referenced_by_other_attachment

123

def test_destroy_should_not_delete_file_referenced_by_other_attachment

124

a = Attachment.create!(:container => Issue.find(1),

124

a = Attachment.create!(:container => Issue.find(1),

125

:file => uploaded_test_file("testfile.txt", "text/plain"),

125

:file => uploaded_test_file("testfile.txt", "text/plain"),

126

:author => User.find(1))

126

:author => User.find(1))

127

diskfile = a.diskfile

127

diskfile = a.diskfile

128

129

copy = a.copy

129

copy = a.copy

130

copy.save!

130

copy.save!

131

132

assert File.exists?(diskfile)

132

assert File.exists?(diskfile)

133

a.destroy

133

a.destroy

134

assert File.exists?(diskfile)

134

assert File.exists?(diskfile)

135

copy.destroy

135

copy.destroy

136

assert !File.exists?(diskfile)

136

assert !File.exists?(diskfile)

137

end

137

end

138

139

def test_create_should_auto_assign_content_type

139

def test_create_should_auto_assign_content_type

140

a = Attachment.new(:container => Issue.find(1),

140

a = Attachment.new(:container => Issue.find(1),

141

:file => uploaded_test_file("testfile.txt", ""),

141

:file => uploaded_test_file("testfile.txt", ""),

142

:author => User.find(1))

142

:author => User.find(1))

143

assert a.save

143

assert a.save

144

assert_equal 'text/plain', a.content_type

144

assert_equal 'text/plain', a.content_type

145

end

145

end

146

147

def test_identical_attachments_at_the_same_time_should_not_overwrite

147

def test_identical_attachments_at_the_same_time_should_not_overwrite

148

a1 = Attachment.create!(:container => Issue.find(1),

148

a1 = Attachment.create!(:container => Issue.find(1),

149

:file => uploaded_test_file("testfile.txt", ""),

149

:file => uploaded_test_file("testfile.txt", ""),

150

:author => User.find(1))

150

:author => User.find(1))

151

a2 = Attachment.create!(:container => Issue.find(1),

151

a2 = Attachment.create!(:container => Issue.find(1),

152

:file => uploaded_test_file("testfile.txt", ""),

152

:file => uploaded_test_file("testfile.txt", ""),

153

:author => User.find(1))

153

:author => User.find(1))

154

assert a1.disk_filename != a2.disk_filename

154

assert a1.disk_filename != a2.disk_filename

155

end

155

end

156

157

def test_filename_should_be_basenamed

157

def test_filename_should_be_basenamed

158

a = Attachment.new(:file => MockFile.new(:original_filename => "path/to/the/file"))

158

a = Attachment.new(:file => MockFile.new(:original_filename => "path/to/the/file"))

159

assert_equal 'file', a.filename

159

assert_equal 'file', a.filename

160

end

160

end

161

162

def test_filename_should_be_sanitized

162

def test_filename_should_be_sanitized

163

a = Attachment.new(:file => MockFile.new(:original_filename => "valid:[] invalid:?%*|\"'<>chars"))

163

a = Attachment.new(:file => MockFile.new(:original_filename => "valid:[] invalid:?%*|\"'<>chars"))

164

assert_equal 'valid_[] invalid_chars', a.filename

164

assert_equal 'valid_[] invalid_chars', a.filename

165

end

165

end

166

167

def test_diskfilename

167

def test_diskfilename

168

assert Attachment.disk_filename("test_file.txt") =~ /^\d{12}_test_file.txt$/

168

assert Attachment.disk_filename("test_file.txt") =~ /^\d{12}_test_file.txt$/

169

assert_equal 'test_file.txt', Attachment.disk_filename("test_file.txt")[13..-1]

169

assert_equal 'test_file.txt', Attachment.disk_filename("test_file.txt")[13..-1]

170

assert_equal '770c509475505f37c2b8fb6030434d6b.txt', Attachment.disk_filename("test_accentué.txt")[13..-1]

170

assert_equal '770c509475505f37c2b8fb6030434d6b.txt', Attachment.disk_filename("test_accentué.txt")[13..-1]

171

assert_equal 'f8139524ebb8f32e51976982cd20a85d', Attachment.disk_filename("test_accentué")[13..-1]

171

assert_equal 'f8139524ebb8f32e51976982cd20a85d', Attachment.disk_filename("test_accentué")[13..-1]

172

assert_equal 'cbb5b0f30978ba03731d61f9f6d10011', Attachment.disk_filename("test_accentué.ça")[13..-1]

172

assert_equal 'cbb5b0f30978ba03731d61f9f6d10011', Attachment.disk_filename("test_accentué.ça")[13..-1]

173

end

173

end

174

175

def test_title

175

def test_title

176

a = Attachment.new(:filename => "test.png")

176

a = Attachment.new(:filename => "test.png")

177

assert_equal "test.png", a.title

177

assert_equal "test.png", a.title

178

179

a = Attachment.new(:filename => "test.png", :description => "Cool image")

179

a = Attachment.new(:filename => "test.png", :description => "Cool image")

180

assert_equal "test.png (Cool image)", a.title

180

assert_equal "test.png (Cool image)", a.title

181

end

181

end

182

183

def test_prune_should_destroy_old_unattached_attachments

183

def test_prune_should_destroy_old_unattached_attachments

184

Attachment.create!(:file => uploaded_test_file("testfile.txt", ""), :author_id => 1, :created_on => 2.days.ago)

184

Attachment.create!(:file => uploaded_test_file("testfile.txt", ""), :author_id => 1, :created_on => 2.days.ago)

185

Attachment.create!(:file => uploaded_test_file("testfile.txt", ""), :author_id => 1, :created_on => 2.days.ago)

185

Attachment.create!(:file => uploaded_test_file("testfile.txt", ""), :author_id => 1, :created_on => 2.days.ago)

186

Attachment.create!(:file => uploaded_test_file("testfile.txt", ""), :author_id => 1)

186

Attachment.create!(:file => uploaded_test_file("testfile.txt", ""), :author_id => 1)

187

188

assert_difference 'Attachment.count', -2 do

188

assert_difference 'Attachment.count', -2 do

189

Attachment.prune

189

Attachment.prune

190

end

190

end

191

end

191

end

192

193

def test_move_from_root_to_target_directory_should_move_root_files

193

def test_move_from_root_to_target_directory_should_move_root_files

194

a = Attachment.find(20)

194

a = Attachment.find(20)

195

assert a.disk_directory.blank?

195

assert a.disk_directory.blank?

196

# Create a real file for this fixture

196

# Create a real file for this fixture

197

File.open(a.diskfile, "w") do |f|

197

File.open(a.diskfile, "w") do |f|

198

f.write "test file at the root of files directory"

198

f.write "test file at the root of files directory"

199

end

199

end

200

assert a.readable?

200

assert a.readable?

201

Attachment.move_from_root_to_target_directory

201

Attachment.move_from_root_to_target_directory

202

203

a.reload

203

a.reload

204

assert_equal '2012/05', a.disk_directory

204

assert_equal '2012/05', a.disk_directory

205

assert a.readable?

205

assert a.readable?

206

end

206

end

207

208

test "Attachmnet.attach_files should attach the file" do

208

test "Attachmnet.attach_files should attach the file" do

209

issue = Issue.first

209

issue = Issue.first

210

assert_difference 'Attachment.count' do

210

assert_difference 'Attachment.count' do

211

Attachment.attach_files(issue,

211

Attachment.attach_files(issue,

212

'1' => {

212

'1' => {

213

'file' => uploaded_test_file('testfile.txt', 'text/plain'),

213

'file' => uploaded_test_file('testfile.txt', 'text/plain'),

214

'description' => 'test'

214

'description' => 'test'

215

})

215

})

216

end

216

end

217

attachment = Attachment.order('id DESC').first

218

attachment = Attachment.first(:order => 'id DESC')

219

assert_equal issue, attachment.container

218

assert_equal issue, attachment.container

220

assert_equal 'testfile.txt', attachment.filename

219

assert_equal 'testfile.txt', attachment.filename

221

assert_equal 59, attachment.filesize

220

assert_equal 59, attachment.filesize

222

assert_equal 'test', attachment.description

221

assert_equal 'test', attachment.description

223

assert_equal 'text/plain', attachment.content_type

222

assert_equal 'text/plain', attachment.content_type

224

assert File.exists?(attachment.diskfile)

223

assert File.exists?(attachment.diskfile)

225

assert_equal 59, File.size(attachment.diskfile)

224

assert_equal 59, File.size(attachment.diskfile)

226

end

225

end

227

226

228

test "Attachmnet.attach_files should add unsaved files to the object as unsaved attachments" do

227

test "Attachmnet.attach_files should add unsaved files to the object as unsaved attachments" do

229

# Max size of 0 to force Attachment creation failures

228

# Max size of 0 to force Attachment creation failures

230

with_settings(:attachment_max_size => 0) do

229

with_settings(:attachment_max_size => 0) do

231

@project = Project.find(1)

230

@project = Project.find(1)

232

response = Attachment.attach_files(@project, {

231

response = Attachment.attach_files(@project, {

233

'1' => {'file' => mock_file, 'description' => 'test'},

232

'1' => {'file' => mock_file, 'description' => 'test'},

234

'2' => {'file' => mock_file, 'description' => 'test'}

233

'2' => {'file' => mock_file, 'description' => 'test'}

235

})

234

})

236

235

237

assert response[:unsaved].present?

236

assert response[:unsaved].present?

238

assert_equal 2, response[:unsaved].length

237

assert_equal 2, response[:unsaved].length

239

assert response[:unsaved].first.new_record?

238

assert response[:unsaved].first.new_record?

240

assert response[:unsaved].second.new_record?

239

assert response[:unsaved].second.new_record?

241

assert_equal response[:unsaved], @project.unsaved_attachments

240

assert_equal response[:unsaved], @project.unsaved_attachments

242

end

241

end

243

end

242

end

244

243

245

def test_latest_attach

244

def test_latest_attach

246

set_fixtures_attachments_directory

245

set_fixtures_attachments_directory

247

a1 = Attachment.find(16)

246

a1 = Attachment.find(16)

248

assert_equal "testfile.png", a1.filename

247

assert_equal "testfile.png", a1.filename

249

assert a1.readable?

248

assert a1.readable?

250

assert (! a1.visible?(User.anonymous))

249

assert (! a1.visible?(User.anonymous))

251

assert a1.visible?(User.find(2))

250

assert a1.visible?(User.find(2))

252

a2 = Attachment.find(17)

251

a2 = Attachment.find(17)

253

assert_equal "testfile.PNG", a2.filename

252

assert_equal "testfile.PNG", a2.filename

254

assert a2.readable?

253

assert a2.readable?

255

assert (! a2.visible?(User.anonymous))

254

assert (! a2.visible?(User.anonymous))

256

assert a2.visible?(User.find(2))

255

assert a2.visible?(User.find(2))

257

assert a1.created_on < a2.created_on

256

assert a1.created_on < a2.created_on

258

257

259

la1 = Attachment.latest_attach([a1, a2], "testfile.png")

258

la1 = Attachment.latest_attach([a1, a2], "testfile.png")

260

assert_equal 17, la1.id

259

assert_equal 17, la1.id

261

la2 = Attachment.latest_attach([a1, a2], "Testfile.PNG")

260

la2 = Attachment.latest_attach([a1, a2], "Testfile.PNG")

262

assert_equal 17, la2.id

261

assert_equal 17, la2.id

263

262

264

set_tmp_attachments_directory

263

set_tmp_attachments_directory

265

end

264

end

266

265

267

def test_thumbnailable_should_be_true_for_images

266

def test_thumbnailable_should_be_true_for_images

268

assert_equal true, Attachment.new(:filename => 'test.jpg').thumbnailable?

267

assert_equal true, Attachment.new(:filename => 'test.jpg').thumbnailable?

269

end

268

end

270

269

271

def test_thumbnailable_should_be_true_for_non_images

270

def test_thumbnailable_should_be_true_for_non_images

272

assert_equal false, Attachment.new(:filename => 'test.txt').thumbnailable?

271

assert_equal false, Attachment.new(:filename => 'test.txt').thumbnailable?

273

end

272

end

274

273

275

if convert_installed?

274

if convert_installed?

276

def test_thumbnail_should_generate_the_thumbnail

275

def test_thumbnail_should_generate_the_thumbnail

277

set_fixtures_attachments_directory

276

set_fixtures_attachments_directory

278

attachment = Attachment.find(16)

277

attachment = Attachment.find(16)

279

Attachment.clear_thumbnails

278

Attachment.clear_thumbnails

280

279

281

assert_difference "Dir.glob(File.join(Attachment.thumbnails_storage_path, '*.thumb')).size" do

280

assert_difference "Dir.glob(File.join(Attachment.thumbnails_storage_path, '*.thumb')).size" do

282

thumbnail = attachment.thumbnail

281

thumbnail = attachment.thumbnail

283

assert_equal "16_8e0294de2441577c529f170b6fb8f638_100.thumb", File.basename(thumbnail)

282

assert_equal "16_8e0294de2441577c529f170b6fb8f638_100.thumb", File.basename(thumbnail)

284

assert File.exists?(thumbnail)

283

assert File.exists?(thumbnail)

285

end

284

end

286

end

285

end

287

else

286

else

288

puts '(ImageMagick convert not available)'

287

puts '(ImageMagick convert not available)'

289

end

288

end

290

end

289

end

	Site-wide shortcuts
/	Use quick search box
g h	Goto home page
g g	Goto my private gists page
g G	Goto my public gists page
g 0-9	Goto bookmarked items from 0-9
n r	New repository page
n g	New gist page

	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

             # encoding: utf-8
             #
             # Redmine - project management software
             # Copyright (C) 2006-2013  Jean-Philippe Lang
             #
             # This program is free software; you can redistribute it and/or
             # modify it under the terms of the GNU General Public License
             # as published by the Free Software Foundation; either version 2
             # of the License, or (at your option) any later version.
             #
             # This program is distributed in the hope that it will be useful,
             # but WITHOUT ANY WARRANTY; without even the implied warranty of
             # MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
             # GNU General Public License for more details.
             #
             # You should have received a copy of the GNU General Public License
             # along with this program; if not, write to the Free Software
             # Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA  02110-1301, USA.
             require File.expand_path('../../test_helper', __FILE__)
             class AttachmentTest < ActiveSupport::TestCase
               fixtures :users, :projects, :roles, :members, :member_roles,
                        :enabled_modules, :issues, :trackers, :attachments
               class MockFile
                 attr_reader :original_filename, :content_type, :content, :size
                 def initialize(attributes)
                   @original_filename = attributes[:original_filename]
                   @content_type = attributes[:content_type]
                   @content = attributes[:content] || "Content"
                   @size = content.size
                 end
               end
               def setup
                 set_tmp_attachments_directory
               end
               def test_container_for_new_attachment_should_be_nil
                 assert_nil Attachment.new.container
               end
               def test_filename_should_remove_eols
                 assert_equal "line_feed", Attachment.new(:filename => "line\nfeed").filename
                 assert_equal "line_feed", Attachment.new(:filename => "some\npath/line\nfeed").filename
                 assert_equal "carriage_return", Attachment.new(:filename => "carriage\rreturn").filename
                 assert_equal "carriage_return", Attachment.new(:filename => "some\rpath/carriage\rreturn").filename
               end
               def test_create
                 a = Attachment.new(:container => Issue.find(1),
                                    :file => uploaded_test_file("testfile.txt", "text/plain"),
                                    :author => User.find(1))
                 assert a.save
                 assert_equal 'testfile.txt', a.filename
                 assert_equal 59, a.filesize
                 assert_equal 'text/plain', a.content_type
                 assert_equal 0, a.downloads
                 assert_equal '1478adae0d4eb06d35897518540e25d6', a.digest
                 assert a.disk_directory
                 assert_match %r{\A\d{4}/\d{2}\z}, a.disk_directory
                 assert File.exist?(a.diskfile)
                 assert_equal 59, File.size(a.diskfile)
               end
               def test_copy_should_preserve_attributes
                 a = Attachment.find(1)
                 copy = a.copy
                 assert_save copy
                 copy = Attachment.order('id DESC').first
                 %w(filename filesize content_type author_id created_on description digest disk_filename disk_directory diskfile).each do |attribute|
                   assert_equal a.send(attribute), copy.send(attribute), "#{attribute} was different"
                 end
               end
               def test_size_should_be_validated_for_new_file
                 with_settings :attachment_max_size => 0 do
                   a = Attachment.new(:container => Issue.find(1),
                                      :file => uploaded_test_file("testfile.txt", "text/plain"),
                                      :author => User.find(1))
                   assert !a.save
                 end
               end
               def test_size_should_not_be_validated_when_copying
                 a = Attachment.create!(:container => Issue.find(1),
                                        :file => uploaded_test_file("testfile.txt", "text/plain"),
                                        :author => User.find(1))
                 with_settings :attachment_max_size => 0 do
                   copy = a.copy
                   assert copy.save
                 end
               end
               def test_description_length_should_be_validated
                 a = Attachment.new(:description => 'a' * 300)
                 assert !a.save
                 assert_not_equal [], a.errors[:description]
               end
               def test_destroy
                 a = Attachment.new(:container => Issue.find(1),
                                    :file => uploaded_test_file("testfile.txt", "text/plain"),
                                    :author => User.find(1))
                 assert a.save
                 assert_equal 'testfile.txt', a.filename
                 assert_equal 59, a.filesize
                 assert_equal 'text/plain', a.content_type
                 assert_equal 0, a.downloads
                 assert_equal '1478adae0d4eb06d35897518540e25d6', a.digest
                 diskfile = a.diskfile
                 assert File.exist?(diskfile)
                 assert_equal 59, File.size(a.diskfile)
                 assert a.destroy
                 assert !File.exist?(diskfile)
               end
               def test_destroy_should_not_delete_file_referenced_by_other_attachment
                 a = Attachment.create!(:container => Issue.find(1),
                                        :file => uploaded_test_file("testfile.txt", "text/plain"),
                                        :author => User.find(1))
                 diskfile = a.diskfile
                 copy = a.copy
                 copy.save!
                 assert File.exists?(diskfile)
                 a.destroy
                 assert File.exists?(diskfile)
                 copy.destroy
                 assert !File.exists?(diskfile)
               end
               def test_create_should_auto_assign_content_type
                 a = Attachment.new(:container => Issue.find(1),
                                    :file => uploaded_test_file("testfile.txt", ""),
                                    :author => User.find(1))
                 assert a.save
                 assert_equal 'text/plain', a.content_type
               end
               def test_identical_attachments_at_the_same_time_should_not_overwrite
                 a1 = Attachment.create!(:container => Issue.find(1),
                                         :file => uploaded_test_file("testfile.txt", ""),
                                         :author => User.find(1))
                 a2 = Attachment.create!(:container => Issue.find(1),
                                         :file => uploaded_test_file("testfile.txt", ""),
                                         :author => User.find(1))
                 assert a1.disk_filename != a2.disk_filename
               end
               def test_filename_should_be_basenamed
                 a = Attachment.new(:file => MockFile.new(:original_filename => "path/to/the/file"))
                 assert_equal 'file', a.filename
               end
               def test_filename_should_be_sanitized
                 a = Attachment.new(:file => MockFile.new(:original_filename => "valid:[] invalid:?%*|\"'<>chars"))
                 assert_equal 'valid_[] invalid_chars', a.filename
               end
               def test_diskfilename
                 assert Attachment.disk_filename("test_file.txt") =~ /^\d{12}_test_file.txt$/
                 assert_equal 'test_file.txt', Attachment.disk_filename("test_file.txt")[13..-1]
                 assert_equal '770c509475505f37c2b8fb6030434d6b.txt', Attachment.disk_filename("test_accentué.txt")[13..-1]
                 assert_equal 'f8139524ebb8f32e51976982cd20a85d', Attachment.disk_filename("test_accentué")[13..-1]
                 assert_equal 'cbb5b0f30978ba03731d61f9f6d10011', Attachment.disk_filename("test_accentué.ça")[13..-1]
               end
               def test_title
                 a = Attachment.new(:filename => "test.png")
                 assert_equal "test.png", a.title
                 a = Attachment.new(:filename => "test.png", :description => "Cool image")
                 assert_equal "test.png (Cool image)", a.title
               end
               def test_prune_should_destroy_old_unattached_attachments
                 Attachment.create!(:file => uploaded_test_file("testfile.txt", ""), :author_id => 1, :created_on => 2.days.ago)
                 Attachment.create!(:file => uploaded_test_file("testfile.txt", ""), :author_id => 1, :created_on => 2.days.ago)
                 Attachment.create!(:file => uploaded_test_file("testfile.txt", ""), :author_id => 1)
                 assert_difference 'Attachment.count', -2 do
                   Attachment.prune
                 end
               end
               def test_move_from_root_to_target_directory_should_move_root_files
                 a = Attachment.find(20)
                 assert a.disk_directory.blank?
                 # Create a real file for this fixture
                 File.open(a.diskfile, "w") do |f|
                   f.write "test file at the root of files directory"
                 end
                 assert a.readable?
                 Attachment.move_from_root_to_target_directory
                 a.reload
                 assert_equal '2012/05', a.disk_directory
                 assert a.readable?
               end
               test "Attachmnet.attach_files should attach the file" do
                 issue = Issue.first
                 assert_difference 'Attachment.count' do
                   Attachment.attach_files(issue,
                     '1' => {
                       'file' => uploaded_test_file('testfile.txt', 'text/plain'),
                       'description' => 'test'
                     })
                 end
+                attachment = Attachment.order('id DESC').first
-                attachment = Attachment.first(:order => 'id DESC')
                 assert_equal issue, attachment.container
                 assert_equal 'testfile.txt', attachment.filename
                 assert_equal 59, attachment.filesize
                 assert_equal 'test', attachment.description
                 assert_equal 'text/plain', attachment.content_type
                 assert File.exists?(attachment.diskfile)
                 assert_equal 59, File.size(attachment.diskfile)
               end
               test "Attachmnet.attach_files should add unsaved files to the object as unsaved attachments" do
                 # Max size of 0 to force Attachment creation failures
                 with_settings(:attachment_max_size => 0) do
                   @project = Project.find(1)
                   response = Attachment.attach_files(@project, {
                                                        '1' => {'file' => mock_file, 'description' => 'test'},
                                                        '2' => {'file' => mock_file, 'description' => 'test'}
                                                      })
                   assert response[:unsaved].present?
                   assert_equal 2, response[:unsaved].length
                   assert response[:unsaved].first.new_record?
                   assert response[:unsaved].second.new_record?
                   assert_equal response[:unsaved], @project.unsaved_attachments
                 end
               end
               def test_latest_attach
                 set_fixtures_attachments_directory
                 a1 = Attachment.find(16)
                 assert_equal "testfile.png", a1.filename
                 assert a1.readable?
                 assert (! a1.visible?(User.anonymous))
                 assert a1.visible?(User.find(2))
                 a2 = Attachment.find(17)
                 assert_equal "testfile.PNG", a2.filename
                 assert a2.readable?
                 assert (! a2.visible?(User.anonymous))
                 assert a2.visible?(User.find(2))
                 assert a1.created_on < a2.created_on
                 la1 = Attachment.latest_attach([a1, a2], "testfile.png")
                 assert_equal 17, la1.id
                 la2 = Attachment.latest_attach([a1, a2], "Testfile.PNG")
                 assert_equal 17, la2.id
                 set_tmp_attachments_directory
               end
               def test_thumbnailable_should_be_true_for_images
                 assert_equal true, Attachment.new(:filename => 'test.jpg').thumbnailable?
               end
               def test_thumbnailable_should_be_true_for_non_images
                 assert_equal false, Attachment.new(:filename => 'test.txt').thumbnailable?
               end
               if convert_installed?
                 def test_thumbnail_should_generate_the_thumbnail
                   set_fixtures_attachments_directory
                   attachment = Attachment.find(16)
                   Attachment.clear_thumbnails
                   assert_difference "Dir.glob(File.join(Attachment.thumbnails_storage_path, '*.thumb')).size" do
                     thumbnail = attachment.thumbnail
                     assert_equal "16_8e0294de2441577c529f170b6fb8f638_100.thumb", File.basename(thumbnail)
                     assert File.exists?(thumbnail)
                   end
                 end
               else
                 puts '(ImageMagick convert not available)'
               end
             end