Skip Menu |

This queue is for tickets about the Net-OpenSSH CPAN distribution.

Report information
The Basics
Id: 41274
Status: resolved
Priority: 0/
Queue: Net-OpenSSH

People
Owner: salva [...] cpan.org
Requestors: ambs [...] cpan.org
Cc:
AdminCc:

Bug Information
Severity: (no value)
Broken in: 0.03
Fixed in: (no value)



Subject: Tests should not try to make connections... not as root, not to localhost
Hello I didn't start using Net::OpenSSH yet but it seems promising for some projects I have in mind. But the test suite is not well suited for every installation: - Not all client machines have ssh running as a server. Thus, it is not possible to test connection to localhost. - Some other servers have SSH but does not let the root user to log in using SSH. I know this will break automatic testing but the only solution I can see at the moment is one of the three: - skip these kind of tests (put them in the distribution but not activate them with make test); - put tests as TODO, so that they do not interfere with installation - ask the user to enter a username and server where to test. In any case, any of the solutions above are better than the current one. I am sure that many of the CPAN Testers Reports you are getting will fail. Keep the good work, See you in Lisbon ;) Alberto (aka ambs)
Subject: Re: [rt.cpan.org #41274] Tests should not try to make connections... not as root, not to localhost
Date: Thu, 27 Nov 2008 15:13:59 -0800 (PST)
To: bug-Net-OpenSSH [...] rt.cpan.org
From: Salvador Fandino <sfandino [...] yahoo.com>
Ola, Show quoted text
----- Original Message ----
> From: Alberto Simões via RT <bug-Net-OpenSSH@rt.cpan.org> > Sent: Thursday, November 27, 2008 11:45:05 PM > Subject: [rt.cpan.org #41274] Tests should not try to make connections... not as root, not to localhost > > Thu Nov 27 17:45:03 2008: Request 41274 was acted upon. > Transaction: Ticket created by AMBS > Queue: Net-OpenSSH > Subject: Tests should not try to make connections... not as root, not to > localhost > Broken in: 0.03 > Severity: (no value) > Owner: Nobody > Requestors: ambs@cpan.org > Status: new > Ticket > > > Hello > > I didn't start using Net::OpenSSH yet but it seems promising for some > projects I have in mind. But the test suite is not well suited for every > installation: > > - Not all client machines have ssh running as a server. Thus, it is not > possible to test connection to localhost. > > - Some other servers have SSH but does not let the root user to log in > using SSH. > > I know this will break automatic testing but the only solution I can see > at the moment is one of the three: > - skip these kind of tests (put them in the distribution but not > activate them with make test); > - put tests as TODO, so that they do not interfere with installation > - ask the user to enter a username and server where to test. > > In any case, any of the solutions above are better than the current one. > I am sure that many of the CPAN Testers Reports you are getting will fail.
Well, the idea is to *try* to connect to localhost without password. If that works, run the test suite and if not, just skip everything. The module is still in development, very unstable and the test reports I get from the CPAN Testers are very valuable to me, even if most of them fail. Maybe, once it gets mature, I will make them fully optional... I will see how it goes, my only real concern is somebody getting annoyed by the module trying to log into its machine.
> Keep the good work, > See you in Lisbon ;)
sure, see you! - Salva