Change Analytics unique visits key naming

doc/development/redis.md

@@ -50,7 +50,6 @@ project:{1}:set_b
 project:{2}:set_c
 ```
 
-
 `set_a` and `set_b` are guaranteed to be held on the same Redis server, while `set_c` is not.
 
 Currently, we validate this in the development and test environments

lib/gitlab/analytics/unique_visits.rb

@@ -52,8 +52,10 @@ module Gitlab
       def key(target_id, time)
         raise "Invalid target id #{target_id}" unless TARGET_IDS.include?(target_id.to_s)
 
+        target_key = target_id.to_s.gsub('analytics', '{analytics}')
         year_week = time.strftime('%G-%V')
-        "#{target_id}-{#{year_week}}"
+
+        "#{target_key}-#{year_week}"
       end
 
       def keys(targets:, timeframe_start:, weeks:)

spec/lib/gitlab/analytics/unique_visits_spec.rb

@@ -47,7 +47,7 @@ RSpec.describe Gitlab::Analytics::UniqueVisits, :clean_gitlab_redis_shared_state
       unique_visits.track_visit(visitor1_id, target1_id)
 
       Gitlab::Redis::SharedState.with do |redis|
-        redis.scan_each(match: "#{target1_id}-*").each do |key|
+        redis.scan_each(match: "{#{target1_id}}-*").each do |key|
           expect(redis.ttl(key)).to be_within(5.seconds).of(12.weeks)
         end
       end