fix: penalizing or rewarding constraints with overlapping time for Employee Scheduling

[zepfred]

Description of the change

This pull request updates the employee-scheduling module constraints to penalize or reward constraints with overlapping time.

Java

Python

Checklist

Development

• The changes have been covered with tests, if necessary.
• You have a green build, with the exception of the flaky tests.
• UI and JS files are fully tested, the user interface works for all modules affected by your changes (e.g., solve and analyze buttons).
• The network calls work for all modules affected by your changes (e.g., solving a problem).
• The console messages are validated for all modules affected by your changes.

Code Review

• This pull request includes an explanatory title and description.
• The GitHub issue is linked.
• At least one other engineer has approved the changes.
• After PR is merged, inform the reporter.

[triceo]

Please figure out an incremental approach to the requirement.

[Christopher-Chianelli]

Note: the part before penalize does not need to change; shifts are less than 24 hours long, and availabilities are based on dates, so shiftStart.date() == availability.date() || shiftEnd.date() == availability.date() is true if and only if there any overlap.

[zepfred]

Since both of you think getOverlappingDurationInMinutes should be calculated once, I'll accept Chris's suggestion about the filter operation.

[triceo]

I think I found a bug in the constraints.

[triceo]

LGTM after the last comment is resolved.