Scalatra测试框架中HTTP请求内容类型处理的改进

在Scalatra测试框架中，HttpComponentsClient类负责模拟HTTP请求以进行集成测试。近期发现该组件在处理请求体时存在一个值得注意的问题：无论实际请求头如何设置，它总是会为POST、PUT和PATCH请求设置默认的text/plain内容类型。

问题分析

HttpComponentsClient类中的attachBody方法负责为HTTP请求附加请求体。该方法逻辑如下：

1. 对于POST、PUT和PATCH请求，它会检查请求是否已包含Content-Type头
2. 如果不存在Content-Type头，则默认使用TEXT_PLAIN类型
3. 使用ByteArrayEntity封装请求体

问题的关键在于请求头的设置顺序。在创建请求时，流程是：先创建请求对象，然后附加请求体，最后才设置请求头。这意味着在attachBody方法执行时，Content-Type头尚未被设置，导致总是使用默认的text/plain类型。

影响范围

这一行为对测试产生了以下影响：

1. 无法准确测试Servlet对未指定Content-Type请求的处理逻辑
2. 强制所有无Content-Type头的请求使用text/plain类型，与真实HTTP行为不符
3. 干扰了Scalatra默认格式处理机制的测试

解决方案

正确的做法应该是：

1. 当请求未指定Content-Type时，不应设置默认值
2. 让ByteArrayEntity的contentType参数为null，以模拟真实HTTP请求行为
3. 这样Servlet就能按照其默认逻辑处理请求格式

技术实现细节

ByteArrayEntity构造函数确实允许contentType参数为null，这符合HTTP协议规范。当不指定Content-Type时，服务器端应自行决定如何处理请求体。Scalatra框架本身支持通过defaultFormat配置来处理这种情况。

改进后的行为

经过修正后：

1. 未指定Content-Type的请求将不再自动获得text/plain类型
2. Servlet可以正确应用其defaultFormat配置
3. 测试结果更符合实际生产环境行为

这一改进使得Scalatra的测试框架能够更准确地模拟真实HTTP请求行为，特别是对于RESTful API测试场景中内容协商和默认格式处理的测试。