Missing server create fields by cosmohdx ยท Pull Request #94 ยท LKDevelopment/hetzner-cloud-php-sdk
Any chance of getting this merged in and released soon?
Would save me a ton of time having to maintain a fork! ๐
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 ;)
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 :)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters