Fix bug that disallowed JSON flags in unity-catalog recipients update command. Added delta sharing CLI tests

[wchau]

Tested locally and through added tests.

Some other minor fixes:

• Disallowed create provider without recipient profile
• Rotate token expiration should be an integer, even though server can parse it fine. It makes the help text comprehensible.

[codecov-commenter]

Codecov Report

Merging #549 (30a6648) into main (8bf8310) will increase coverage by 5.30%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #549      +/-   ##
==========================================
+ Coverage   64.74%   70.04%   +5.30%     
==========================================
  Files          55       55              
  Lines        4731     4731              
==========================================
+ Hits         3063     3314     +251     
+ Misses       1668     1417     -251     

Impacted Files	Coverage Δ
databricks_cli/unity_catalog/delta_sharing_cli.py	84.08% <ø> (+84.08%)	⬆️
databricks_cli/unity_catalog/uc_service.py	27.51% <0.00%> (-0.22%)	⬇️
databricks_cli/unity_catalog/metastore_cli.py	0.00% <0.00%> (ø)
databricks_cli/utils.py	93.75% <0.00%> (+4.16%)	⬆️
databricks_cli/unity_catalog/utils.py	46.15% <0.00%> (+15.38%)	⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

[pietern]

The flag defaults to None though (in the @click.option above). Do you see something else?

[pietern]

It would be great if you can add a test for this. Looking at the implementation is also does a couple other unexpected things; like update the comment and and owner to nothing if they aren't specified. With a couple tests this could have been caught.

cc @adamcain-db

[wchau]

It would be great if you can add a test for this. Looking at the implementation is also does a couple other unexpected things; like update the comment and and owner to nothing if they aren't specified. With a couple tests this could have been caught.

cc @adamcain-db

Add tests for all delta sharing CLI commands. Those things are expected; the nulls will not negate the fields.

[pietern]

Thanks for adding tests!

Those things are expected; the nulls will not negate the fields.

Understood. This is not what I would expect though, and what other APIs implement. Typically null means remove the field, per https://datatracker.ietf.org/doc/html/rfc7386.