Feat: Implement Cleaning Logic on Reserve Task

[fabiodmota]

Description

Request new cleaning tasks for step "GenericCleaning" from Orchestrator
Create Legal Entity, Site and Address representations from the given generic business partner data
Create BPN request identifiers where missing
Post dummy cleaning results to Orchestrator for step "GenericCleaning"
Create configurable scheduling for performing these on regular time interval

#433

Pre-review checks

Please ensure to do as many of the following checks as possible, before asking for committer review:

• DEPENDENCIES are up-to-date. Dash license tool. Committers can open IP issues for restricted libs.
• Copyright and license header are present on all affected files

[fabiodmota]

Logic diagram can be seen on this #433 (comment)

[alexsilva-CGI]

Looks good to me. Logic for cleaning seems also good.

[martinfkaeser]

If the Pool validates a BP (#518) as discussed today, the logic when to create an address needs to be changed.
There we said a cleaning service shouldn't send redundant data. So it should only set an address if it is an additional address. So legalAddress, siteMainAddress and (additional) address shouldn't share a BPN-A-Ref (no matter if real BPN or BpnRequestId).

Did I get that right? @nicoprow @rainer-exxcellent @fabiodmota

[fabiodmota]

If the Pool validates a BP (#518) as discussed today, the logic when to create an address needs to be changed. There we said a cleaning service shouldn't send redundant data. So it should only set an address if it is an additional address. So legalAddress, siteMainAddress and (additional) address shouldn't share a BPN-A-Ref (no matter if real BPN or BpnRequestId).

Did I get that right? @nicoprow @rainer-exxcellent @fabiodmota

So based on that description i changed the code on bpdm-cleaning-service-dummy/src/main/kotlin/org/eclipse/tractusx/bpdm/cleaning/service/CleaningServiceDummy.kt the addres will only be passed based on the address type

  private fun shouldCreateAddress(
        addressType: AddressType?,
        addressPartner: LogisticAddressDto
    ): LogisticAddressDto? {
        val addressDto = if (addressType == AddressType.AdditionalAddress) {
            addressPartner
        } else {
            null
        }
        return addressDto
    }

is this the correct logic ?