Skip to content
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.

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

feat: add support for deletedAt and NGSI-LD Null #1281

Open
wants to merge 32 commits into
base: develop
Choose a base branch
from

Conversation

bobeal
Copy link
Member

@bobeal bobeal commented Dec 8, 2024

First iteration with support in core and temporal API

Copy link
Contributor

github-actions bot commented Dec 8, 2024

Test Results

   70 files  + 1     70 suites  +1   1m 22s ⏱️ ±0s
1 136 tests +26  1 135 ✅ +25  0 💤 ±0  1 ❌ +1 
1 175 runs  +26  1 174 ✅ +25  0 💤 ±0  1 ❌ +1 

For more details on these failures, see this check.

Results for commit 8f70cea. ± Comparison against base commit a76f793.

This pull request removes 197 and adds 67 tests. Note that renamed tests count towards both.
                                    { "id":…, withTemporalValues=true, withAudit=false, expectation={
                      "@id": "https://uri…
                      "@type": "@json",
                      …
                    "@value": "/A/B"
                    "@value": "/C/D"
                    "@value": 20
                    "…
                    {
                  "@type": "https://uri.etsi.org/ngsi-ld/DateTime",
…
com.egm.stellio.search.authorization.service.EntityAccessRightsServiceTests ‑ getSubjectAccessRights should include deleted entities if it is asked for()
com.egm.stellio.search.authorization.service.EntityAccessRightsServiceTests ‑ getSubjectAccessRights should not include deleted entities if it is not asked for()
com.egm.stellio.search.authorization.web.EntityAccessControlHandlerTests ‑ get authorized entities should ask for deleted entities if includeDeleted query param is true()
com.egm.stellio.search.authorization.web.EntityAccessControlHandlerTests ‑ get authorized entities should not ask for deleted entities if includeDeleted query param is not provided()
com.egm.stellio.search.entity.service.EntityAttributeServiceTests ‑ it should delete an attribute in merge operation if its value is NGSI-LD null()
com.egm.stellio.search.entity.service.EntityAttributeServiceTests ‑ it should delete an attribute in partial attribute update operation if its value is NGSI-LD null()
com.egm.stellio.search.entity.service.EntityAttributeServiceTests ‑ it should delete an attribute in update operation if its value is NGSI-LD null()
com.egm.stellio.search.entity.service.EntityAttributeServiceTests ‑ it should flag and audit a deleted attribute()
com.egm.stellio.search.entity.service.EntityAttributeServiceTests ‑ it should flag and audit all deleted attributes of an entity()
com.egm.stellio.search.entity.service.EntityAttributeServiceTests ‑ it should flag and audit all instances of a deleted attribute()
…

♻️ This comment has been updated with latest results.

@bobeal bobeal force-pushed the feature/919-add-support-for-ngsild-null branch 3 times, most recently from 00d12a8 to 807172e Compare December 14, 2024 16:11
@bobeal bobeal self-assigned this Dec 17, 2024
@bobeal bobeal added core Relates to core API temporal Relates to temporal API labels Dec 17, 2024
@bobeal bobeal force-pushed the feature/919-add-support-for-ngsild-null branch from 44d85c2 to 34f39d7 Compare December 19, 2024 05:52
@bobeal bobeal force-pushed the feature/919-add-support-for-ngsild-null branch 2 times, most recently from c04cca5 to 4b03b4a Compare December 20, 2024 09:41
@bobeal bobeal force-pushed the feature/919-add-support-for-ngsild-null branch from b0aa632 to 6123e50 Compare December 21, 2024 14:36
@bobeal bobeal force-pushed the feature/919-add-support-for-ngsild-null branch from aa4e834 to d522201 Compare December 23, 2024 11:00
@bobeal bobeal marked this pull request as ready for review December 23, 2024 16:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
core Relates to core API feature New feature or request ngsild-1.6.1 temporal Relates to temporal API
Projects
Status: In progress
Development

Successfully merging this pull request may close these issues.

Add support for the deletedAt temporal property Add support for NGSI-LD Null (4.5.0 / 5.4)
3 participants