Commit d207d69b authored by Dmitriy Zaporozhets's avatar Dmitriy Zaporozhets

Merge branch 'exec_warning' into 'master'

Explain why GitlabShell#exec_cmd has no tests
parents 3df55367 bb4f64d6
...@@ -59,6 +59,7 @@ class GitlabShell ...@@ -59,6 +59,7 @@ class GitlabShell
api.allowed?(@git_cmd, @repo_name, @key_id, '_any') api.allowed?(@git_cmd, @repo_name, @key_id, '_any')
end end
# This method is not covered by Rspec because it ends the current Ruby process.
def exec_cmd(*args) def exec_cmd(*args)
Kernel::exec({'PATH' => ENV['PATH'], 'GL_ID' => ENV['GL_ID']}, *args, unsetenv_others: true) Kernel::exec({'PATH' => ENV['PATH'], 'GL_ID' => ENV['GL_ID']}, *args, unsetenv_others: true)
end 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