utPLSQL / utPLSQL-PLSQL-Developer

utPLSQL Plugin for PL/SQL Developer
Apache License 2.0
26 stars 10 forks source link

Show Disable Reason in Result Window #25

Open simasch opened 2 years ago

simasch commented 2 years ago

utPLSQL adds a reason to a disabled test: https://github.com/utPLSQL/utPLSQL/issues/610

Add the reason to the result window if the test is disabled.

jgebal commented 2 years ago

Currently the ut_realtime_reporter is producing following output with disabled flag:

<test>
    <disabled>true</disabled>
</test>

If test is not disabled we get the following outcome.

<test>
    <disabled>false</disabled>
</test>

To keep backward compatibility we should not change that behaviour and add another flag. I suggest introduction of <disabled_reason><![CDATA[Disabled reason description]]></disabled_reason>

So the results would be one of:

jgebal commented 2 years ago

@simasch @lwasylow @PhilippSalvisberg

lwasylow commented 2 years ago

I allowed myself to correct your post @jgebal as the word fale got me really confused :P I totally agree that a new flag called disabled_reason would be best way.

PhilippSalvisberg commented 2 years ago

I explained in utPLSQL/utPLSQL#1192 why I would use CamelCase and name the node disabledReason. IMO the next step is to implement the change outlined there.

PhilippSalvisberg commented 2 years ago

Version 1.4.0 of the SQLDev extension can show the disabled reason. Here's a screenshot:

image