Add Failure Policy for Jobs SDK

[siri-varma]

Description

Add Failure Policy for the Job

Issue reference

We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.

Please reference the issue this PR will close: #1429

Checklist

Please make sure you've completed the relevant tasks for this PR, out of the following list:

• Code compiles correctly
• Created/updated tests
• Extended the documentation

[siri-varma]

The integration test will fail because a version of dapr that supports failures policy has not been released yet.

[artur-ciocanu]

@siri-varma nice patch! I have added some comments, mostly around naming convention.

[artur-ciocanu]

@siri-varma could you please check why the some tests are failing:

Error:  Errors: 
Error:    BindingIT.httpOutputBindingError:46 » Runtime org.opentest4j.AssertionFailedError: expected: <404> but was: <403>
Error:    BindingIT.httpOutputBindingErrorIgnoredByComponent:69 » Runtime org.opentest4j.AssertionFailedError: expected: <404> but was: <403>
Error:    DaprJobsIT.testJobScheduleCreationWithConstantFailurePolicy:194 NullPointer Cannot invoke "io.dapr.client.domain.FailurePolicy.getFailurePolicyKind()" because the return value of "io.dapr.client.domain.GetJobResponse.getFailurePolicy()" is null
Error:    DaprJobsIT.testJobScheduleCreationWithDropFailurePolicy:172 NullPointer Cannot invoke "io.dapr.client.domain.FailurePolicy.getFailurePolicyKind()" because the return value of "io.dapr.client.domain.GetJobResponse.getFailurePolicy()" is null

[siri-varma]

@artur-ciocanu Once a new version with the required protos comes out, the integration tests should succeed.

[siri-varma]

@dapr/approvers-java-sdk This PR is ready for review. Thank you.

[siri-varma]

Fixed all the merge conflicts

[artur-ciocanu]

@siri-varma overall looks good, but I think the code that deals with failure policy could be a little bit simple. Ideally you should use helper methods and ensure that you leverage early returns to avoid all those nested if/else.

[siri-varma]

@artur-ciocanu Addressed all the comments

[artur-ciocanu]

Nice patch! Thank you.

[codecov]

Codecov Report

❌ Patch coverage is 98.03922% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 78.73%. Comparing base (d759c53) to head (28017f8).
⚠️ Report is 190 commits behind head on master.

Files with missing lines	Patch %	Lines
...ain/java/io/dapr/client/domain/GetJobResponse.java	66.66%	1 Missing ⚠️

Additional details and impacted files

@@             Coverage Diff              @@
##             master    #1448      +/-   ##
============================================
+ Coverage     76.91%   78.73%   +1.81%     
- Complexity     1592     1918     +326     
============================================
  Files           145      237      +92     
  Lines          4843     5924    +1081     
  Branches        562      610      +48     
============================================
+ Hits           3725     4664     +939     
- Misses          821      938     +117     
- Partials        297      322      +25     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.