Skip Menu |

This queue is for tickets about the Amazon-SQS-Simple CPAN distribution.

Report information
The Basics
Id: 104224
Status: resolved
Priority: 0/
Queue: Amazon-SQS-Simple
People
Owner: PENFOLD [...] cpan.org
Requestors: simon.proctor [...] gmail.com
Cc:
AdminCc:
Bug Information
Severity: Important
Broken in: 2.04
Fixed in: 2.05

History Show all quoted text
  Tue May 05 04:51:27 2015 https://www.google.com/accounts/o8/id?id=AItOawksNL5OJ4luv2sxGrtiS7isfIXDnCONDVM - Ticket created
Subject: Update to 2.04 broken tests
The patch to update the constructors in 2.04 has unfortunately broken some of the unit tests meaning you can't install the module without skipping testing. Looking at the tests dev-methods.t, dev-methods-batch.t and dev-methods-regression-2008-01-01.t they are expecting $ENV{AWS_ACCESS_KEY} and $ENV{AWS_SECRET_KEY} to be set. I think the easiest plan would be to skip these tests if the environment variables aren't set.
  Wed May 06 03:16:35 2015 https://www.google.com/accounts/o8/id?id=AItOawksNL5OJ4luv2sxGrtiS7isfIXDnCONDVM - Correspondence added
Actually on further examination the issue seems to be there's a whole load of new unit tests in 2.04.
  Wed May 06 03:16:36 2015 The RT System itself - Status changed from 'new' to 'open'
  Wed Sep 07 07:20:42 2016 osfameron [...] cpan.org - Correspondence added
On Tue May 05 04:51:27 2015, https://www.google.com/accounts/o8/id?id=AItOawksNL5OJ4luv2sxGrtiS7isfIXDnCONDVM wrote: Show quoted text
> The patch to update the constructors in 2.04 has unfortunately broken > some of the unit tests meaning you can't install the module without > skipping testing. > > Looking at the tests dev-methods.t, dev-methods-batch.t and dev- > methods-regression-2008-01-01.t they are expecting > $ENV{AWS_ACCESS_KEY} and $ENV{AWS_SECRET_KEY} to be set. > > I think the easiest plan would be to skip these tests if the > environment variables aren't set.
Agreed, this is biting us too. Should be skipped or at the very list documented in installation instructions.
  Mon Jan 23 07:24:19 2017 PENFOLD [...] cpan.org - Correspondence added
On Wed Sep 07 07:20:42 2016, OSFAMERON wrote: Show quoted text
> On Tue May 05 04:51:27 2015, > https://www.google.com/accounts/o8/id?id=AItOawksNL5OJ4luv2sxGrtiS7isfIXDnCONDVM > wrote:
> > The patch to update the constructors in 2.04 has unfortunately broken > > some of the unit tests meaning you can't install the module without > > skipping testing. > > > > Looking at the tests dev-methods.t, dev-methods-batch.t and dev- > > methods-regression-2008-01-01.t they are expecting > > $ENV{AWS_ACCESS_KEY} and $ENV{AWS_SECRET_KEY} to be set. > > > > I think the easiest plan would be to skip these tests if the > > environment variables aren't set.
> > Agreed, this is biting us too. Should be skipped or at the very list > documented in installation instructions.
Tests skip in 2.05. (Actually, make dist shouldn't include them.)
  Mon Jan 23 07:24:19 2017 PENFOLD [...] cpan.org - Status changed from 'open' to 'resolved'
  Mon Jan 23 07:24:25 2017 PENFOLD [...] cpan.org - Taken
  Mon Jan 23 07:24:25 2017 PENFOLD [...] cpan.org - Fixed in 2.05 added