RuboCop项目中InternalAffairs/ExampleDescription检查的插值字符串处理问题

在RuboCop项目中，InternalAffairs/ExampleDescription是一个用于检查RSpec测试用例描述的代码风格检查器。该检查器的主要目的是确保测试用例的描述能够清晰地表达测试意图，遵循一致的格式规范。

近期发现了一个关于该检查器在处理包含字符串插值的测试描述时的特殊情况。当测试用例的描述中包含Ruby的字符串插值语法（如#{foo}）时，InternalAffairs/ExampleDescription检查器会出现错误识别（false negative）的情况，即未能正确识别出不符合规范的测试描述。

问题具体表现

考虑以下RSpec测试用例示例：

it "does not register an offense if #{foo} does not use bar" do
  expect_offense(<<~RUBY)
    attribute :baz, :datetime
    ^^^^^^^^^^^^^^^^^^^^^^^^^ Lorem ipsum dolor sit amet, consectetur.
  RUBY
end

在这个例子中，测试描述包含了字符串插值#{foo}。按照InternalAffairs/ExampleDescription的规范，这种动态生成的描述通常是不推荐的，因为它降低了测试描述的可读性和可维护性。然而，当前的检查器实现未能正确识别这种情况，导致检查结果出现错误识别。

技术背景

RuboCop的InternalAffairs/ExampleDescription检查器主要用于确保：

1. 测试描述清晰表达测试意图
2. 避免使用过于笼统或模糊的描述
3. 保持描述风格一致
4. 避免使用动态生成的描述（如字符串插值）

字符串插值在测试描述中的使用会带来几个问题：

1. 可读性降低：阅读测试报告时无法直接看到完整的描述
2. 可维护性差：当插值变量变化时，描述含义可能改变
3. 调试困难：当测试失败时，难以快速定位问题

解决方案

RuboCop团队已经通过提交修复了这个问题。修复方案主要包括：

1. 增强AST解析能力，能够识别字符串插值节点
2. 对包含插值的测试描述给出适当的警告
3. 提供自动修正建议，将动态描述改为静态描述

最佳实践建议

基于此问题的修复，建议开发者在编写RSpec测试时：

1. 尽量使用静态的、明确的测试描述
2. 避免在描述中使用字符串插值
3. 如果确实需要动态描述，考虑使用明确的上下文说明
4. 保持描述简洁但信息完整

例如，将上面的例子改写为：

it "does not register an offense when attribute does not use bar" do
  expect_offense(<<~RUBY)
    attribute :baz, :datetime
    ^^^^^^^^^^^^^^^^^^^^^^^^^ Lorem ipsum dolor sit amet, consectetur.
  RUBY
end

或者当需要区分不同情况时：

context "with #{foo}" do
  it "does not register an offense when attribute does not use bar" do
    expect_offense(<<~RUBY)
      attribute :baz, :datetime
      ^^^^^^^^^^^^^^^^^^^^^^^^^ Lorem ipsum dolor sit amet, consectetur.
    RUBY
  end
end

这种写法既保持了描述的清晰性，又能够表达测试的不同上下文。