allow restrictions to be retrieved only by item / location

[hansmorb]

closes #1178
UnitTests schon vorgebaut in experiment/unittests UnitTests sind mit dabei

[datengraben]

Ich verstehe nicht wieso du die Tests in einem separaten Branch eincheckst. Ausgeführt werden sie hier in diesem Branch doch auch. Welchen Grund hat das?

[hansmorb]

Ich verstehe nicht wieso du die Tests in einem separaten Branch eincheckst. Ausgeführt werden sie hier in diesem Branch doch auch. Welchen Grund hat das?

Das stimmt, der Fehler ist aufgeflogen als ich neue Unit Tests auf dem anderen Branch gebastelt habe. Aber habe sie auch mal hier eingefügt.

[hansmorb]

Das ist potentiell ein heikler Change, weil die filterPosts sehr oft von vielen Methoden verwendet wird.

[datengraben]

@hansmorb Stimmt, grundsätzlich würde ich sagen sollten wir einfach versuchen via Unit-Tests für Repository/Restriction alles abzufrühstücken.
Also noch folgende Fälle:

• Query nur mit einer Location-ID
• Query nur mit einer Item-ID
• Und vielleicht noch einen Fall mit einem Array mit mehreren Elementen.

Zusätzlich habe ich mal eine Liste aller Aufrufer gemacht. Ggf. können wir die Testfälle für diese Methoden auf Vollständigkeit überprüfen. Oder um uns noch sicherer zu fühlen schreiben wir ein paar E2E Tests.

Liste der Callee's (via PHPStorm)

• src/Helper/Wordpress.php
  • getRelatedPostsIdsForItem
  • getRelatedPostsIdsForLocation
• src/Model/Day.php
  • getRestrictions
• src/Model/Item.php
  • getRestrictions
• src/Model/Location.php
  • getRestrictions
• src/View/Item.php
  • getTemplateData
• src/View/Location.php
  • getTemplateData

[codecov]

Codecov Report

Merging #1179 (3d46b78) into master (8abd9e2) will increase coverage by 0.20%.
Report is 1 commits behind head on master.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master    #1179      +/-   ##
============================================
+ Coverage     35.85%   36.06%   +0.20%     
- Complexity     2115     2117       +2     
============================================
  Files            83       83              
  Lines          8596     8602       +6     
============================================
+ Hits           3082     3102      +20     
+ Misses         5514     5500      -14     

Files	Coverage Δ
src/Repository/Restriction.php	87.61% <100.00%> (+0.75%)	⬆️

... and 2 files with indirect coverage changes