fix(namespace): incorrect namespacing of test classes

[JonnyPower]

closes #296

What does this PR do?

Correctly applies namespace to namespaced test classes

What issues does this PR fix or reference?

#296

Functionality Before

Classes were reporting with namespace and __ - this is not correct, and resubmitting a test run with that full name fails. Correct namespacing of apex classes is NS.<class>

Functionality After

Correctly return class full names

[JonnyPower]

@mshanemc / @jeffb-sfdc is there a process I need to follow to have PRs queued up for review? I notice on other PRs there's a requirement for a GUS number internally, is that something you can help with?

[JonnyPower]

Hi @AnanyaJha - what's the process for getting reviewed / kicking off CI builds?

[AnanyaJha]

Hi @AnanyaJha - what's the process for getting reviewed / kicking off CI builds?

Hi @JonnyPower ! Apologies the team hasn't been able to take a look at your PR yet - I'll ask a member of our engineering team to review and they should be able to kick off the appropriate builds.

[JonnyPower]

Hi @AnanyaJha, any update? these are getting quite stale and the more stale they become the higher risk of integration issues that will need remediated.

I have another PR also in the same state; #324

[JonnyPower]

@AnanyaJha bumping thread 📭

[JonnyPower]

@AnanyaJha :tumbleweed:

[JonnyPower]

@shetzel could you have a look at this one too?

[shetzel]

@JonnyPower - I added it to my list. Thanks for the contribution!

[shetzel]

@JonnyPower - this looks good to me. I have an equivalent PR that is up to date with main and passed all tests. (#339)
I'd like for you to get the credit as author though so can you please update your fork with the latest from main so we can merge your PR rather than mine? If you don't care, I'll just merge 339 since it's the same.

[shetzel]

fixed with equivalent PR: #339

[JonnyPower]

thanks @shetzel - appreciate it!