jro_apps/redmine Commit - r12900:80ba11a02944

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 AttachmentsControllerTest < ActionController::TestCase

22

class AttachmentsControllerTest < ActionController::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

:versions, :wiki_pages, :wikis, :documents

25

:versions, :wiki_pages, :wikis, :documents

26

27

def setup

27

def setup

28

User.current = nil

28

User.current = nil

29

set_fixtures_attachments_directory

29

set_fixtures_attachments_directory

30

end

30

end

31

32

def teardown

32

def teardown

33

set_tmp_attachments_directory

33

set_tmp_attachments_directory

34

end

34

end

35

36

def test_show_diff

36

def test_show_diff

37

['inline', 'sbs'].each do |dt|

37

['inline', 'sbs'].each do |dt|

38

# 060719210727_changeset_utf8.diff

38

# 060719210727_changeset_utf8.diff

39

get :show, :id => 14, :type => dt

39

get :show, :id => 14, :type => dt

40

assert_response :success

40

assert_response :success

41

assert_template 'diff'

41

assert_template 'diff'

42

assert_equal 'text/html', @response.content_type

42

assert_equal 'text/html', @response.content_type

43

assert_tag 'th',

43

assert_tag 'th',

44

:attributes => {:class => /filename/},

44

:attributes => {:class => /filename/},

45

:content => /issues_controller.rb\t\(révision 1484\)/

45

:content => /issues_controller.rb\t\(révision 1484\)/

46

assert_tag 'td',

46

assert_tag 'td',

47

:attributes => {:class => /line-code/},

47

:attributes => {:class => /line-code/},

48

:content => /Demande créée avec succès/

48

:content => /Demande créée avec succès/

49

end

49

end

50

set_tmp_attachments_directory

50

set_tmp_attachments_directory

51

end

51

end

52

53

def test_show_diff_replace_cannot_convert_content

53

def test_show_diff_replace_cannot_convert_content

54

with_settings :repositories_encodings => 'UTF-8' do

54

with_settings :repositories_encodings => 'UTF-8' do

55

['inline', 'sbs'].each do |dt|

55

['inline', 'sbs'].each do |dt|

56

# 060719210727_changeset_iso8859-1.diff

56

# 060719210727_changeset_iso8859-1.diff

57

get :show, :id => 5, :type => dt

57

get :show, :id => 5, :type => dt

58

assert_response :success

58

assert_response :success

59

assert_template 'diff'

59

assert_template 'diff'

60

assert_equal 'text/html', @response.content_type

60

assert_equal 'text/html', @response.content_type

61

assert_tag 'th',

61

assert_tag 'th',

62

:attributes => {:class => "filename"},

62

:attributes => {:class => "filename"},

63

:content => /issues_controller.rb\t\(r\?vision 1484\)/

63

:content => /issues_controller.rb\t\(r\?vision 1484\)/

64

assert_tag 'td',

64

assert_tag 'td',

65

:attributes => {:class => /line-code/},

65

:attributes => {:class => /line-code/},

66

:content => /Demande cr\?\?e avec succ\?s/

66

:content => /Demande cr\?\?e avec succ\?s/

67

end

67

end

68

end

68

end

69

set_tmp_attachments_directory

69

set_tmp_attachments_directory

70

end

70

end

71

72

def test_show_diff_latin_1

72

def test_show_diff_latin_1

73

with_settings :repositories_encodings => 'UTF-8,ISO-8859-1' do

73

with_settings :repositories_encodings => 'UTF-8,ISO-8859-1' do

74

['inline', 'sbs'].each do |dt|

74

['inline', 'sbs'].each do |dt|

75

# 060719210727_changeset_iso8859-1.diff

75

# 060719210727_changeset_iso8859-1.diff

76

get :show, :id => 5, :type => dt

76

get :show, :id => 5, :type => dt

77

assert_response :success

77

assert_response :success

78

assert_template 'diff'

78

assert_template 'diff'

79

assert_equal 'text/html', @response.content_type

79

assert_equal 'text/html', @response.content_type

80

assert_tag 'th',

80

assert_tag 'th',

81

:attributes => {:class => "filename"},

81

:attributes => {:class => "filename"},

82

:content => /issues_controller.rb\t\(révision 1484\)/

82

:content => /issues_controller.rb\t\(révision 1484\)/

83

assert_tag 'td',

83

assert_tag 'td',

84

:attributes => {:class => /line-code/},

84

:attributes => {:class => /line-code/},

85

:content => /Demande créée avec succès/

85

:content => /Demande créée avec succès/

86

end

86

end

87

end

87

end

88

set_tmp_attachments_directory

88

set_tmp_attachments_directory

89

end

89

end

90

91

def test_save_diff_type

91

def test_save_diff_type

92

user1 = User.find(1)

92

user1 = User.find(1)

93

user1.pref[:diff_type] = nil

93

user1.pref[:diff_type] = nil

94

user1.preference.save

94

user1.preference.save

95

user = User.find(1)

95

user = User.find(1)

96

assert_nil user.pref[:diff_type]

96

assert_nil user.pref[:diff_type]

97

98

@request.session[:user_id] = 1 # admin

98

@request.session[:user_id] = 1 # admin

99

get :show, :id => 5

99

get :show, :id => 5

100

assert_response :success

100

assert_response :success

101

assert_template 'diff'

101

assert_template 'diff'

102

user.reload

102

user.reload

103

assert_equal "inline", user.pref[:diff_type]

103

assert_equal "inline", user.pref[:diff_type]

104

get :show, :id => 5, :type => 'sbs'

104

get :show, :id => 5, :type => 'sbs'

105

assert_response :success

105

assert_response :success

106

assert_template 'diff'

106

assert_template 'diff'

107

user.reload

107

user.reload

108

assert_equal "sbs", user.pref[:diff_type]

108

assert_equal "sbs", user.pref[:diff_type]

109

end

109

end

110

111

def test_diff_show_filename_in_mercurial_export

111

def test_diff_show_filename_in_mercurial_export

112

set_tmp_attachments_directory

112

set_tmp_attachments_directory

113

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

113

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

114

:file => uploaded_test_file("hg-export.diff", "text/plain"),

114

:file => uploaded_test_file("hg-export.diff", "text/plain"),

115

:author => User.find(1))

115

:author => User.find(1))

116

assert a.save

116

assert a.save

117

assert_equal 'hg-export.diff', a.filename

117

assert_equal 'hg-export.diff', a.filename

118

119

get :show, :id => a.id, :type => 'inline'

119

get :show, :id => a.id, :type => 'inline'

120

assert_response :success

120

assert_response :success

121

assert_template 'diff'

121

assert_template 'diff'

122

assert_equal 'text/html', @response.content_type

122

assert_equal 'text/html', @response.content_type

123

assert_select 'th.filename', :text => 'test1.txt'

123

assert_select 'th.filename', :text => 'test1.txt'

124

end

124

end

125

126

def test_show_text_file

126

def test_show_text_file

127

get :show, :id => 4

127

get :show, :id => 4

128

assert_response :success

128

assert_response :success

129

assert_template 'file'

129

assert_template 'file'

130

assert_equal 'text/html', @response.content_type

130

assert_equal 'text/html', @response.content_type

131

set_tmp_attachments_directory

131

set_tmp_attachments_directory

132

end

132

end

133

134

def test_show_text_file_utf_8

134

def test_show_text_file_utf_8

135

set_tmp_attachments_directory

135

set_tmp_attachments_directory

136

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

136

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

137

:file => uploaded_test_file("japanese-utf-8.txt", "text/plain"),

137

:file => uploaded_test_file("japanese-utf-8.txt", "text/plain"),

138

:author => User.find(1))

138

:author => User.find(1))

139

assert a.save

139

assert a.save

140

assert_equal 'japanese-utf-8.txt', a.filename

140

assert_equal 'japanese-utf-8.txt', a.filename

141

142

str_japanese = "\xe6\x97\xa5\xe6\x9c\xac\xe8\xaa\x9e"

142

str_japanese = "\xe6\x97\xa5\xe6\x9c\xac\xe8\xaa\x9e"

143

str_japanese.force_encoding('UTF-8') if str_japanese.respond_to?(:force_encoding)

143

str_japanese.force_encoding('UTF-8') if str_japanese.respond_to?(:force_encoding)

144

145

get :show, :id => a.id

145

get :show, :id => a.id

146

assert_response :success

146

assert_response :success

147

assert_template 'file'

147

assert_template 'file'

148

assert_equal 'text/html', @response.content_type

148

assert_equal 'text/html', @response.content_type

149

assert_tag :tag => 'th',

149

assert_tag :tag => 'th',

150

:content => '1',

150

:content => '1',

151

:attributes => { :class => 'line-num' },

151

:attributes => { :class => 'line-num' },

152

:sibling => { :tag => 'td', :content => /#{str_japanese}/ }

152

:sibling => { :tag => 'td', :content => /#{str_japanese}/ }

153

end

153

end

154

155

def test_show_text_file_replace_cannot_convert_content

155

def test_show_text_file_replace_cannot_convert_content

156

set_tmp_attachments_directory

156

set_tmp_attachments_directory

157

with_settings :repositories_encodings => 'UTF-8' do

157

with_settings :repositories_encodings => 'UTF-8' do

158

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

158

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

159

:file => uploaded_test_file("iso8859-1.txt", "text/plain"),

159

:file => uploaded_test_file("iso8859-1.txt", "text/plain"),

160

:author => User.find(1))

160

:author => User.find(1))

161

assert a.save

161

assert a.save

162

assert_equal 'iso8859-1.txt', a.filename

162

assert_equal 'iso8859-1.txt', a.filename

163

164

get :show, :id => a.id

164

get :show, :id => a.id

165

assert_response :success

165

assert_response :success

166

assert_template 'file'

166

assert_template 'file'

167

assert_equal 'text/html', @response.content_type

167

assert_equal 'text/html', @response.content_type

168

assert_tag :tag => 'th',

168

assert_tag :tag => 'th',

169

:content => '7',

169

:content => '7',

170

:attributes => { :class => 'line-num' },

170

:attributes => { :class => 'line-num' },

171

:sibling => { :tag => 'td', :content => /Demande cr\?\?e avec succ\?s/ }

171

:sibling => { :tag => 'td', :content => /Demande cr\?\?e avec succ\?s/ }

172

end

172

end

173

end

173

end

174

175

def test_show_text_file_latin_1

175

def test_show_text_file_latin_1

176

set_tmp_attachments_directory

176

set_tmp_attachments_directory

177

with_settings :repositories_encodings => 'UTF-8,ISO-8859-1' do

177

with_settings :repositories_encodings => 'UTF-8,ISO-8859-1' do

178

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

178

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

179

:file => uploaded_test_file("iso8859-1.txt", "text/plain"),

179

:file => uploaded_test_file("iso8859-1.txt", "text/plain"),

180

:author => User.find(1))

180

:author => User.find(1))

181

assert a.save

181

assert a.save

182

assert_equal 'iso8859-1.txt', a.filename

182

assert_equal 'iso8859-1.txt', a.filename

183

184

get :show, :id => a.id

184

get :show, :id => a.id

185

assert_response :success

185

assert_response :success

186

assert_template 'file'

186

assert_template 'file'

187

assert_equal 'text/html', @response.content_type

187

assert_equal 'text/html', @response.content_type

188

assert_tag :tag => 'th',

188

assert_tag :tag => 'th',

189

:content => '7',

189

:content => '7',

190

:attributes => { :class => 'line-num' },

190

:attributes => { :class => 'line-num' },

191

:sibling => { :tag => 'td', :content => /Demande créée avec succès/ }

191

:sibling => { :tag => 'td', :content => /Demande créée avec succès/ }

192

end

192

end

193

end

193

end

194

195

def test_show_text_file_should_send_if_too_big

195

def test_show_text_file_should_send_if_too_big

196

with_settings :file_max_size_displayed => 512 do

196

with_settings :file_max_size_displayed => 512 do

197

Attachment.find(4).update_attribute :filesize, 754.kilobyte

197

Attachment.find(4).update_attribute :filesize, 754.kilobyte

198

get :show, :id => 4

198

get :show, :id => 4

199

assert_response :success

199

assert_response :success

200

assert_equal 'application/x-ruby', @response.content_type

200

assert_equal 'application/x-ruby', @response.content_type

201

end

201

end

202

set_tmp_attachments_directory

202

set_tmp_attachments_directory

203

end

203

end

204

205

def test_show_other

205

def test_show_other

206

get :show, :id => 6

206

get :show, :id => 6

207

assert_response :success

207

assert_response :success

208

assert_equal 'application/octet-stream', @response.content_type

208

assert_equal 'application/octet-stream', @response.content_type

209

set_tmp_attachments_directory

209

set_tmp_attachments_directory

210

end

210

end

211

212

def test_show_file_from_private_issue_without_permission

212

def test_show_file_from_private_issue_without_permission

213

get :show, :id => 15

213

get :show, :id => 15

214

assert_redirected_to '/login?back_url=http%3A%2F%2Ftest.host%2Fattachments%2F15'

214

assert_redirected_to '/login?back_url=http%3A%2F%2Ftest.host%2Fattachments%2F15'

215

set_tmp_attachments_directory

215

set_tmp_attachments_directory

216

end

216

end

217

218

def test_show_file_from_private_issue_with_permission

218

def test_show_file_from_private_issue_with_permission

219

@request.session[:user_id] = 2

219

@request.session[:user_id] = 2

220

get :show, :id => 15

220

get :show, :id => 15

221

assert_response :success

221

assert_response :success

222

assert_tag 'h2', :content => /private.diff/

222

assert_tag 'h2', :content => /private.diff/

223

set_tmp_attachments_directory

223

set_tmp_attachments_directory

224

end

224

end

225

226

def test_show_file_without_container_should_be_allowed_to_author

226

def test_show_file_without_container_should_be_allowed_to_author

227

set_tmp_attachments_directory

227

set_tmp_attachments_directory

228

attachment = Attachment.create!(:file => uploaded_test_file("testfile.txt", "text/plain"), :author_id => 2)

228

attachment = Attachment.create!(:file => uploaded_test_file("testfile.txt", "text/plain"), :author_id => 2)

229

230

@request.session[:user_id] = 2

230

@request.session[:user_id] = 2

231

get :show, :id => attachment.id

231

get :show, :id => attachment.id

232

assert_response 200

232

assert_response 200

233

end

233

end

234

235

def test_show_file_without_container_should_be_denied_to_other_users

235

def test_show_file_without_container_should_be_denied_to_other_users

236

set_tmp_attachments_directory

236

set_tmp_attachments_directory

237

attachment = Attachment.create!(:file => uploaded_test_file("testfile.txt", "text/plain"), :author_id => 2)

237

attachment = Attachment.create!(:file => uploaded_test_file("testfile.txt", "text/plain"), :author_id => 2)

238

239

@request.session[:user_id] = 3

239

@request.session[:user_id] = 3

240

get :show, :id => attachment.id

240

get :show, :id => attachment.id

241

assert_response 403

241

assert_response 403

242

end

242

end

243

244

def test_show_invalid_should_respond_with_404

244

def test_show_invalid_should_respond_with_404

245

get :show, :id => 999

245

get :show, :id => 999

246

assert_response 404

246

assert_response 404

247

end

247

end

248

249

def test_download_text_file

249

def test_download_text_file

250

get :download, :id => 4

250

get :download, :id => 4

251

assert_response :success

251

assert_response :success

252

assert_equal 'application/x-ruby', @response.content_type

252

assert_equal 'application/x-ruby', @response.content_type

253

etag = @response.etag

253

etag = @response.etag

254

assert_not_nil etag

254

assert_not_nil etag

255

256

@request.env["HTTP_IF_NONE_MATCH"] = etag

256

@request.env["HTTP_IF_NONE_MATCH"] = etag

257

get :download, :id => 4

257

get :download, :id => 4

258

assert_response 304

258

assert_response 304

259

260

set_tmp_attachments_directory

260

set_tmp_attachments_directory

261

end

261

end

262

263

def test_download_version_file_with_issue_tracking_disabled

263

def test_download_version_file_with_issue_tracking_disabled

264

Project.find(1).disable_module! :issue_tracking

264

Project.find(1).disable_module! :issue_tracking

265

get :download, :id => 9

265

get :download, :id => 9

266

assert_response :success

266

assert_response :success

267

end

267

end

268

269

def test_download_should_assign_content_type_if_blank

269

def test_download_should_assign_content_type_if_blank

270

Attachment.find(4).update_attribute(:content_type, '')

270

Attachment.find(4).update_attribute(:content_type, '')

271

272

get :download, :id => 4

272

get :download, :id => 4

273

assert_response :success

273

assert_response :success

274

assert_equal 'text/x-ruby', @response.content_type

274

assert_equal 'text/x-ruby', @response.content_type

275

set_tmp_attachments_directory

275

set_tmp_attachments_directory

276

end

276

end

277

278

def test_download_missing_file

278

def test_download_missing_file

279

get :download, :id => 2

279

get :download, :id => 2

280

assert_response 404

280

assert_response 404

281

set_tmp_attachments_directory

281

set_tmp_attachments_directory

282

end

282

end

283

284

def test_download_should_be_denied_without_permission

284

def test_download_should_be_denied_without_permission

285

get :download, :id => 7

285

get :download, :id => 7

286

assert_redirected_to '/login?back_url=http%3A%2F%2Ftest.host%2Fattachments%2Fdownload%2F7'

286

assert_redirected_to '/login?back_url=http%3A%2F%2Ftest.host%2Fattachments%2Fdownload%2F7'

287

set_tmp_attachments_directory

287

set_tmp_attachments_directory

288

end

288

end

289

290

if convert_installed?

290

if convert_installed?

291

def test_thumbnail

291

def test_thumbnail

292

Attachment.clear_thumbnails

292

Attachment.clear_thumbnails

293

@request.session[:user_id] = 2

293

@request.session[:user_id] = 2

294

295

get :thumbnail, :id => 16

294

get :thumbnail, :id => 16

296

assert_response :success

295

assert_response :success

297

assert_equal 'image/png', response.content_type

296

assert_equal 'image/png', response.content_type

297

298

etag = @response.etag

299

assert_not_nil etag

300

301

@request.env["HTTP_IF_NONE_MATCH"] = etag

302

get :thumbnail, :id => 16

303

assert_response 304

298

end

304

end

299

305

300

def test_thumbnail_should_not_exceed_maximum_size

306

def test_thumbnail_should_not_exceed_maximum_size

301

Redmine::Thumbnail.expects(:generate).with {|source, target, size| size == 800}

307

Redmine::Thumbnail.expects(:generate).with {|source, target, size| size == 800}

302

308

303

@request.session[:user_id] = 2

309

@request.session[:user_id] = 2

304

get :thumbnail, :id => 16, :size => 2000

310

get :thumbnail, :id => 16, :size => 2000

305

end

311

end

306

312

307

def test_thumbnail_should_round_size

313

def test_thumbnail_should_round_size

308

Redmine::Thumbnail.expects(:generate).with {|source, target, size| size == 250}

314

Redmine::Thumbnail.expects(:generate).with {|source, target, size| size == 250}

309

315

310

@request.session[:user_id] = 2

316

@request.session[:user_id] = 2

311

get :thumbnail, :id => 16, :size => 260

317

get :thumbnail, :id => 16, :size => 260

312

end

318

end

313

319

314

def test_thumbnail_should_return_404_for_non_image_attachment

320

def test_thumbnail_should_return_404_for_non_image_attachment

315

@request.session[:user_id] = 2

321

@request.session[:user_id] = 2

316

322

317

get :thumbnail, :id => 15

323

get :thumbnail, :id => 15

318

assert_response 404

324

assert_response 404

319

end

325

end

320

326

321

def test_thumbnail_should_return_404_if_thumbnail_generation_failed

327

def test_thumbnail_should_return_404_if_thumbnail_generation_failed

322

Attachment.any_instance.stubs(:thumbnail).returns(nil)

328

Attachment.any_instance.stubs(:thumbnail).returns(nil)

323

@request.session[:user_id] = 2

329

@request.session[:user_id] = 2

324

330

325

get :thumbnail, :id => 16

331

get :thumbnail, :id => 16

326

assert_response 404

332

assert_response 404

327

end

333

end

328

334

329

def test_thumbnail_should_be_denied_without_permission

335

def test_thumbnail_should_be_denied_without_permission

330

get :thumbnail, :id => 16

336

get :thumbnail, :id => 16

331

assert_redirected_to '/login?back_url=http%3A%2F%2Ftest.host%2Fattachments%2Fthumbnail%2F16'

337

assert_redirected_to '/login?back_url=http%3A%2F%2Ftest.host%2Fattachments%2Fthumbnail%2F16'

332

end

338

end

333

else

339

else

334

puts '(ImageMagick convert not available)'

340

puts '(ImageMagick convert not available)'

335

end

341

end

336

342

337

def test_destroy_issue_attachment

343

def test_destroy_issue_attachment

338

set_tmp_attachments_directory

344

set_tmp_attachments_directory

339

issue = Issue.find(3)

345

issue = Issue.find(3)

340

@request.session[:user_id] = 2

346

@request.session[:user_id] = 2

341

347

342

assert_difference 'issue.attachments.count', -1 do

348

assert_difference 'issue.attachments.count', -1 do

343

assert_difference 'Journal.count' do

349

assert_difference 'Journal.count' do

344

delete :destroy, :id => 1

350

delete :destroy, :id => 1

345

assert_redirected_to '/projects/ecookbook'

351

assert_redirected_to '/projects/ecookbook'

346

end

352

end

347

end

353

end

348

assert_nil Attachment.find_by_id(1)

354

assert_nil Attachment.find_by_id(1)

349

j = Journal.order('id DESC').first

355

j = Journal.order('id DESC').first

350

assert_equal issue, j.journalized

356

assert_equal issue, j.journalized

351

assert_equal 'attachment', j.details.first.property

357

assert_equal 'attachment', j.details.first.property

352

assert_equal '1', j.details.first.prop_key

358

assert_equal '1', j.details.first.prop_key

353

assert_equal 'error281.txt', j.details.first.old_value

359

assert_equal 'error281.txt', j.details.first.old_value

354

assert_equal User.find(2), j.user

360

assert_equal User.find(2), j.user

355

end

361

end

356

362

357

def test_destroy_wiki_page_attachment

363

def test_destroy_wiki_page_attachment

358

set_tmp_attachments_directory

364

set_tmp_attachments_directory

359

@request.session[:user_id] = 2

365

@request.session[:user_id] = 2

360

assert_difference 'Attachment.count', -1 do

366

assert_difference 'Attachment.count', -1 do

361

delete :destroy, :id => 3

367

delete :destroy, :id => 3

362

assert_response 302

368

assert_response 302

363

end

369

end

364

end

370

end

365

371

366

def test_destroy_project_attachment

372

def test_destroy_project_attachment

367

set_tmp_attachments_directory

373

set_tmp_attachments_directory

368

@request.session[:user_id] = 2

374

@request.session[:user_id] = 2

369

assert_difference 'Attachment.count', -1 do

375

assert_difference 'Attachment.count', -1 do

370

delete :destroy, :id => 8

376

delete :destroy, :id => 8

371

assert_response 302

377

assert_response 302

372

end

378

end

373

end

379

end

374

380

375

def test_destroy_version_attachment

381

def test_destroy_version_attachment

376

set_tmp_attachments_directory

382

set_tmp_attachments_directory

377

@request.session[:user_id] = 2

383

@request.session[:user_id] = 2

378

assert_difference 'Attachment.count', -1 do

384

assert_difference 'Attachment.count', -1 do

379

delete :destroy, :id => 9

385

delete :destroy, :id => 9

380

assert_response 302

386

assert_response 302

381

end

387

end

382

end

388

end

383

389

384

def test_destroy_without_permission

390

def test_destroy_without_permission

385

set_tmp_attachments_directory

391

set_tmp_attachments_directory

386

assert_no_difference 'Attachment.count' do

392

assert_no_difference 'Attachment.count' do

387

delete :destroy, :id => 3

393

delete :destroy, :id => 3

388

end

394

end

389

assert_response 302

395

assert_response 302

390

assert Attachment.find_by_id(3)

396

assert Attachment.find_by_id(3)

391

end

397

end

392

end

398

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-2014  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 AttachmentsControllerTest < ActionController::TestCase
               fixtures :users, :projects, :roles, :members, :member_roles,
                        :enabled_modules, :issues, :trackers, :attachments,
                        :versions, :wiki_pages, :wikis, :documents
               def setup
                 User.current = nil
                 set_fixtures_attachments_directory
               end
               def teardown
                 set_tmp_attachments_directory
               end
               def test_show_diff
                 ['inline', 'sbs'].each do |dt|
                   # 060719210727_changeset_utf8.diff
                   get :show, :id => 14, :type => dt
                   assert_response :success
                   assert_template 'diff'
                   assert_equal 'text/html', @response.content_type
                   assert_tag 'th',
                     :attributes => {:class => /filename/},
                     :content => /issues_controller.rb\t\(révision 1484\)/
                   assert_tag 'td',
                     :attributes => {:class => /line-code/},
                     :content => /Demande créée avec succès/
                 end
                 set_tmp_attachments_directory
               end
               def test_show_diff_replace_cannot_convert_content
                 with_settings :repositories_encodings => 'UTF-8' do
                   ['inline', 'sbs'].each do |dt|
                     # 060719210727_changeset_iso8859-1.diff
                     get :show, :id => 5, :type => dt
                     assert_response :success
                     assert_template 'diff'
                     assert_equal 'text/html', @response.content_type
                     assert_tag 'th',
                       :attributes => {:class => "filename"},
                       :content => /issues_controller.rb\t\(r\?vision 1484\)/
                     assert_tag 'td',
                       :attributes => {:class => /line-code/},
                       :content => /Demande cr\?\?e avec succ\?s/
                   end
                 end
                 set_tmp_attachments_directory
               end
               def test_show_diff_latin_1
                 with_settings :repositories_encodings => 'UTF-8,ISO-8859-1' do
                   ['inline', 'sbs'].each do |dt|
                     # 060719210727_changeset_iso8859-1.diff
                     get :show, :id => 5, :type => dt
                     assert_response :success
                     assert_template 'diff'
                     assert_equal 'text/html', @response.content_type
                     assert_tag 'th',
                       :attributes => {:class => "filename"},
                       :content => /issues_controller.rb\t\(révision 1484\)/
                     assert_tag 'td',
                       :attributes => {:class => /line-code/},
                       :content => /Demande créée avec succès/
                   end
                 end
                 set_tmp_attachments_directory
               end
               def test_save_diff_type
                 user1 = User.find(1)
                 user1.pref[:diff_type] = nil
                 user1.preference.save
                 user = User.find(1)
                 assert_nil user.pref[:diff_type]
                 @request.session[:user_id] = 1 # admin
                 get :show, :id => 5
                 assert_response :success
                 assert_template 'diff'
                 user.reload
                 assert_equal "inline", user.pref[:diff_type]
                 get :show, :id => 5, :type => 'sbs'
                 assert_response :success
                 assert_template 'diff'
                 user.reload
                 assert_equal "sbs", user.pref[:diff_type]
               end
               def test_diff_show_filename_in_mercurial_export
                 set_tmp_attachments_directory
                 a = Attachment.new(:container => Issue.find(1),
                                    :file => uploaded_test_file("hg-export.diff", "text/plain"),
                                    :author => User.find(1))
                 assert a.save
                 assert_equal 'hg-export.diff', a.filename
                 get :show, :id => a.id, :type => 'inline'
                 assert_response :success
                 assert_template 'diff'
                 assert_equal 'text/html', @response.content_type
                 assert_select 'th.filename', :text => 'test1.txt'
               end
               def test_show_text_file
                 get :show, :id => 4
                 assert_response :success
                 assert_template 'file'
                 assert_equal 'text/html', @response.content_type
                 set_tmp_attachments_directory
               end
               def test_show_text_file_utf_8
                 set_tmp_attachments_directory
                 a = Attachment.new(:container => Issue.find(1),
                                    :file => uploaded_test_file("japanese-utf-8.txt", "text/plain"),
                                    :author => User.find(1))
                 assert a.save
                 assert_equal 'japanese-utf-8.txt', a.filename
                 str_japanese = "\xe6\x97\xa5\xe6\x9c\xac\xe8\xaa\x9e"
                 str_japanese.force_encoding('UTF-8') if str_japanese.respond_to?(:force_encoding)
                 get :show, :id => a.id
                 assert_response :success
                 assert_template 'file'
                 assert_equal 'text/html', @response.content_type
                 assert_tag :tag => 'th',
                            :content => '1',
                            :attributes => { :class => 'line-num' },
                            :sibling => { :tag => 'td', :content => /#{str_japanese}/ }
               end
               def test_show_text_file_replace_cannot_convert_content
                 set_tmp_attachments_directory
                 with_settings :repositories_encodings => 'UTF-8' do
                   a = Attachment.new(:container => Issue.find(1),
                                      :file => uploaded_test_file("iso8859-1.txt", "text/plain"),
                                      :author => User.find(1))
                   assert a.save
                   assert_equal 'iso8859-1.txt', a.filename
                   get :show, :id => a.id
                   assert_response :success
                   assert_template 'file'
                   assert_equal 'text/html', @response.content_type
                   assert_tag :tag => 'th',
                              :content => '7',
                              :attributes => { :class => 'line-num' },
                              :sibling => { :tag => 'td', :content => /Demande cr\?\?e avec succ\?s/ }
                 end
               end
               def test_show_text_file_latin_1
                 set_tmp_attachments_directory
                 with_settings :repositories_encodings => 'UTF-8,ISO-8859-1' do
                   a = Attachment.new(:container => Issue.find(1),
                                      :file => uploaded_test_file("iso8859-1.txt", "text/plain"),
                                      :author => User.find(1))
                   assert a.save
                   assert_equal 'iso8859-1.txt', a.filename
                   get :show, :id => a.id
                   assert_response :success
                   assert_template 'file'
                   assert_equal 'text/html', @response.content_type
                   assert_tag :tag => 'th',
                              :content => '7',
                              :attributes => { :class => 'line-num' },
                              :sibling => { :tag => 'td', :content => /Demande créée avec succès/ }
                   end
               end
               def test_show_text_file_should_send_if_too_big
                 with_settings :file_max_size_displayed => 512 do
                   Attachment.find(4).update_attribute :filesize, 754.kilobyte
                   get :show, :id => 4
                   assert_response :success
                   assert_equal 'application/x-ruby', @response.content_type
                 end
                 set_tmp_attachments_directory
               end
               def test_show_other
                 get :show, :id => 6
                 assert_response :success
                 assert_equal 'application/octet-stream', @response.content_type
                 set_tmp_attachments_directory
               end
               def test_show_file_from_private_issue_without_permission
                 get :show, :id => 15
                 assert_redirected_to '/login?back_url=http%3A%2F%2Ftest.host%2Fattachments%2F15'
                 set_tmp_attachments_directory
               end
               def test_show_file_from_private_issue_with_permission
                 @request.session[:user_id] = 2
                 get :show, :id => 15
                 assert_response :success
                 assert_tag 'h2', :content => /private.diff/
                 set_tmp_attachments_directory
               end
               def test_show_file_without_container_should_be_allowed_to_author
                 set_tmp_attachments_directory
                 attachment = Attachment.create!(:file => uploaded_test_file("testfile.txt", "text/plain"), :author_id => 2)
                 @request.session[:user_id] = 2
                 get :show, :id => attachment.id
                 assert_response 200
               end
               def test_show_file_without_container_should_be_denied_to_other_users
                 set_tmp_attachments_directory
                 attachment = Attachment.create!(:file => uploaded_test_file("testfile.txt", "text/plain"), :author_id => 2)
                 @request.session[:user_id] = 3
                 get :show, :id => attachment.id
                 assert_response 403
               end
               def test_show_invalid_should_respond_with_404
                 get :show, :id => 999
                 assert_response 404
               end
               def test_download_text_file
                 get :download, :id => 4
                 assert_response :success
                 assert_equal 'application/x-ruby', @response.content_type
                 etag = @response.etag
                 assert_not_nil etag
                 @request.env["HTTP_IF_NONE_MATCH"] = etag
                 get :download, :id => 4
                 assert_response 304
                 set_tmp_attachments_directory
               end
               def test_download_version_file_with_issue_tracking_disabled
                 Project.find(1).disable_module! :issue_tracking
                 get :download, :id => 9
                 assert_response :success
               end
               def test_download_should_assign_content_type_if_blank
                 Attachment.find(4).update_attribute(:content_type, '')
                 get :download, :id => 4
                 assert_response :success
                 assert_equal 'text/x-ruby', @response.content_type
                 set_tmp_attachments_directory
               end
               def test_download_missing_file
                 get :download, :id => 2
                 assert_response 404
                 set_tmp_attachments_directory
               end
               def test_download_should_be_denied_without_permission
                 get :download, :id => 7
                 assert_redirected_to '/login?back_url=http%3A%2F%2Ftest.host%2Fattachments%2Fdownload%2F7'
                 set_tmp_attachments_directory
               end
               if convert_installed?
                 def test_thumbnail
                   Attachment.clear_thumbnails
                   @request.session[:user_id] = 2
                   get :thumbnail, :id => 16
                   assert_response :success
                   assert_equal 'image/png', response.content_type
+                  etag = @response.etag
+                  assert_not_nil etag
+                  @request.env["HTTP_IF_NONE_MATCH"] = etag
+                  get :thumbnail, :id => 16
+                  assert_response 304
                 end
                 def test_thumbnail_should_not_exceed_maximum_size
                   Redmine::Thumbnail.expects(:generate).with {|source, target, size| size == 800}
                   @request.session[:user_id] = 2
                   get :thumbnail, :id => 16, :size => 2000
                 end
                 def test_thumbnail_should_round_size
                   Redmine::Thumbnail.expects(:generate).with {|source, target, size| size == 250}
                   @request.session[:user_id] = 2
                   get :thumbnail, :id => 16, :size => 260
                 end
                 def test_thumbnail_should_return_404_for_non_image_attachment
                   @request.session[:user_id] = 2
                   get :thumbnail, :id => 15
                   assert_response 404
                 end
                 def test_thumbnail_should_return_404_if_thumbnail_generation_failed
                   Attachment.any_instance.stubs(:thumbnail).returns(nil)
                   @request.session[:user_id] = 2
                   get :thumbnail, :id => 16
                   assert_response 404
                 end
                 def test_thumbnail_should_be_denied_without_permission
                   get :thumbnail, :id => 16
                   assert_redirected_to '/login?back_url=http%3A%2F%2Ftest.host%2Fattachments%2Fthumbnail%2F16'
                 end
               else
                 puts '(ImageMagick convert not available)'
               end
               def test_destroy_issue_attachment
                 set_tmp_attachments_directory
                 issue = Issue.find(3)
                 @request.session[:user_id] = 2
                 assert_difference 'issue.attachments.count', -1 do
                   assert_difference 'Journal.count' do
                     delete :destroy, :id => 1
                     assert_redirected_to '/projects/ecookbook'
                   end
                 end
                 assert_nil Attachment.find_by_id(1)
                 j = Journal.order('id DESC').first
                 assert_equal issue, j.journalized
                 assert_equal 'attachment', j.details.first.property
                 assert_equal '1', j.details.first.prop_key
                 assert_equal 'error281.txt', j.details.first.old_value
                 assert_equal User.find(2), j.user
               end
               def test_destroy_wiki_page_attachment
                 set_tmp_attachments_directory
                 @request.session[:user_id] = 2
                 assert_difference 'Attachment.count', -1 do
                   delete :destroy, :id => 3
                   assert_response 302
                 end
               end
               def test_destroy_project_attachment
                 set_tmp_attachments_directory
                 @request.session[:user_id] = 2
                 assert_difference 'Attachment.count', -1 do
                   delete :destroy, :id => 8
                   assert_response 302
                 end
               end
               def test_destroy_version_attachment
                 set_tmp_attachments_directory
                 @request.session[:user_id] = 2
                 assert_difference 'Attachment.count', -1 do
                   delete :destroy, :id => 9
                   assert_response 302
                 end
               end
               def test_destroy_without_permission
                 set_tmp_attachments_directory
                 assert_no_difference 'Attachment.count' do
                   delete :destroy, :id => 3
                 end
                 assert_response 302
                 assert Attachment.find_by_id(3)
               end
             end