BEGIN {
$ENV{GITALIST_CONFIG} = $Bin;
- use_ok 'Catalyst::Test', 'Gitalist'
+ $ENV{GITALIST_REPO_DIR} = '';
+ use_ok 'Catalyst::Test', 'Gitalist';
}
-ok( request('/')->is_success, 'Request should succeed' );
-
-for my $p (qw/ bare.git repo1 nodescription /) {
- my $path = '/summary?p=' . $p;
+for my $p ('', qw/ repo1 nodescription bare.git opml /) {
+ my $path = '/' . $p;
ok( request($path)->is_success, "$path should succeed");
}
-is request('/summary?p=DoesNotExist')->code, 404,
- '/summary?p=DoesNotExist 404s';
+my $response = request('/DoesNotExist');
+is $response->code, 404, 'invalid repository 404s';
+like $response->content, qr/Page not found/, 'invalid repository handled correctly';
+
+
{
# URI tests for repo1
local *test = curry_test_uri('repo1');
- test('/summary');
- test('/shortlog');
- test('/log');
- test('/commit');
- test('/commitdiff', 'h=36c6c6708b8360d7023e8a1649c45bcf9b3bd818');
- test('/tree', 'h=145dc3ef5d307be84cb9b325d70bd08aeed0eceb;hb=36c6c6708b8360d7023e8a1649c45bcf9b3bd818');
-# $test_repo1->('/', 'a=blob;f=file1');
+ test('');
+ test('shortlog');
+ test('log');
+ test('reflog');
+ test('36c6c6708b8360d7023e8a1649c45bcf9b3bd818');
+ test('36c6c6708b8360d7023e8a1649c45bcf9b3bd818/tree');
+ test('36c6c6708b8360d7023e8a1649c45bcf9b3bd818/diff');
+ test('36c6c6708b8360d7023e8a1649c45bcf9b3bd818/patch');
+ test('36c6c6708b8360d7023e8a1649c45bcf9b3bd818/patches/1');
+ test('36c6c6708b8360d7023e8a1649c45bcf9b3bd818/patches/2');
+
+ TODO: {
+ local $TODO = "FIXME";
+ test('search', 'type=commit&text=added');
+
+
+ test('blob', 'f=dir1/file2;hb=36c6c6708b8360d7023e8a1649c45bcf9b3bd818');
+
+ # FIXME - What's the difference here?
+ #test('patch', 'h=3f7567c7bdf7e7ebf410926493b92d398333116e');
+ #test('patch', 'h=3f7567c7bdf7e7ebf410926493b92d398333116e;hp=3bc0634310b9c62222bb0e724c11ffdfb297b4ac');
+ #test('patches', 'h=3f7567c7bdf7e7ebf410926493b92d398333116e');
+ #test('patches', 'h=3f7567c7bdf7e7ebf410926493b92d398333116e;hp=3bc0634310b9c62222bb0e724c11ffdfb297b4ac');
+ }
}
done_testing;
sub test_uri {
- my ($p, $uri, $qs) = @_;
+ my ($uri, $qs) = @_;
$qs ||= '';
- my $request = "$uri?p=repo1;$qs";
- warn("request: $request");
+ my $request = "/$uri";
+ $request .= "?$qs" if defined $qs;
my $response = request($request);
- ok($response->is_success, "ok $p - $uri");
+ ok($response->is_success, "ok $uri - $qs");
+ return $response;
}
sub curry_test_uri {
- my $p = shift;
+ my $prefix = shift;
+ my $to_curry = shift || \&test_uri;
sub {
my $uri = shift;
- test_uri($p, $uri);
+ $to_curry->("$prefix/$uri", @_);
};
-};
+}