mention that the file method can return undef

mojolicious · Nov 29, 2014 · b279513 · b279513
1 parent cfbb272
commit b279513
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 2 deletions.
diff --git a/lib/Mojolicious/Static.pm b/lib/Mojolicious/Static.pm
@@ -210,8 +210,9 @@ Serve static file for L<Mojolicious::Controller> object.
   my $asset = $static->file('../lib/MyApp.pm');
 
 Build L<Mojo::Asset::File> or L<Mojo::Asset::Memory> object for a file,
-relative to L</"paths"> or from L</"classes">. Note that this method does not
-protect from traversing to parent directories.
+relative to L</"paths"> or from L</"classes">, or return C<undef> if it
+doesn't exist. Note that this method does not protect from traversing to
+parent directories.
 
   my $content = $static->file('foo/bar.html')->slurp;
 

diff --git a/t/mojolicious/app.t b/t/mojolicious/app.t
@@ -57,6 +57,7 @@ is_deeply $t->app->commands->namespaces,
 is $t->app, $t->app->commands->app, 'applications are equal';
 is $t->app->static->file('hello.txt')->slurp,
   "Hello Mojo from a development static file!\n", 'right content';
+is $t->app->static->file('does_not_exist.html'), undef, 'no file';
 is $t->app->moniker, 'mojolicious_test', 'right moniker';
 is $t->app->secrets->[0], $t->app->moniker, 'secret defaults to moniker';
 is $t->app->renderer->template_handler(

diff --git a/t/mojolicious/production_app.t b/t/mojolicious/production_app.t
@@ -32,6 +32,7 @@ is_deeply $t->app->commands->namespaces,
 is $t->app, $t->app->commands->app, 'applications are equal';
 is $t->app->static->file('hello.txt')->slurp,
   "Hello Mojo from a static file!\n", 'right content';
+is $t->app->static->file('does_not_exist.html'), undef, 'no file';
 is $t->app->moniker, 'mojolicious_test', 'right moniker';
 
 # Default namespaces