New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Improve test coverage in DataObjectTest #11010

Open

3 tasks

sabina-talipova opened this issue Oct 18, 2023 · 0 comments

Open

3 tasks

Improve test coverage in DataObjectTest #11010

sabina-talipova opened this issue Oct 18, 2023 · 0 comments

Labels

affects/v5 type/enhancement

Contributor

sabina-talipova commented Oct 18, 2023 •

edited

Loading

Description

There are a few test methods that should be updated:

testFieldExistence - Re-enable all test cases for field existence. See
testFieldInheritance - Re-enable all test cases for field inheritance aggregation. See
testMerge: See
- test "left" priority flag
- test includeRelations flag
- test includeRelations in combination with overwriteWithEmpty
- test has_one relations
- test has_many and many_many relations

sabina-talipova added type/enhancement affects/v5 labels

This was referenced Oct 18, 2023

Remove all TODO comments in the code silverstripe/.github#116

Closed

MNT Remove TODO comments #11032

Merged

GuySartorelli changed the title ~~TODO: DataObjectTest updates~~ Improve test coverage in DataObjectTest

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment