Public

Project privacy set to public. By default, its content is available to everyone (authenticated or not). Please note that more restrictive permissions might exist on some items.

stable

Clone or download

Read-only

fix: DuckTypedFieldChecker don't throw all exceptions

Download Browse

Part of story #10710 search on fields with duck typi When searching on lists, if a label is not present in one the field an exception is thrown. But in XTS context this is not fatal: if one field don't have the label it doesn't matter, but if all fields don't have it, it matters *Testing:* With 2 trackers with a list field: - Search on a label existing in only one field -> it should pass - Search on a label existing in all fields -> it should pass - Search on a label existing in no field -> it should not pass (an error is shown) Change-Id: Ie0331f5c8f04cc6546a84a95ded1c29236feec23

+103 −8
Kevin Traini (ktraini) 2024-03-15 09:31
Kevin Traini (ktraini) 2024-03-15 12:53
d53868cad7fa50fbaa8c312a8b6cfb9cdffb2c4f
d381a09c693f655c044eca10595e620d8724917c
git #tuleap/stable/d53868cad7fa50fbaa8c312a8b6cfb9cdffb2c4f

Modified Files

Side by side diff
Inline diff
Name
M plugins/crosstracker/include/CrossTracker/Report/Query/Advanced/QueryValidation/DuckTypedField/DuckTypedFieldChecker.php +7 −2 Go to diff View file
M plugins/crosstracker/tests/unit/CrossTracker/Report/Query/Advanced/QueryValidation/DuckTypedField/DuckTypedFieldCheckerTest.php +89 −5 Go to diff View file
M plugins/crosstracker/tests/unit/CrossTracker/Tests/Builders/InvalidSearchableCollectorParametersBuilder.php +6 −0 Go to diff View file
M plugins/tracker/tests/unit/Builders/Fields/List/ListStaticBindBuilder.php +1 −1 Go to diff View file