fix(opensearch): handle non existing SAMLOptions in domain

[MrCloudSec]

Description

Handle non existing SAMLOptions in OpenSearch Domain.

License

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 86.31%. Comparing base (ee178c2) to head (d6f2ab0).
Report is 450 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #4086      +/-   ##
==========================================
- Coverage   86.39%   86.31%   -0.09%     
==========================================
  Files         790      790              
  Lines       24729    24729              
==========================================
- Hits        21364    21344      -20     
- Misses       3365     3385      +20     

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

[pedrooot]

LGTM!

[jfagoagas]

Could you please add get() for all the calls in the function? Thanks!

[MrCloudSec]

Could you please add get() for all the calls in the function? Thanks!

At which calls do you refer? Remember that we have to do this one by one because since we have to make sure to handle each specific case correctly.

[jfagoagas]

Could you please add get() for all the calls in the function? Thanks!

At which calls do you refer? Remember that we have to do this one by one because since we have to make sure to handle each specific case correctly.

I meant for all the calls within that function but it's good for now.

[jfagoagas]

🔝