plone / guillotina

Python AsyncIO data API to manage billions of resources
https://guillotina.readthedocs.io/en/latest/
Other
187 stars 51 forks source link

Update Transaction.contains, Transaction.tpc_commit #1053

Closed Qiwn closed 3 years ago

Qiwn commented 3 years ago

Update Transaction.contains, it also checks in the deleted dict Transaction.tpc_commit: process deleted items before added items (When we want to remove an existing obj, and add a new obj with the same id)

codecov-io commented 3 years ago

Codecov Report

Merging #1053 (7c3941f) into 5.x (06ba219) will increase coverage by 0.1%. The diff coverage is 100.0%.

Impacted file tree graph

@@           Coverage Diff           @@
##             5.x   #1053     +/-   ##
=======================================
+ Coverage   94.7%   94.7%   +0.1%     
=======================================
  Files        296     296             
  Lines      27255   27268     +13     
=======================================
+ Hits       25808   25822     +14     
+ Misses      1447    1446      -1     
Impacted Files Coverage Δ
guillotina/db/transaction.py 90.4% <100.0%> (+0.1%) :arrow_up:
guillotina/tests/test_transactions.py 100.0% <100.0%> (ø)
guillotina/db/storages/pg.py 89.3% <0.0%> (-0.1%) :arrow_down:
guillotina/contrib/pubsub/utility.py 86.9% <0.0%> (+2.2%) :arrow_up:
vangheem commented 3 years ago

5.3.63 released

Qiwn commented 3 years ago

thank you!