New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix setting up fixtures in t/direct/rest.t #2

Open
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
1 participant
@skington

skington commented Mar 2, 2015

I got assigned this class for the Pull Request challenge, so after resolving the dependencies issue, set about to the test failures that I found.

I haven't delved very deeply into the code, but it appears that the failure was about the inability to create multiple records in one POST call. If it ever worked, idioms like

my $row = $form->model->create;

in CatalystX::Controller::ExtJS::Rest suggest that that's no longer possible.

So the fix is simply to do three calls rather than one; once that's done, the tests pass happily.

This is the setup I've been using, although I found similar errors on other Perl versions:

Generated by Dist::Zilla::Plugin::ReportVersions::Tiny v1.10

perl: 5.018002 (wanted any version) on linux from /opt/perlbrew/perls/perl-5.18.2/bin/perl

Catalyst::Action::REST => 1.19 (want 0.88)

Catalyst::Action::RenderView => 0.16 (want any version)

Catalyst::Runtime => 5.90084 (want 5.80024)

CatalystX::ExtJS::Direct => 2.1.5 (want v2.1.5)

DBIx::Class => 0.082810 (want 0.08127)

HTML::Entities => 3.69 (want any version)

HTML::FormFu::ExtJS => 0.090 (want any version)

HTML::FormFu::Model::DBIC => 2.00 (want 0.08)

JavaScript::Dumper => 0.011 (want 0.011)

Lingua::EN::Inflect => 1.895 (want any version)

Module::Build => 0.4211 (want 0.28)

Moose => 2.1403 (want 1.21)

Path::Class => 0.35 (want any version)

Test::More => 1.001014 (want 0.88)

Try::Tiny => 0.22 (want any version)

If it ever worked, this attempt to create multiple records in one POS…
…T call is now failing, so explicitly create records individually. Also fix a cut-and-paste error in a test name.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment