Do no treat non-VCS test components as updatable

What does this MR do and why?

Exit early in setting up the test environment for non-VCS test components. openbao is such an example. This component uses the gitlab:secrets_management:openbao:download_or_clone Rake task that downloads the openbao binary without setting up the GIT repository in tmp/tests/openbao. When the test environment is being set up, the test helper calls component_timed_setup which invokes the GIT binary with the following arguments:

git -C tmp/tests/openbao merge-base --is-ancestor <COMMIT_SHA> HEAD

Due to the design of GIT, it traverses all directories until it finds the .git directory against which it can execute the command mentioned above. In our case, the GIT reaches the GitLab repository, which might be slow to respond.

It was reported by team members that this patch reduces the setup time from 2+ minutes to 2 seconds depending on the GIT repo status:

Before: Test environment set up in 160.159391346 seconds

After: Test environment set up in 2.480034855 seconds

References

N/A

MR acceptance checklist

Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

Screenshots or screen recordings

Not required

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

spec/support/helpers/test_env.rb

@@ -535,6 +535,8 @@ def component_needs_update?(component_folder, expected_version)
   end
 
   def component_ahead_of_target?(component_folder, expected_version)
+    return false unless Dir.exist?(File.join(component_folder, '.git'))
+
     # The HEAD of the component_folder will be used as heuristic for the version
     # of the binaries, allowing to use Git to determine if HEAD is later than
     # the expected version. Note: Git considers HEAD to be an anchestor of HEAD.
@@ -556,6 +558,8 @@ def component_matches_git_sha?(component_folder, expected_version)
 
     return false unless Dir.exist?(component_folder)
 
+    return false unless Dir.exist?(File.join(component_folder, '.git'))
+
     sha, exit_status = Gitlab::Popen.popen(%W[#{Gitlab.config.git.bin_path} rev-parse HEAD], component_folder)
     return false if exit_status != 0