[midPoint-git] [Evolveum/midpoint] f82d65: Fix two tests
mederly
noreply at github.com
Thu Aug 29 10:23:52 CEST 2024
Branch: refs/heads/tmp/inbounds-fix
Home: https://github.com/Evolveum/midpoint
Commit: f82d65c0a33eeef171a17fd5b86f69759b9eb85e
https://github.com/Evolveum/midpoint/commit/f82d65c0a33eeef171a17fd5b86f69759b9eb85e
Author: Pavol Mederly <mederly at evolveum.com>
Date: 2024-08-29 (Thu, 29 Aug 2024)
Changed paths:
M model/model-intest/src/test/java/com/evolveum/midpoint/model/intest/TestModelServiceContract.java
M model/model-intest/src/test/java/com/evolveum/midpoint/model/intest/negative/TestAssignmentErrors.java
M repo/repo-test-util/src/main/java/com/evolveum/midpoint/test/AbstractIntegrationTest.java
Log Message:
-----------
Fix two tests
- TestAssignmentErrors is now adapted to caching
- TestModelServiceContract is fixed regarding some earlier changes
(TestModelServiceContractCaching now caches more aggressively)
Commit: 5856f0c80153d25cee123fe8c64aa50aafa6f025
https://github.com/Evolveum/midpoint/commit/5856f0c80153d25cee123fe8c64aa50aafa6f025
Author: Pavol Mederly <mederly at evolveum.com>
Date: 2024-08-29 (Thu, 29 Aug 2024)
Changed paths:
M model/model-api/src/main/java/com/evolveum/midpoint/model/api/InboundSourceData.java
M model/model-impl/src/main/java/com/evolveum/midpoint/model/impl/lens/projector/focus/inbounds/FullInboundsProcessing.java
M model/model-impl/src/main/java/com/evolveum/midpoint/model/impl/lens/projector/focus/inbounds/prep/FullInboundsSource.java
Log Message:
-----------
Fix inbounds regarding deltas
#1 in 441ac351a6b6c8c28be8baeafcfb5a85b2b80757 was perhaps too naive,
as it did not take volatile resources in to account: the old state
of the object plus delta may be different from its new state.
So, here we are more cautious: we assume that if there's no delta
for an attribute, the new state is the authoritative one. It's still
not 100% correct (as there may be deltas on volatile attributes), but
it's the best that could be done here.
This should fix TestVolatility and TestInternalCorrelationMultiAccounts.
Compare: https://github.com/Evolveum/midpoint/compare/f82d65c0a33e%5E...5856f0c80153
To unsubscribe from these emails, change your notification settings at https://github.com/Evolveum/midpoint/settings/notifications
More information about the midPoint-svn
mailing list