Commit graph

15 commits

Author SHA1 Message Date
Lin Jen-Shin
e65bc0f175 Path could also have slashes! Feedback:
https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/5142#note_14347729
2016-08-24 16:02:56 +08:00
Lin Jen-Shin
bc3493f947 Use only one before block, feedback:
https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/5142/diffs#note_14347758
2016-08-24 14:40:18 +08:00
Lin Jen-Shin
75df5f6c73 Fixed a missing rename 2016-08-17 13:42:06 +08:00
Lin Jen-Shin
abf1cffff8 Fix tests, explicitly set the status 2016-08-16 00:46:51 +08:00
Lin Jen-Shin
cc3dbf83f4 Empty lines around blocks 2016-08-10 18:57:51 +08:00
Lin Jen-Shin
fa02d8fcc5 Merge shared context into controller test and update accordingly 2016-07-21 19:09:46 +08:00
Lin Jen-Shin
85409a5a10 Use ci_commits.ref (Pipeline#ref) to find builds 2016-07-18 20:10:50 +08:00
Lin Jen-Shin
2e90abf254 It could be redirecting or downloading in Rails or API 2016-07-15 01:45:04 +08:00
Lin Jen-Shin
d7bbee7593 Update routes based on feedback from:
https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/5142#note_13058785

And note that job/build_name could contain `/`
2016-07-14 16:45:46 +08:00
Lin Jen-Shin
5b227f351f rubocop likes this better 2016-07-13 23:39:03 +08:00
Lin Jen-Shin
b043729d49 Share more stuffs 2016-07-13 23:23:05 +08:00
Lin Jen-Shin
8735d95af1 Implement API for downloading artifacts from ref and build name:
Basically:

    GET /api/projects/:id/artifacts/:ref_name/:build_name

Also added tests for it.
2016-07-13 23:07:37 +08:00
Lin Jen-Shin
6c80b597f5 Introduce path_from_ref and save some typing 2016-07-13 21:44:27 +08:00
Lin Jen-Shin
3336828152 No need for a separate line now 2016-07-13 17:49:42 +08:00
Lin Jen-Shin
e383254070 Add all the tests and fix stuffs along the way:
It turns out they are different:

    builds.success.latest.first

and

    builds.latest.success.first

If we put success first, that latest would also filter via success,
and which is what we want here.
2016-07-12 23:15:08 +08:00