🐛 fix for pending transactions

[matusdrobuliak66]

What do these changes do?

• 🐛 In the previous PR, I introduced an option to run the pricing unit with a cost of 0 credits. This PR fixes an issue with the if statement that would evaluate bool(Decimal(0)) as False, which is not intended.

Related issue/s

• relates run dynamic services when price/hour is 0 even when out of credits osparc-issues#1438
• relates TIP v3 - osparc-devteam-tasks osparc-issues#1309

How to test

services/resource-usage-tracker/tests/unit/with_dbs/test_process_rabbitmq_message_with_billing_cost_0.py

Dev-ops checklist

• No ENV changes or I properly updated ENV (read the instruction)

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 65.0%. Comparing base (cafbf96) to head (aff537f).
Report is 219 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff            @@
##           master   #5852      +/-   ##
=========================================
- Coverage    84.5%   65.0%   -19.6%     
=========================================
  Files          10     593     +583     
  Lines         214   29515   +29301     
  Branches       25     205     +180     
=========================================
+ Hits          181   19189   +19008     
- Misses         23   10274   +10251     
- Partials       10      52      +42     

Flag	Coverage Δ
integrationtests	64.0% <ø> (?)
unittests	87.0% <100.0%> (+2.4%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files	Coverage Δ
..._usage_tracker/resource_tracker_background_task.py	95.0% <100.0%> (ø)
...usage_tracker/resource_tracker_process_messages.py	81.2% <100.0%> (ø)

... and 601 files with indirect coverage changes

[bisgaard-itis]

Thanks for the fix 😄

[sanderegg]

you do not cover the case where the pricing_unit_cost would be None with this fix. Please check my comment, thanks!

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud