MODULES-11049 - Implement default privileges changes

[mtancoigne]

Following the feature request, this PR allow to grant or revoke default privileges on objects in a schema.

I still have to update the documentation.

[puppet-community-rangefinder]

postgresql::server::default_privileges is a type

that may have no external impact to Forge modules.

This module is declared in 70 of 576 indexed public Puppetfiles.

---

These results were generated with Rangefinder, a tool that helps predict the downstream impact of breaking changes to elements used in Puppet modules. You can run this on the command line to get a full report.

Exact matches are those that we can positively identify via namespace and the declaring modules' metadata. Non-namespaced items, such as Puppet 3.x functions, will always be reported as near matches only.

[sheenaajay]

Thanks @mtancoigne for your contribution. The tests are failing on some OS with the following error
Default_privileges is only useable with PostgreSQL >= 9.6

The default version on each OS is given here https://github.com/puppetlabs/puppetlabs-postgresql/blob/main/manifests/globals.pp#L167

We may have to restrict the tests not to run on those OSes which have defaults <9.6

[mtancoigne]

Good to know, thank you, i'll update the PR!

[sheenaajay]

Thanks @mtancoigne for updating the tests. The unit tests are failing.
https://github.com/puppetlabs/puppetlabs-postgresql/pull/1267/checks?check_run_id=2543787040#step:10:37
Could you please take a look.
Meanwhile testing the default privileges changes locally. Thank you.

[mtancoigne]

I updated the tests and will squash the commits once it's all green.

[sheenaajay]

Thanks @mtancoigne for the changes. Tested for different object types and its looking good.

As you have mentioned, could you please update the document for this new feature.

Great work.Thank you.

[mtancoigne]

You're welcome, I'll try to update documentation as soon as possible this week.

[mtancoigne]

README was updated and commits were squashed.

[sheenaajay]

Could you please add the summary and parameter details which will help with autogenerating REFERENCE.md file prior to each release.
For example
https://github.com/puppetlabs/puppetlabs-postgresql/blob/main/manifests/server/database_grant.pp#L1
https://github.com/puppetlabs/puppetlabs-postgresql/blob/main/REFERENCE.md

[mtancoigne]

Of course, i'll do it this week.

[sheenaajay]

Thanks alot @mtancoigne for incorporating the comments

[mtancoigne]

It's done.

[sheenaajay]

Thank you @mtancoigne for your contribution.

[ekohl]

This is wrong. A unit test should not include the acceptance helper and has broken the test suite further than it already was.