1 files changed, 41 insertions, 7 deletions

diff --git a/spec/lib/generators/gitlab/snowplow_event_definition_generator_spec.rb b/spec/lib/generators/gitlab/snowplow_event_definition_generator_spec.rb
index 4e172dd32f0..d9fa6b931ad 100644
--- a/spec/lib/generators/gitlab/snowplow_event_definition_generator_spec.rb
+++ b/spec/lib/generators/gitlab/snowplow_event_definition_generator_spec.rb
@@ -5,6 +5,7 @@ require 'spec_helper'
 RSpec.describe Gitlab::SnowplowEventDefinitionGenerator, :silence_stdout do
   let(:ce_temp_dir) { Dir.mktmpdir }
   let(:ee_temp_dir) { Dir.mktmpdir }
+  let(:timestamp) { Time.current.to_i }
   let(:generator_options) { { 'category' => 'Groups::EmailCampaignsController', 'action' => 'click' } }
 
   before do
@@ -12,6 +13,10 @@ RSpec.describe Gitlab::SnowplowEventDefinitionGenerator, :silence_stdout do
     stub_const("#{described_class}::EE_DIR", ee_temp_dir)
   end
 
+  around do |example|
+    freeze_time { example.run }
+  end
+
   after do
     FileUtils.rm_rf([ce_temp_dir, ee_temp_dir])
   end
@@ -22,16 +27,41 @@ RSpec.describe Gitlab::SnowplowEventDefinitionGenerator, :silence_stdout do
     end
 
     let(:sample_event_dir) { 'lib/generators/gitlab/snowplow_event_definition_generator' }
+    let(:file_name) { Dir.children(ce_temp_dir).first }
 
     it 'creates CE event definition file using the template' do
       sample_event = ::Gitlab::Config::Loader::Yaml.new(fixture_file(File.join(sample_event_dir, 'sample_event.yml'))).load_raw!
 
       described_class.new([], generator_options).invoke_all
 
-      event_definition_path = File.join(ce_temp_dir, 'groups__email_campaigns_controller_click.yml')
+      event_definition_path = File.join(ce_temp_dir, file_name)
       expect(::Gitlab::Config::Loader::Yaml.new(File.read(event_definition_path)).load_raw!).to eq(sample_event)
     end
 
+    describe 'generated filename' do
+      it 'includes timestamp' do
+        described_class.new([], generator_options).invoke_all
+
+        expect(file_name).to include(timestamp.to_s)
+      end
+
+      it 'removes special characters' do
+        generator_options = { 'category' => '"`ui:[mavenpackages | t5%348()-=@ ]`"', 'action' => 'click' }
+
+        described_class.new([], generator_options).invoke_all
+
+        expect(file_name).to include('uimavenpackagest')
+      end
+
+      it 'cuts name if longer than 100 characters' do
+        generator_options = { 'category' => 'a' * 100, 'action' => 'click' }
+
+        described_class.new([], generator_options).invoke_all
+
+        expect(file_name.length).to eq(100)
+      end
+    end
+
     context 'event definition already exists' do
       before do
         stub_const('Gitlab::VERSION', '12.11.0-pre')
@@ -44,7 +74,7 @@ RSpec.describe Gitlab::SnowplowEventDefinitionGenerator, :silence_stdout do
         stub_const('Gitlab::VERSION', '13.11.0-pre')
         described_class.new([], generator_options.merge('force' => true)).invoke_all
 
-        event_definition_path = File.join(ce_temp_dir, 'groups__email_campaigns_controller_click.yml')
+        event_definition_path = File.join(ce_temp_dir, file_name)
         event_data = ::Gitlab::Config::Loader::Yaml.new(File.read(event_definition_path)).load_raw!
 
         expect(event_data).to eq(sample_event)
@@ -56,13 +86,17 @@ RSpec.describe Gitlab::SnowplowEventDefinitionGenerator, :silence_stdout do
       end
     end
 
-    it 'creates EE event definition file using the template' do
-      sample_event = ::Gitlab::Config::Loader::Yaml.new(fixture_file(File.join(sample_event_dir, 'sample_event_ee.yml'))).load_raw!
+    describe 'EE' do
+      let(:file_name) { Dir.children(ee_temp_dir).first }
 
-      described_class.new([], generator_options.merge('ee' => true)).invoke_all
+      it 'creates EE event definition file using the template' do
+        sample_event = ::Gitlab::Config::Loader::Yaml.new(fixture_file(File.join(sample_event_dir, 'sample_event_ee.yml'))).load_raw!
 
-      event_definition_path = File.join(ee_temp_dir, 'groups__email_campaigns_controller_click.yml')
-      expect(::Gitlab::Config::Loader::Yaml.new(File.read(event_definition_path)).load_raw!).to eq(sample_event)
+        described_class.new([], generator_options.merge('ee' => true)).invoke_all
+
+        event_definition_path = File.join(ee_temp_dir, file_name)
+        expect(::Gitlab::Config::Loader::Yaml.new(File.read(event_definition_path)).load_raw!).to eq(sample_event)
+      end
     end
   end
 end