Skip to content

Commit

Permalink
Removed Test::MockObject as a test dependency.
Browse files Browse the repository at this point in the history
This was able to be done in native Perl trivially and should
not require adding a dependency to be installed.
  • Loading branch information
symkat committed Sep 9, 2014
1 parent 81fc00d commit 20a97da
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 6 deletions.
1 change: 0 additions & 1 deletion Makefile.PL
Expand Up @@ -12,7 +12,6 @@ requires 'File::HomeDir' => '0';

test_requires 'Test::More' => '0.42';
test_requires 'DBD::SQLite' => '0';
test_requires 'Test::MockObject' => '1.09';
test_requires 'Config::Any' => '0.23';

WriteAll;
12 changes: 7 additions & 5 deletions t/02_load_credentials.t
Expand Up @@ -2,12 +2,14 @@
use warnings;
use strict;
use Test::More;
use Test::MockObject;
use DBIx::Class::Schema::Config;

Test::MockObject->fake_module(
'Config::Any',
'load_stems' => sub {
{
package Config::Any;

$INC{"Config/Any.pm"} = __FILE__;

sub load_stems {
return [
{
'some_file' => {
Expand Down Expand Up @@ -40,7 +42,7 @@ Test::MockObject->fake_module(
}
]
}
);
}

my $tests = [
{
Expand Down

2 comments on commit 20a97da

@ribasushi
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is still found in t/00_load.t

@symkat
Copy link
Owner Author

@symkat symkat commented on 20a97da Sep 9, 2014

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yep, noticed that when Travis failed!

Please sign in to comment.