Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Render the test_log value correctly when config has http url #4366

Merged
merged 1 commit into from
Nov 18, 2021

Conversation

b10n1k
Copy link
Contributor

@b10n1k b10n1k commented Nov 17, 2021

The gitrepodir returns the url directly as it appears in the config file,
when the renderInvestigationTab expects the value of data-testgiturl retrieved from investigation_ajax
in the form of http://github.com///commit

Apply the same substitution as in git before return

Signed-off-by: ybonatakis ybonatakis@suse.com

The `gitrepodir` returns the url directly as it appears in the config file,
when the `renderInvestigationTab` expects the value of `data-testgiturl` retrieved from `investigation_ajax`
in the form of _http://github.com/<user>/<repo>/commit_

Apply the same substitution as in git before return

Signed-off-by: ybonatakis <ybonatakis@suse.com>
@codecov
Copy link

codecov bot commented Nov 17, 2021

Codecov Report

Merging #4366 (59291eb) into master (633b27c) will increase coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #4366   +/-   ##
=======================================
  Coverage   97.94%   97.94%           
=======================================
  Files         371      371           
  Lines       33714    33719    +5     
=======================================
+ Hits        33021    33026    +5     
  Misses        693      693           
Impacted Files Coverage Δ
lib/OpenQA/Utils.pm 99.50% <100.00%> (+<0.01%) ⬆️
t/16-utils.t 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 633b27c...59291eb. Read the comment docs.

@mergify mergify bot merged commit 75fb859 into os-autoinst:master Nov 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants