add filename in log error

peczenyj · Dec 30, 2011 · 1f4b7eb · 1f4b7eb
1 parent 53297bb
commit 1f4b7eb
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 7 deletions.
diff --git a/Uplpipe-Agent/bin/uplpipe.daemon.pl b/Uplpipe-Agent/bin/uplpipe.daemon.pl
@@ -1,6 +1,7 @@
 use common::sense;
 use Uplpipe::Agent;
 use Daemon::Generic;
+use Config::Tiny;
 use POE;
 use POE::Component::DirWatch;
 use Log::Log4perl;

diff --git a/Uplpipe-Agent/etc/uplpipe.conf b/Uplpipe-Agent/etc/uplpipe.conf
@@ -7,8 +7,7 @@ log4perl.appender.LOGFILE.layout=PatternLayout
 log4perl.appender.LOGFILE.layout.ConversionPattern=[%d :: %c :: %p] %m%n
 
 [amazon]
-aws_access_key_id=example_aws_access_key_id
-aws_secret_access_key=example_aws_secret_access_key
+
 retry=1
 
 [app]

diff --git a/Uplpipe-Agent/t/02-basics.t b/Uplpipe-Agent/t/02-basics.t
@@ -219,7 +219,7 @@ subtest "failure in unlink file" => sub {
 	ok(! $result, "process status should be true");
 	ok_s3($s3);
 	ok_ua($ua);
-	ok_logger_error($logger, 'one or more errors: No such file or directory');	
+	ok_logger_error($logger, 'one or more errors for file /not/exist/a: No such file or directory');	
 };
 
 sub get_ua_invalid{
@@ -263,7 +263,7 @@ subtest "failure in update rest api" => sub {
 	ok(! $result, "process status should be true");
 	ok_s3($s3);
 	ok_ua_invalid($ua);
-	ok_logger_error($logger, 'one or more errors: ops');	
+	ok_logger_error($logger, 'one or more errors for file /not/exist/a: ops');	
 };
 
 sub get_s3_invalid {
@@ -314,7 +314,7 @@ subtest "failure in update acl in s3" => sub {
 	ok(! $result, "process status should be true");
 	ok_s3_invalid($s3);
 	ok_ua_not_call($ua);
-	ok_logger_error($logger, 'one or more errors: >err:>errstr');	
+	ok_logger_error($logger, 'one or more errors for file /not/exist/a: >err:>errstr');	
 };
 
 sub get_s3_invalid2{
@@ -366,7 +366,7 @@ subtest "upload successfull but head_key shows different etag/md5" => sub {
 	ok(! $result, "process status should be false");
 	ok_s3_invalid2($s3);
 	ok_ua_invalid2($ua);
-	ok_logger_error($logger, 'one or more errors: store a in s3 but check file returns false');
+	ok_logger_error($logger, 'one or more errors for file /not/exist/a: store a in s3 but check file returns false');
 };
 
 sub get_s3_invalid3{
@@ -405,7 +405,7 @@ subtest "failure to upload to s3" => sub {
 	ok(! $result, "process status should be false");
 	ok_s3_invalid2($s3);
 	ok_ua_invalid2($ua);
-	ok_logger_error($logger, 'one or more errors: >err:>errstr');	
+	ok_logger_error($logger, 'one or more errors for file /not/exist/a: >err:>errstr');	
 };
 
 done_testing();