NC | lifecycle | fix expire-delete-marker issues

[nadavMiz]

Describe the Problem

three problems:

1. if both noncurrent and expire delete marker rules were defined together. the delete marker rules would set the state as is_finished causing nocurrent rule to not run
2. list_version did not pass properly between nocurrent rules causing it to try to load again
3. for expiration rule the default was to get the number of dats from the rule. but if there was no number of days defined, say when expire delete marker rule is used. then it will assume number of days 0

Explain the Changes

1. check if isFinished only if version list is not defined. if it is defined than it got it from a previous action that determined that the rule is not finished
2. move the versions_list to a params object, and pass it between the actions. that way it will be saved between actions. and we will not need to list again
3. for getting the expiration add a condition for days argument. also set the default to be -1, which means do not run. so if both days and date are not defined (currently expire delete marker), the rule will not run

Issues: Fixed #xxx / Gap #xxx

Testing Instructions:

• Doc added/updated
• Tests added