Commit fc5f8cb8 authored by Lin Jen-Shin's avatar Lin Jen-Shin

Merge branch '347202-package-and-qa-fails-consistently-in-master' into 'master'

Don't fail check_if_qa_only_spec_changes if tmp/changed_files.txt missing

See merge request gitlab-org/gitlab!76101
parents 6b90a7a6 4fafca23
......@@ -6,13 +6,17 @@
# all the files are end-to-end test spec files.
abort("ERROR: Please specify the file containing the list of changed files and a file where the qa only spec files will be written") if ARGV.size != 2
file_contents = File.read(ARGV.shift).split(' ')
all_files_are_qa_specs = file_contents.all? { |file_path| file_path =~ %r{^qa\/qa\/specs\/features\/} }
changed_files_path = ARGV.shift
output_file = ARGV.shift
return unless File.exist?(changed_files_path)
changed_files = File.read(changed_files_path).split(' ')
all_files_are_qa_specs = changed_files.all? { |file_path| file_path =~ %r{^qa\/qa\/specs\/features\/} }
if all_files_are_qa_specs
qa_spec_paths_trimmed = file_contents.map { |path| path.sub('qa/', '') }
qa_spec_paths_trimmed = changed_files.map { |path| path.sub('qa/', '') }
File.write(output_file, qa_spec_paths_trimmed.join(' '))
end
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment