copy_tuner_client 1.5.0 → 2.1.0

data/skills/copy-tuner-to-t-migrate/SKILL.md

@@ -0,0 +1,131 @@
+---
+name: copy-tuner-to-t-migrate
+description: >-
+  copy_tuner_client v2.0.0 への移行で、削除された独自ヘルパー tt の呼び出しを Rails 標準の t（translate）へ
+  置換するスキル。tt は PR #122 で存在理由を失い gem から削除されたため、残った tt(...) は NoMethodError に
+  なる。大半は機械的に tt( → t( で済むが、訳文を文字列加工している箇所だけは t ではなく I18n.t へ移す必要があり
+  （開発環境のマーカートークン混入バグ対策）、そこは 1 件ずつ確認する。対象は app 配下の .rb / .haml / .erb。
+disable-model-invocation: true
+license: MIT
+---
+
+# tt → t 移行スキル（copy_tuner_client v2.0.0）
+
+copy_tuner_client がかつて `ActionView` にフックして生やしていた独自ヘルパー **`tt`**（シグネチャは
+`tt(key, **options)`、引数は `t` と完全同一）を、Rails 標準の `t`（`translate`）へ置き換えるワークフロー。
+gem を v2.0.0 に上げたアプリのビューに `tt(...)` が残っていると **`NoMethodError`（未定義ヘルパー）** になる。
+
+このスキルは**破壊的な一括書き換え**を含むため、ユーザが明示的に呼んだときだけ動く（`disable-model-invocation`）。
+
+## なぜ tt を t へ置換するのか
+
+`tt` はもともと「**copyray マーカーを注入しない生の訳文を取る**」ためのヘルパーだった。旧方式では `t('key')` の
+戻り値そのものにマーカーが埋め込まれ、`truncate` / `length` 等で**訳文を文字列加工する箇所**でマーカー長が
+混入して壊れる問題があり、その対策が `tt` だった。
+
+PR #122 のマーカー方式刷新で、マーカー注入は戻り値ではなく middleware（`CopyrayMiddleware` → `Rewriter`）で
+行い HTML 配信前に完全除去するようになった。これにより通常の `t` がどこでも安全に使えるようになり、`tt` は
+存在理由を失って削除された。背景の詳細は `skills/copy-tuner/SKILL.md` の「翻訳ヘルパー（`t`）」と
+「落とし穴: 開発環境で訳文にマーカートークンが混入する」を参照。
+
+**ただし罠が一つ残る。** マーカー除去は「最終的に HTML へ出力される訳文」に対してのみ効く。development で
+`t('key')` の戻り値を**ビュー内で文字列加工する**コードは、除去前のマーカー込み文字列に作用してしまう（本番では
+middleware 自体が無いので再現しない＝開発環境だけ壊れる）。
+
+→ つまり **`tt` を素朴に `t` へ変えてよいのは「文字列加工していない箇所」だけ**。文字列加工している `tt` を
+`t` にすると、`tt` がもともと潰していたバグをそのまま復活させてしまう。そこは middleware のラッパーを通らない
+**`I18n.t`（絶対キー）** へ移すのが正解で、機械的には決められないため 1 件ずつ確認する。
+
+## 変換の 3 分類
+
+| 分類 | 例 | 扱い |
+|---|---|---|
+| **safe** | `tt('views.foo')` / `= tt '.title'` / `tt(key, default: x)` | `tt(` → `t(` に決定論的一括変換 |
+| **suspicious** | `truncate(tt('k'))` / `tt('k').length` / `tt('k')[0..n]` / `tt('k') =~ /re/` / `tt('k').gsub(...)` 等 | `I18n.t('絶対キー')` へ。**1 件ずつ確認**。スクリプトは触らない |
+| **other** | `def tt` / `alias tt`（定義側）/ app 外（`lib/` 等）/ `tt` を含む別識別子の誤検出 | 最後にまとめて提示。自動変換しない |
+
+**重要（定義側の罠）:** アプリが v2.0.0 を待つ間の後方互換として `ApplicationHelper` に `def tt(key, **) = t(key, **)`
+のような **`tt` の定義**を生やしていることがある。これは「呼び出し」ではないので `t(` へ機械置換すると Rails の `t`
+を再定義して破滅する。スクリプトは `def tt` / `alias tt` を **other** に隔離して自動変換しない。**呼び出しを全て
+`t` / `I18n.t` へ移し終えた後**に、この定義を手で削除する（順序を逆にすると呼び出しが `NoMethodError` になる）。
+
+判定はヒューリスティック（完全な AST ではない）。**誤判定のコストが非対称**なので疑わしきは suspicious に倒す:
+safe を取りこぼす（本当は怪しいのに safe）とマーカー混入バグが再発して重大、suspicious を過剰検出しても人間が
+1 件確認するだけで軽微。
+
+## ワークフロー
+
+### 1. 利用箇所の洗い出しと 3 分類
+
+同梱スクリプトで `app/` 配下の `.rb` / `.haml` / `.erb` を走査し、3 分類でレポートする（変更は加えない）。
+Rails context は不要なので素の `ruby` で動く。
+
+```bash
+ruby skills/copy-tuner-to-t-migrate/scripts/migrate_tt.rb --report
+```
+
+safe / suspicious / other の件数と該当箇所（`ファイル:行: コード`）が出る。まず全体像をユーザと共有する。
+
+### 2. safe な箇所を一括変換
+
+safe 分類は引数がそのまま通る（`tt(key, **options)` → `t(key, **options)`）ので決定論的に置換できる。
+**件数をユーザに伝え、承認を得てから**適用する（`--apply-safe` はファイルを直接書き換えるため）。
+
+```bash
+ruby skills/copy-tuner-to-t-migrate/scripts/migrate_tt.rb --apply-safe
+```
+
+suspicious / other の行は**一切触らない**（行番号で限定しているため）。適用後に `git diff` を見せて、safe 行
+だけが `t(` になったことを確認してもらう。
+
+### 3. suspicious な箇所を 1 件ずつ確認
+
+ここが人間の判断が要る核心。手順 1 の suspicious 各件について、次を 1 件ずつ提示して確認を取る:
+
+- 該当コード（`ファイル:行`）と、なぜ怪しいか（戻り値を文字列加工している＝マーカー混入の危険）
+- 提案する置換: `t(...)` ではなく **`I18n.t('絶対キー')`**（`I18n` モジュール直呼びはラッパーを通らずマーカーが付かない）
+
+注意点:
+- **たとえユーザが「全部まとめて I18n.t にして」と言っても、suspicious は必ず 1 件ずつ提示する。** 相対キーを含む件は絶対キーをユーザと確認しないと置換できないためで、省略すると誤ったキーを書き込む危険がある。
+- **相対キー（先頭ドット `tt('.foo')`）は `I18n.t` では解決できない。** 絶対キー（`I18n.t('views.foo.bar')`）へ
+  書き換える必要があり、partial のパスからキーを補う判断が要る。スクリプトはこの相対キーの suspicious を
+  `相対キー(.foo)は絶対キーへ書き換えが必要` と強調表示するので、特に丁寧に確認する。ヘルパーや
+  コントローラ（`app/helpers/`, `app/controllers/`）に相対キーがある場合は、呼び出し元のビューのパスを
+  調べて基点キーを特定するか、ユーザに直接絶対キーを確認する。絶対キーが不明なときは
+  `grep -r "キー末尾部分" config/locales/` でロケールファイルを検索して候補を絞り込む。
+- 本当に文字列加工していない（誤検出）なら、その場合は `t(...)` でよい。ユーザの判断に従う。
+
+確認が取れた箇所だけ Edit で個別に置換する（スクリプトでは変換しない）。
+
+### 4. その他ヒット（定義側・app 外・誤検出）を提示
+
+手順 1 の other をまとめて提示する。
+
+- **`def tt` / `alias tt`（定義側）**: 後方互換シムが残っていることが多い。**呼び出しを全て移し終えてから手で削除**する
+  （3 分類表の「定義側の罠」参照）。早く消すと残った呼び出しが `NoMethodError` になる。
+- **app 外（`lib/` のヘルパーやコントローラ等）**: ビューヘルパー `tt` がそのスコープから見えるかは別問題なので、
+  自動変換せずユーザに判断を委ねる。
+
+念のため、スクリプトの単語境界判定をすり抜けた可能性に備えて素朴な grep でも最終確認する:
+
+```bash
+git grep -nwI tt -- app   # -I でバイナリ（画像等）の誤マッチを除外
+```
+
+### 5. 検証
+
+- `git grep -nwI tt -- app` の残りが「未対応の suspicious + まだ消していない定義側」だけであること
+  （safe は 0 件、`I18n.t` 化した suspicious も `tt` としては消える）。
+- 対象アプリで `bundle exec rspec`（または該当アプリのテスト）を回し、`NoMethodError` が出ないこと。
+- development で実画面を開き、文字列加工していた箇所にマーカートークン（`⟦CT:...⟧`）が混入していないこと。
+
+## スクリプトの責務（決定論的な範囲のみ）
+
+`scripts/migrate_tt.rb` は**機械的に確定できる部分だけ**を担う:
+
+- `--report`（既定）: 3 分類を出力。`--json` で機械可読出力も可能。**ファイルは変更しない**。
+- `--apply-safe`: safe 分類のみ `tt(` → `t(` を適用。suspicious / other は触らない。
+- `--root DIR`: リポジトリルート指定（既定はカレント）。
+
+suspicious の確定変換（`I18n.t` 化・相対→絶対キー）は機械的に決められないため**スクリプトは行わない**。
+必ず手順 3 で人間が 1 件ずつ確認する。

data/skills/copy-tuner-to-t-migrate/scripts/migrate_tt.rb

@@ -0,0 +1,189 @@
+# frozen_string_literal: true
+
+# copy_tuner_client v2.0.0 移行: tt → t 置換スクリプト（SKILL.md 手順 1〜2）。
+#
+# copy_tuner_client が生やしていた独自ヘルパー tt は PR #122 で削除された。gem を上げた
+# アプリのビューに tt(...) が残ると NoMethodError になるため t へ移す。大半は機械的に
+# tt( → t( で済むが、訳文を文字列加工している箇所だけは素朴に t へ変えると開発環境で
+# マーカートークン混入バグが再発する（skills/copy-tuner/SKILL.md の罠を参照）。そこは
+# I18n.t（絶対キー）へ変換すべきなので、このスクリプトは自動変換せず「怪しい」として抽出する。
+#
+# Rails context は不要なので素の ruby で動く:
+#
+#   ruby skills/copy-tuner-to-t-migrate/scripts/migrate_tt.rb --report
+#   ruby skills/copy-tuner-to-t-migrate/scripts/migrate_tt.rb --apply-safe
+#
+# 出力する 3 分類:
+#   (1) safe       : 文字列加工されていない tt 呼び出し。tt( → t( に決定論的変換できる。
+#   (2) suspicious : 戻り値が同一行で文字列加工されている tt 呼び出し。I18n.t 提案・要確認。
+#   (3) other      : app 外 / tt を含む別識別子の誤検出など。自動変換せず目視で確認。
+#
+# 判定はヒューリスティック（完全な AST 解析ではない）。誤判定のコストは
+#   safe 誤分類（怪しいのに safe）= マーカー混入バグ再発 → 重大
+#   suspicious 誤分類（安全なのに suspicious）= 人間が 1 件確認するだけ → 軽微
+# なので、疑わしきは suspicious に倒す。
+
+require 'optparse'
+
+# NOTE: 中断を呼び出し側（Bash 手順）へ確実に伝えるため warn + exit(1) を使う。
+def die(message)
+  warn message
+  exit(1)
+end
+
+# ---- 引数 ----
+options = { mode: :report, root: '.' }
+parser =
+  OptionParser.new do |opts|
+    opts.banner = 'Usage: ruby migrate_tt.rb [--report | --apply-safe] [--root DIR]'
+    opts.on('--report', '3 分類を出力する（既定）。ファイルは変更しない') { options[:mode] = :report }
+    opts.on('--apply-safe', 'safe 分類のみ tt( → t( を実ファイルに適用する') { options[:mode] = :apply_safe }
+    opts.on('--root DIR', 'リポジトリルート（既定: カレントディレクトリ）') { |v| options[:root] = v }
+    opts.on('--json', 'report を JSON で出力する（機械処理用）') { options[:json] = true }
+    opts.on('-h', '--help', 'ヘルプ') do
+      puts opts
+      exit(0)
+    end
+  end
+parser.parse!(ARGV)
+
+ROOT = File.expand_path(options[:root])
+die("ルートが見つからない: #{ROOT}") unless File.directory?(ROOT)
+
+# 対象: app 配下の .rb / .haml / .erb。それ以外のヒットは other 扱いで提示のみ。
+APP_GLOB = File.join(ROOT, 'app', '**', '*.{rb,haml,erb}')
+# app 外も走査して other に入れる（lib/ 等の取りこぼし提示用）。
+ALL_GLOB = File.join(ROOT, '**', '*.{rb,haml,erb}')
+
+# 単語境界の tt 呼び出し。tt( だけでなく haml の `= tt '...'`（括弧なし）も拾う。
+# 直前が識別子文字（attr/http/setting 等の一部や `.tt` のメソッド呼び出し）でないことを (?<![\w.]) で担保。
+TT_CALL = /(?<![\w.])tt(\s*\(|\s+['":@])/
+
+# tt の「定義」側（呼び出しではない）。アプリが後方互換で `def tt` を生やしている場合がある。
+# これを safe として t( へ機械置換すると Rails の t を再定義してしまい破滅的なので、定義は別扱いにする。
+TT_DEFINITION = /\b(?:def\s+(?:self\.)?tt\b|alias(?:_method)?\s+:?tt\b|alias(?:_method)?\s+['"]tt['"])/
+
+# tt(...) の戻り値が同一行で文字列加工されているかの判定に使う語彙。
+STRING_HELPERS = %w[truncate simple_format strip_tags highlight excerpt word_wrap].freeze
+STRING_METHODS = %w[
+  length size bytesize slice first last truncate truncate_words
+  gsub sub gsub! sub! strip lstrip rstrip chomp chop chars bytes lines
+  scan match match? =~ start_with? end_with? include? index rindex
+  upcase downcase capitalize ljust rjust center delete squeeze tr
+  to_i to_f to_sym html_safe
+].freeze
+
+# (a) truncate( ... tt( ... ) のように文字列加工ヘルパーの引数になっている
+STRING_HELPER_RE = /\b(?:#{STRING_HELPERS.join('|')})\s*\(\s*[^)]*\btt\b/
+# (b) tt(...) または tt '...' の直後に .method / =~ / [ が続く。引数の閉じ括弧をまたぐ単純近似。
+STRING_METHOD_RE = /\btt\b\s*(?:\([^\n]*?\)|['":@][^\n,]*)\s*(?:\.\s*(?:#{STRING_METHODS.join('|')})\b|=~|\[)/
+
+# 先頭ドットの相対キー tt('.foo') は I18n.t では解決できず絶対キー化が要る → 強調表示用。
+RELATIVE_KEY_RE = /\btt\b\s*(?:\(\s*)?['":]\s*\./
+
+def relative(path)
+  path.sub("#{ROOT}/", '')
+end
+
+safe = []
+suspicious = []
+other = []
+
+app_set = Dir.glob(APP_GLOB).to_set
+
+# 1 ファイル分を走査し、ヒット行を safe / suspicious / other へ振り分ける。
+def scan_file(path, in_app:)
+  rel = relative(path)
+  File.readlines(path).each_with_index do |line, idx|
+    next unless line =~ TT_CALL
+
+    entry = { file: rel, lineno: idx + 1, code: line.rstrip }
+    yield(classify_line(line, entry, in_app: in_app))
+  end
+rescue StandardError
+  nil
+end
+
+# ヒット 1 行を分類し、[:safe | :suspicious | :other, entry] を返す。
+def classify_line(line, entry, in_app:)
+  # tt の定義（def/alias）は呼び出しではない。t( へ置換すると Rails の t を壊すので絶対に自動変換しない。
+  if line.match?(TT_DEFINITION)
+    entry[:reason] = 'tt の定義（def/alias）。自動変換禁止。呼び出しを全て t へ移した後に手動で削除する'
+    return [:other, entry]
+  end
+
+  unless in_app
+    entry[:reason] = 'app 外（要目視）'
+    return [:other, entry]
+  end
+
+  return [:safe, entry] unless line.match?(STRING_HELPER_RE) || line.match?(STRING_METHOD_RE)
+
+  entry[:relative_key] = line.match?(RELATIVE_KEY_RE)
+  entry[:hint] = if entry[:relative_key]
+                   'I18n.t へ。相対キー(.foo)は絶対キーへ書き換えが必要'
+                 else
+                   'I18n.t(絶対キー) へ置換を検討'
+                 end
+  [:suspicious, entry]
+end
+
+buckets = { safe: safe, suspicious: suspicious, other: other }
+Dir.glob(ALL_GLOB).each do |path|
+  # vendor / node_modules / tmp 等のノイズを除外
+  next if relative(path).start_with?('vendor/', 'node_modules/', 'tmp/', '.git/')
+
+  scan_file(path, in_app: app_set.include?(path)) do |bucket, entry|
+    buckets[bucket] << entry
+  end
+end
+
+if options[:json]
+  require 'json'
+  puts JSON.pretty_generate(safe: safe, suspicious: suspicious, other: other)
+  exit(0)
+end
+
+def print_section(title, entries)
+  puts "\n== #{title} (#{entries.size}) =="
+  entries.each do |e|
+    suffix = if e[:hint]
+               "  # #{e[:hint]}"
+             else
+               (e[:reason] ? "  # #{e[:reason]}" : '')
+             end
+    puts "#{e[:file]}:#{e[:lineno]}: #{e[:code].strip}#{suffix}"
+  end
+end
+
+if options[:mode] == :report
+  print_section('safe（tt( → t( に決定論的変換できる）', safe)
+  print_section('suspicious（文字列加工あり・I18n.t 提案・要 1 件ずつ確認）', suspicious)
+  print_section('other（app 外・誤検出・要目視）', other)
+  puts "\n合計: safe=#{safe.size} suspicious=#{suspicious.size} other=#{other.size}"
+  exit(0)
+end
+
+# ---- --apply-safe: safe 分類のみ tt( → t( を適用 ----
+# 同一ファイル内の safe 行だけを対象に、単語境界の tt を t へ置換する。
+# suspicious / other 行は触らない（行番号で限定）。
+by_file = safe.group_by { |e| e[:file] }
+changed = 0
+by_file.each do |rel, entries|
+  path = File.join(ROOT, rel)
+  lines = File.readlines(path)
+  target_linenos = entries.to_set { |e| e[:lineno] }
+  target_linenos.each do |lineno|
+    line = lines[lineno - 1]
+    # safe 行内の tt 呼び出しのみ t へ。tt('...') や tt(... の tt を t に。
+    new_line = line.gsub(TT_CALL) { "t#{Regexp.last_match(1)}" }
+    if new_line != line
+      lines[lineno - 1] = new_line
+      changed += 1
+    end
+  end
+  File.write(path, lines.join)
+end
+
+puts "safe 置換を適用: #{changed} 箇所 / #{by_file.size} ファイル"
+puts "残った suspicious=#{suspicious.size} other=#{other.size} は手作業で対応すること（--report で再確認）"

data/spec/copy_tuner_client/cache_spec.rb

@@ -24,16 +24,6 @@ describe 'CopyTunerClient::Cache' do
     expect(cache.keys).to match_array(%w[en.test.key en.test.other_key])
   end
 
-  it 'exclude_key_regexpが設定されている場合、該当データを除外すること' do
-    cache = build_cache(exclude_key_regexp: /^en\.test\.other_key$/)
-    cache['en.test.key']       = 'expected'
-    cache['en.test.other_key'] = 'expected'
-
-    cache.download
-
-    expect(cache.queued.keys).to match_array(%w[en.test.key])
-  end
-
   it 'local_first_key_regexpにマッチするキー（locale除く）はアップロードキューに入れないこと' do
     cache = build_cache(local_first_key_regexp: /\Aviews\./)
     cache['ja.views.foo']    = 'local value'
@@ -284,6 +274,7 @@ describe 'CopyTunerClient::Cache' do
   it 'トップレベルからflushできること' do
     cache = build_cache
     CopyTunerClient.configure do |config|
+      config.project_id = 1
       config.cache = cache
     end
     expect(cache).to receive(:flush).at_least(:once)
@@ -410,6 +401,7 @@ describe 'CopyTunerClient::Cache' do
 
     it 'トップレベル定数から呼び出せること' do
       CopyTunerClient.configure do |config|
+        config.project_id = 1
         config.cache = cache
       end
       expect(cache).to receive(:export)

data/spec/copy_tuner_client/client_spec.rb

@@ -207,6 +207,7 @@ describe 'CopyTunerClient' do
     client = build_client
     allow(client).to receive(:download)
     CopyTunerClient.configure do |config|
+      config.project_id = 1
       config.client = client
     end
     expect(client).to receive(:deploy)

data/spec/copy_tuner_client/configuration_spec.rb

@@ -253,27 +253,24 @@ describe CopyTunerClient::Configuration do
     end
   end
 
-  describe '#exclude_key_regexp= (deprecated)' do
-    let(:config) { CopyTunerClient::Configuration.new }
-    let(:deprecator) { instance_double(ActiveSupport::Deprecation, warn: nil) }
-
-    before { allow(ActiveSupport::Deprecation).to receive(:new).and_return(deprecator) }
-
-    it 'warns when a value is set' do
-      expect(deprecator).to receive(:warn).with(/exclude_key_regexp is deprecated/)
-
-      config.exclude_key_regexp = /\Aja\.views\./
+  describe 'project_id の必須化' do
+    let(:config) do
+      config = CopyTunerClient::Configuration.new
+      config.api_key = 'abc123'
+      config
     end
 
-    it 'stores the assigned value' do
-      config.exclude_key_regexp = /\Aja\.views\./
-      expect(config.exclude_key_regexp).to eq(/\Aja\.views\./)
+    it 'project_id が未設定のとき apply が ArgumentError を出すこと' do
+      expect { config.apply }.to raise_error(ArgumentError, 'project_id is required')
     end
 
-    it 'does not warn when set to nil' do
-      expect(deprecator).not_to receive(:warn)
+    it 'project_id が未設定のとき project_url が ArgumentError を出すこと' do
+      expect { config.project_url }.to raise_error(ArgumentError, 'project_id is required')
+    end
 
-      config.exclude_key_regexp = nil
+    it 'project_id を設定すると project_url がそれを使った URL を返すこと' do
+      config.project_id = 77
+      expect(config.project_url).to include('/projects/77')
     end
   end
 end
@@ -294,6 +291,9 @@ shared_context 'stubbed configuration' do
     allow(CopyTunerClient::Poller).to receive(:new).and_return(poller)
     allow(CopyTunerClient::ProcessGuard).to receive(:new).and_return(process_guard)
     subject.logger = logger
+    # NOTE: apply は project_id 必須になったため、未設定だと raise する。applied 系テストは
+    #       project_id 自体を検証しないので適当な値を補っておく
+    subject.project_id ||= 1
     apply
   end
 end

data/spec/copy_tuner_client/copyray/marker_spec.rb

@@ -0,0 +1,41 @@
+require 'spec_helper'
+require 'copy_tuner_client/copyray/marker'
+
+describe CopyTunerClient::Copyray::Marker do
+  describe '.encode' do
+    it 'キーを可視トークンの区切り記号で囲む' do
+      expect(described_class.encode('views.home.index.message')).to eq '⟦CT:views.home.index.message⟧'
+    end
+
+    it 'ドット・アンダースコア・スラッシュを含むキーをそのまま保持する' do
+      expect(described_class.encode('en.foo_bar.baz/qux')).to eq '⟦CT:en.foo_bar.baz/qux⟧'
+    end
+
+    it 'マルチバイトキーをそのまま保持する' do
+      expect(described_class.encode('ja.見出し')).to eq '⟦CT:ja.見出し⟧'
+    end
+  end
+
+  describe '::SCAN_REGEXP' do
+    it 'エンコード済みトークンからキーを取り出す' do
+      key = 'views.home.index.message'
+      match = described_class::SCAN_REGEXP.match(described_class.encode(key))
+      expect(match[1]).to eq key
+    end
+
+    it 'ドット・アンダースコア・スラッシュ・マルチバイトを含むキーを取り出す' do
+      key = 'ja.foo_bar.baz/qux.見出し'
+      match = described_class::SCAN_REGEXP.match(described_class.encode(key))
+      expect(match[1]).to eq key
+    end
+
+    it '非貪欲マッチで隣接するトークンを分離する' do
+      text = "#{described_class.encode('a.b')}hello#{described_class.encode('c.d')}"
+      expect(text.scan(described_class::SCAN_REGEXP).flatten).to eq ['a.b', 'c.d']
+    end
+
+    it 'プレーンテキストにはマッチしない' do
+      expect('just a normal sentence with CT: and brackets [x]').not_to match described_class::SCAN_REGEXP
+    end
+  end
+end