Use the read method on the return of open_url to actually get the con…

[gerases]

Summary

Use the read method on the return of open_url to actually get the content. Otherwise the following error is returned by puppetserver:

Error: Could not retrieve catalog from remote server: Error 500 on SERVER: Server Error: Evaluation Error: Error while evaluating a Function Call,
 no implicit conversion of Tempfile into String (file: <file_path>, line: 8, column: 15) on node <node_name>

[CLAassistant]

All committers have signed the CLA.

[gerases]

I am not found of doubles, but something like:

Yep, will do within a few days, thank you. Do you agree it's a bug though?

[smortex]

Yep, will do within a few days, thank you.

Thanks!

Do you agree it's a bug though?

Yup!

[gerases]

@smortex, I decided to use StringIO in the tests. I'm not sure how to restart the build though.

[gerases]

Also I see there's no way to test using a docker image, which is a bit of a hassle. Is there a plan to add that similar to other repos of vox pupuli? By the way, why "pupuli" (which is incorrect) and not "populi"?

[bastelfreak]

@gerases puppetlabs/stdlib uses Litmus to spin up SUTs (systems under test), whereas Vox Pupuli uses Beaker. But as Beaker, Litmus supports Docker (and Podman I think?) Containers for SUTs. I just don't know how to use Litmus.

And Vox Pupuli is correct, it's just not 'people', but 'puppet': https://latin-dictionary.net/search/english/puppet

[gerases]

And Vox Pupuli is correct, it's just not 'people', but 'puppet': https://latin-dictionary.net/search/english/puppet

Oh, fantastic then. Didn't think of "pupulus". Thanks.

[gerases]

@bastelfreak Could this be merged?

[bastelfreak]

I'm a bit hesitant on merging because I don't know if this change breaks the setup for other users. I need to test it and I currently don't have the time.

[ekohl]

I'm surprised I didn't link it, but https://github.com/puppetlabs/puppetlabs-stdlib/pull/1415 was my attempt to address it.