Don't rely on js for badge length setting

[asdil12]

JS doesn't work when SVG is shown via tag.

Ticket: https://progress.opensuse.org/issues/124173

[asdil12]

[Martchus]

I generally like the lookup-table approach. Seems much simpler/faster than your previous if-else-cascade attempt.

By the way, how did you come up with these scaling factors? Maybe it is worth adding a comment?

[asdil12]

The lookup table was created using the HTML testpage (see https://progress.opensuse.org/issues/124173#note-12) which looks like #5045 (comment).

openQA/lib/OpenQA/WebAPI/Controller/Test.pm

Lines 42 to 44 in 3f9cca7

	# scaling factors for chars - determined by using a test page (see https://progress.opensuse.org/issues/124173#note-12)
	# showing 20x the same char in a badge with one badge for each char or symbol
	# and then adjusting the lookup table values until the size of each badge matched.

[codecov]

Codecov Report

Merging #5045 (d20e18a) into master (39af241) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master    #5045   +/-   ##
=======================================
  Coverage   98.23%   98.23%           
=======================================
  Files         381      381           
  Lines       35870    35876    +6     
=======================================
+ Hits        35238    35244    +6     
  Misses        632      632           

Impacted Files	Coverage Δ
lib/OpenQA/WebAPI/Controller/Test.pm	95.52% <100.00%> (+0.06%)	⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.