fix: Default height/width to 100% if either doesn't exist

[ajay-sentry]

Purpose/Motivation

Fixes sentry issue: https://codecov.sentry.io/issues/2768942613/?environment=epic&environment=dd&environment=enova&environment=mailchimp&environment=production&environment=roblox&environment=snowflake&environment=prodvana&project=5514400&project=5215654&query=is%3Aunresolved+issue.priority%3A%5Bhigh%2C+medium%5D&referrer=issue-stream&statsPeriod=7d&stream_index=9

We just want to default a numerical value in the event both the .get() and the default from the get are non-integer values.

The sentry issue is complaining that the default value is '', and if you do '' or 100 (like in the PR), we will use 100

These eventually bubble up to here:

codecov-api/graphs/helpers/graph_utils.py

Lines 102 to 117 in c9bc2c8

	def _make_svg(width, height, elements, viewPortWidth=None, viewPortHeight=None):
	return (
	'<svg baseProfile="full" width="{0}" height="{1}" viewBox="0 0 {4} {5}" version="1.1"\n'
	'xmlns="http://www.w3.org/2000/svg" xmlns:ev="http://www.w3.org/2001/xml-events"\n'
	'xmlns:xlink="http://www.w3.org/1999/xlink">\n'
	"{2}\n"
	"{3}\n"
	"</svg>".format(
	width,
	height,
	style_n_defs,
	"\n".join(elements),
	viewPortWidth or width,
	viewPortHeight or height,
	)
	)

Links to relevant tickets

Closes codecov/engineering-team#2204

Notes to Reviewer

Anything to note to the team? Any tips on how to review, or where to start?

Legal Boilerplate

Look, I get it. The entity doing business as "Sentry" was incorporated in the State of Delaware in 2015 as Functional Software, Inc. In 2022 this entity acquired Codecov and as result Sentry is going to need some rights from me in order to utilize my contributions in this PR. So here's the deal: I retain all rights, title and interest in and to my contributions, and by keeping this boilerplate intact I confirm that Sentry can use, modify, copy, and redistribute my contributions, under Sentry's choice of terms.

[codecov-notifications]

Codecov Report

All modified and coverable lines are covered by tests ✅

✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 96.02%. Comparing base (e245d32) to head (8a027e6).

Changes have been made to critical files, which contain lines commonly executed in production. Learn more

✅ All tests successful. No failed tests found.

Additional details and impacted files

@@             Coverage Diff             @@
##               main       #733   +/-   ##
===========================================
  Coverage   96.02000   96.02000           
===========================================
  Files           814        814           
  Lines         18428      18428           
===========================================
  Hits          17695      17695           
  Misses          733        733           

Flag	Coverage Δ
unit	91.73% <ø> (ø)
unit-latest-uploader	91.73% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.