Bug: Fix local variable scoping issue affecting false-pos test output

[simplyNour]

Fixes: #2635

Current F+ test result for "Gradle"

After Fix

[simplyNour]

@ppfeister
Can you please review this PR?
Also, let me know if you agree with the extended more fail-proof solution of updating the pipeline to catch failures that are due to python raised exceptions rather than test failures.
Thanks in advance.

[ppfeister]

Can't believe this wasn't caught yet. Good one!

Also, let me know if you agree with the extended more fail-proof solution of updating the pipeline to catch failures that are due to python raised exceptions rather than test failures.

Could you elaborate on this point?

Generally, pipelines should fail when an (unexpected) error is thrown, because that's the point of testing. Do you mean fixing it so that the site validation workflow fails properly? (catch normally implies a graceful fail and continue, so making sure we're on the same page)

(marked as hacktoberfest-accepted so it counts immediately, but waiting to merge based on above feedback in case this site is a good test case)