GeezerVison

Hi @WayneNani,
Thank you very much for creating this.
I'm on holiday currently so will look when I'm back, possibly @jgebal will be able earlier.

One thing could you please update the documentation reporters section with screens etc.?

Kind regards Lukasz

Hey @lwasylow ,
thanks for the quick reply! I changed the documentation as well, thanks for the heads-up.

Hey @lwasylow and @jgebal ,

is there anything else I can do to streamline the process for you?

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

Hello @WayneNani

As you have probably noticed, the project was on pause for quite some time.
I am planning to give it a small refresh and so will aim to integrate your pull-request.

Will you be available to address the review comments (if any)?

Hey @jgebal,

I am happy to address any remarks you have.

@WayneNani
One of widely used elements of utPLSQL is nested suites (with suitepaths) and contexts within suite.

How would the TAP reporter show results of nested suites or suite with contexts?

I would expect the report to show it as a subtest according to the TAP specification.

Even it it is already implemented in this PR, it is not mentioned in documentation of reporter and it is not expressed in the tests.

I would expect reporter to support the Commented Subtests (those are recommended).

@WayneNani
The PR is clean and implementation is really nice.
Thank you for the contribution.
The only concerns is around support for nested suites and contexts.
It is be required to have that tested and documented.

Once this is done. We will include this change in next release (coming soon).

@jgebal Thanks for the review! You are right, I forgot to include tests regarding suites and contexts and have fixed that. I decided to use the existing reporters test package which comes with a suitepath and context and left the escaping test package nearly untouched.
I also included a test to still include the subtest even when it only consists of disabled tests.

Because our pipeline doesn't seem to work with external PRs anymore, I will pull and deploy your changes locally to take a look.

@WayneNani
The PR looks really good.
Beautiful contribution tot he framework.

Thank you 👍

Starting with next release we will be only suppoorting DB 19c and above.

Hopefully you don't need the reporter for any unsupported version of Oracle.