Add *.vue, *.md, and *.scss to allowlist to reuse frontend fixtures

scripts/gitlab_component_helpers.sh

@@ -118,7 +118,7 @@ function check_fixtures_download() {
   if [[ -z "${CI_MERGE_REQUEST_IID:-}" ]]; then
     return 1
   else
-    if tooling/bin/find_only_js_changes && ! fixtures_archive_doesnt_exist; then
+    if tooling/bin/find_only_allowed_files_changes && ! fixtures_archive_doesnt_exist; then
       return 0
     else
       return 1

spec/tooling/lib/tooling/find_changes_spec.rb

@@ -182,8 +182,8 @@
     end
   end
 
-  describe '#only_js_files_changed' do
-    subject { instance.only_js_files_changed }
+  describe '#only_allowed_files_changed' do
+    subject { instance.only_allowed_files_changed }
 
     context 'when fetching changes from changed files' do
       let(:from) { :changed_files }
@@ -200,8 +200,16 @@
         end
       end
 
-      context 'when changed files contain not only *.js changes' do
-        let(:changed_files_file_content) { 'a.js b.rb' }
+      context 'when changed files contain both *.vue and *.js changes' do
+        let(:changed_files_file_content) { 'a.js b.vue' }
+
+        it 'returns true' do
+          expect(subject).to be true
+        end
+      end
+
+      context 'when changed files contain not allowed changes' do
+        let(:changed_files_file_content) { 'a.js b.vue c.rb' }
 
         it 'returns false' do
           expect(subject).to be false

tooling/bin/find_only_js_changes → tooling/bin/find_only_allowed_files_changes

@@ -3,10 +3,10 @@
 
 require_relative '../lib/tooling/find_changes'
 
-if Tooling::FindChanges.new(from: :api).only_js_files_changed
-  puts "Only JS files were changed"
+if Tooling::FindChanges.new(from: :api).only_allowed_files_changed
+  puts "Only files with extensions #{ALLOWED_FILE_TYPES.join(', ')} were changed"
   exit 0
 else
-  puts "Changes were made to files other than JS files"
+  puts "Changes were made to files with extensions other than #{ALLOWED_FILE_TYPES.join(', ')}"
   exit 1
 end

tooling/lib/tooling/find_changes.rb

@@ -8,6 +8,8 @@ module Tooling
   class FindChanges
     include Helpers::PredictiveTestsHelper
 
+    ALLOWED_FILE_TYPES = ['.js', '.vue', '.md', '.scss'].freeze
+
     def initialize(
       from:,
       changed_files_pathname: nil,
@@ -41,8 +43,8 @@ def execute
       end
     end
 
-    def only_js_files_changed
-      file_changes.any? && file_changes.all? { |file| file.end_with?('.js') }
+    def only_allowed_files_changed
+      file_changes.any? && file_changes.all? { |file| ALLOWED_FILE_TYPES.include?(File.extname(file)) }
     end
 
     private