plone / guillotina

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

Fix @pytest.mark.app_settings modified original settings dictionary #1167

Closed masipcat closed 2 years ago

masipcat commented 2 years ago

The bug can be reproduced with pytest guillotina/tests -k test_app_settings_mark -vvv (and undoing the fix)

codecov-commenter commented 2 years ago

Codecov Report

Merging #1167 (2ae6d79) into master (d608d78) will increase coverage by 0.1%. The diff coverage is 100.0%.

Impacted file tree graph

@@           Coverage Diff            @@
##           master   #1167     +/-   ##
========================================
+ Coverage    94.6%   94.6%   +0.1%     
========================================
  Files         377     377             
  Lines       32598   32606      +8     
========================================
+ Hits        30819   30827      +8     
  Misses       1779    1779             
Impacted Files Coverage Δ
guillotina/factory/app.py 94.2% <100.0%> (ø)
guillotina/tests/test_configure.py 98.3% <100.0%> (+0.1%) :arrow_up: