Fix for generating report from multiple different databases

[suryagupta4]

Description

Fixed the generation of report for the test runs from multiple different databases

GitHub Issues

List the GitHub issues impacted by this PR:

GitHub Issue #
dell/csm#921

Checklist:

• I have performed a self-review of my own code to ensure there are no formatting, vetting, linting, or security issues
• I have verified that new and existing unit tests pass locally with my changes
• I have not allowed coverage numbers to degenerate
• I have maintained at least 90% code coverage
• I have commented my code, particularly in hard-to-understand areas
• I have made corresponding changes to the documentation
• I have added tests that prove my fix is effective or that my feature works
• I have maintained backward compatibility

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Please also list any relevant details for your test configuration

• Ran make test
  

• Verified by generating report for multiple test runs from different databases. No errors received.
  
  

[adarsh-dell]

Will it be okay to add some debug/warn level log message, In case if we are unable to process the metric for some of SC/DB.

[AkshaySainiDell]

I agree with @adarsh-dell. Along with logger you can move line 131 inside this if block. This way we can remove testRunFound variable.

Suggested change

	if err != nil {
	continue
	}
	if err != nil {
	log.Warnf("A warning message..")
	invalidTestRuns = append(invalidTestRuns, testRun)
	continue
	}

[AkshaySainiDell]

lgtm