Skip to content
体验新版
项目
组织
正在加载...
登录
切换导航
打开侧边栏
张重言
rails
提交
00d1b132
R
rails
项目概览
张重言
/
rails
通知
1
Star
0
Fork
0
代码
文件
提交
分支
Tags
贡献者
分支图
Diff
Issue
0
列表
看板
标记
里程碑
合并请求
0
Wiki
0
Wiki
分析
仓库
DevOps
项目成员
Pages
R
rails
项目概览
项目概览
详情
发布
仓库
仓库
文件
提交
分支
标签
贡献者
分支图
比较
Issue
0
Issue
0
列表
看板
标记
里程碑
合并请求
0
合并请求
0
Pages
分析
分析
仓库分析
DevOps
Wiki
0
Wiki
成员
成员
收起侧边栏
关闭侧边栏
动态
分支图
创建新Issue
提交
Issue看板
体验新版 GitCode,发现更多精彩内容 >>
提交
00d1b132
编写于
5月 20, 2014
作者:
Y
Yves Senn
浏览文件
操作
浏览文件
下载
差异文件
Merge pull request #15178 from zuhao/refactor_actionpack_respond_with_test
Deregister csv renderer after test to prevent leak.
上级
fe06e9ac
ac36b456
变更
2
隐藏空白更改
内联
并排
Showing
2 changed file
with
35 addition
and
0 deletion
+35
-0
actionpack/lib/action_controller/metal/renderers.rb
actionpack/lib/action_controller/metal/renderers.rb
+16
-0
actionpack/test/controller/mime/respond_with_test.rb
actionpack/test/controller/mime/respond_with_test.rb
+19
-0
未找到文件。
actionpack/lib/action_controller/metal/renderers.rb
浏览文件 @
00d1b132
...
...
@@ -6,6 +6,11 @@ def self.add_renderer(key, &block)
Renderers
.
add
(
key
,
&
block
)
end
# See <tt>Renderers.remove</tt>
def
self
.
remove_renderer
(
key
)
Renderers
.
remove
(
key
)
end
class
MissingRenderer
<
LoadError
def
initialize
(
format
)
super
"No renderer defined for format:
#{
format
}
"
...
...
@@ -83,6 +88,17 @@ def self.add(key, &block)
RENDERERS
<<
key
.
to_sym
end
# This method is the opposite of add method.
#
# Usage:
#
# ActionController::Renderers.remove(:csv)
def
self
.
remove
(
key
)
RENDERERS
.
delete
(
key
.
to_sym
)
method
=
"_render_option_
#{
key
}
"
remove_method
(
method
)
if
method_defined?
(
method
)
end
module
All
extend
ActiveSupport
::
Concern
include
Renderers
...
...
actionpack/test/controller/mime/respond_with_test.rb
浏览文件 @
00d1b132
...
...
@@ -643,6 +643,8 @@ def test_uses_renderer_if_an_api_behavior
get
:index
,
format:
'csv'
assert_equal
Mime
::
CSV
,
@response
.
content_type
assert_equal
"c,s,v"
,
@response
.
body
ensure
ActionController
::
Renderers
.
remove
:csv
end
def
test_raises_missing_renderer_if_an_api_behavior_with_no_renderer
...
...
@@ -652,6 +654,23 @@ def test_raises_missing_renderer_if_an_api_behavior_with_no_renderer
end
end
def
test_removing_renderers
ActionController
::
Renderers
.
add
:csv
do
|
obj
,
options
|
send_data
obj
.
to_csv
,
type:
Mime
::
CSV
end
@controller
=
CsvRespondWithController
.
new
@request
.
accept
=
"text/csv"
get
:index
,
format:
'csv'
assert_equal
Mime
::
CSV
,
@response
.
content_type
ActionController
::
Renderers
.
remove
:csv
assert_raise
ActionController
::
MissingRenderer
do
get
:index
,
format:
'csv'
end
ensure
ActionController
::
Renderers
.
remove
:csv
end
def
test_error_is_raised_if_no_respond_to_is_declared_and_respond_with_is_called
@controller
=
EmptyRespondWithController
.
new
@request
.
accept
=
"*/*"
...
...
编辑
预览
Markdown
is supported
0%
请重试
或
添加新附件
.
添加附件
取消
You are about to add
0
people
to the discussion. Proceed with caution.
先完成此消息的编辑!
取消
想要评论请
注册
或
登录