hetzner-cloud-php-sdk

hetzner-cloud-php-sdk copied to clipboard

This commit adds the missing fields public_net and placement_group to server create functions.

Any chance of getting this merged in and released soon? Would save me a ton of time having to maintain a fork! 🙏

Sure sorry I missed it! Could you maybe add a test too? Just to make sure it works as expected :)

The package as it stands doesn't even have tests for creating servers, so how do you know server creation works at the moment? 😉

@mikebarlow yes correct, however if there are new features it should be tried to introduce tests where they are missing :) And yes the create server part is not well testes.

The package as it stands doesn't even have tests for creating servers, so how do you know server creation works at the moment? 😉

I just used it. But you are right, tests are missing. May I add them or will the tests just stay here for an other 8 month ;)

It would be good if there are tests :) even if they are only for your change.

I just checked, you have public_net already inside your tests. Only the parameter at creation was missing.

I added the https://docs.hetzner.cloud/#placement-groups section to the api for tests with the placement group. The placement group field at the server is only an id, needs to be existing, idk what kind of test you are expecting.

Hey @cosmohdx,

sorry i was on vacation. What i expect is simply that the whole passing through is tested, so that the expected values are on the correct places :) It looks like there a a few failing tests. Could you maybe fix them? Then everything is good to go :)