[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: Running 'pkd' in Nightly and Coverage Builds
[Thread Prev] | [Thread Next]
- Subject: Re: Running 'pkd' in Nightly and Coverage Builds
- From: Jon Simons <jon@xxxxxxxxxxxxx>
- Reply-to: libssh@xxxxxxxxxx
- Date: Mon, 26 Jan 2015 10:31:44 -0800
- To: libssh@xxxxxxxxxx
Ah, I am confused here: why would we need either cwrap or 'sshd' for doing the testing that 'pkd_hello' already does? 'pkd' implements the server-side half of libssh-based pubkey auth and simple data transmission. There shouldn't be any reason to need to use preload-based testing here or to deal with any 'sshd', for the codepaths that 'pkd' tests.A ssh server needs to open a port. This port is normally port 22 which can only be opened a root. We do not really want to use another port, we can't be sure that this port is in use on a system another developer runs the test.
Hmm, it seems complicated to me to introduce an outside dependency and to use preloading for this. I would think that if a unit test can't bind to some port, it should just error out, ideally with a good error message. I'm not sure I understand why we would care about using port 22 for testing. Thanks for detailing the motivation here. -Jon
Re: Running 'pkd' in Nightly and Coverage Builds | Aris Adamantiadis <aris@xxxxxxxxxxxx> |
Running 'pkd' in Nightly and Coverage Builds | Jon Simons <jon@xxxxxxxxxxxxx> |
Re: Running 'pkd' in Nightly and Coverage Builds | Andreas Schneider <asn@xxxxxxxxxxxxxx> |
Re: Running 'pkd' in Nightly and Coverage Builds | Jon Simons <jon@xxxxxxxxxxxxx> |
Re: Running 'pkd' in Nightly and Coverage Builds | Andreas Schneider <asn@xxxxxxxxxxxxxx> |