summaryrefslogtreecommitdiff
path: root/spec
Commit message (Collapse)AuthorAgeFilesLines
* Move spec to proper place and fix unused variablerefactor-complex-methodsDmitriy Zaporozhets2015-11-131-0/+0
| | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* Merge pull request #9820 from huacnlee/avoid-render-form-in-notes-listDmitriy Zaporozhets2015-11-122-9/+0
|\ | | | | Avoid render edit_form in each notes.
| * Avoid render edit_form in each notes.Jason Lee2015-11-122-9/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Use RJS to render edit note feature. Before: ``` Rendered projects/notes/_note.html.haml (27.9ms) Rendered projects/_zen.html.haml (0.3ms) Rendered projects/notes/_hints.html.haml (0.7ms) Rendered projects/_md_preview.html.haml (3.9ms) Rendered projects/notes/_edit_form.html.haml (6.9ms) Rendered projects/notes/_note.html.haml (17.7ms) Rendered projects/_zen.html.haml (0.3ms) Rendered projects/notes/_hints.html.haml (0.6ms) Rendered projects/_md_preview.html.haml (3.4ms) Rendered projects/notes/_edit_form.html.haml (7.0ms) ``` After: ``` Rendered projects/notes/_note.html.haml (13.8ms) Rendered projects/notes/_note.html.haml (7.1ms) Rendered projects/notes/_note.html.haml (9.5ms) Rendered projects/notes/_note.html.haml (8.5ms) ``` This change reduce at least 6ms * N ('N' - number of notes).
* | Merge branch 'caches' into 'master' Kamil Trzciński2015-11-121-0/+82
|\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Allow to define cache in `.gitlab-ci.yml` This extends `.gitlab-ci.yml` syntax to allow specifying caching files and directories between builds, making it easy to preserve ex. gems. ``` cache: paths: - .bundle - vendor/ before_script: - bundle install --path vendor/ rspec: script: - bundle exec rspec ``` This is based on Build Artifacts changes. /cc@dzaporozhets See merge request !1786
| * | Allow to define cache in `.gitlab-ci.yml`cachesKamil Trzcinski2015-11-101-0/+82
| | |
* | | Merge branch 'artifacts' into 'master' Kamil Trzciński2015-11-117-15/+312
|\ \ \ | |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Implement Build Artifacts This implements #3028 1. It stores artifacts in shared/artifacts, 1. It adds `artifacts` to `.gitlab-ci.yml`, 1. We use GitLab Workhorse to offload artifacts uploading, 1. To download artifacts it uses GitLab Workhorse X-Sendfile extension, 1. There's one "artifact" per-build. The new upload removes previous one and creates a new one, 1. Default max artifact size is set to 100MB - this can be changed in settings. Missing things: 1. Support for `.gitlab-ci.yml`: `artifacts: true or git-ls-files` which will upload all non tracked files, 1. Artifacts passing between builds. GitLab Workhorse changes: https://gitlab.com/gitlab-org/gitlab-workhorse/merge_requests/5 GitLab Runner changes: https://gitlab.com/gitlab-org/gitlab-ci-multi-runner/merge_requests/46 Syntax: ``` artifacts: untracked: true # default: false paths: # default: empty - bin/files ``` See merge request !1584
| * | Fix testsartifactsKamil Trzcinski2015-11-101-2/+2
| | |
| * | Use normal file upload mechanism to upload artifactsKamil Trzcinski2015-11-101-8/+17
| | |
| * | Move tmp artifacts to shared/artifacts/tmp/. Check for GitLab-Workhorse nowKamil Trzcinski2015-11-101-1/+1
| | |
| * | Change artifacts syntax to allow uploading untracked filesKamil Trzcinski2015-11-101-5/+18
| | |
| * | Implement Build ArtifactsKamil Trzcinski2015-11-107-15/+290
| |/ | | | | | | | | | | | | - Offloads uploading to GitLab Workhorse - Use /authorize request for fast uploading - Added backup recipes for artifacts - Support download acceleration using X-Sendfile
* | Merge branch 'add-allow-failure-status' into 'master' Stan Hu2015-11-101-1/+3
|\ \ | |/ |/| | | | | | | | | Add allow_failure field to commit status API Closes #3196 See merge request !1685
| * Add allow_failure field to commit status APIStan Hu2015-11-031-1/+3
| | | | | | | | Closes #3196
* | Merge remote-tracking branch 'origin/release-notes'Dmitriy Zaporozhets2015-11-102-0/+25
|\ \ | | | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
| * | Add tests to release notes featureDmitriy Zaporozhets2015-11-062-4/+15
| | | | | | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
| * | Add release modelDmitriy Zaporozhets2015-11-052-0/+14
| | | | | | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* | | Track the amount of times views are renderedYorick Peterse2015-11-091-0/+57
| | |
* | | Fixed Hash key style in Sherlock::Query specYorick Peterse2015-11-091-1/+1
| | |
* | | Added specs and source documentation for SherlockYorick Peterse2015-11-098-0/+639
| | |
* | | Merge branch 'fix-commits-manual-merge' into 'master' Douwe Maan2015-11-091-0/+19
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix bug where manually merged branches in a MR would end up with an empty diff This bug manifested in 8.1 with the refactoring of `RefreshService`. Here's what happens: 1. User create a new branch `foo`. 2. User creates a merge request for `foo`. 3. User merges `foo` into `master` by hand. 4. `RefreshService` reloads the merge request. Since `master` is equivalent to `foo`, this results in an empty diff. 5. `RefreshService` then closes the MR. This wasn't an issue when you use the normal "Accept Merge Request" flow because the act of clicking the button closes the merge request, so step 4 never happens. Closes #3314 See merge request !1758
| * | | Add spec for manual merge of merge requestStan Hu2015-11-061-0/+19
| | | |
* | | | Fix testsshared_runnersKamil Trzcinski2015-11-083-3/+31
| | | |
* | | | CI details cleanupKamil Trzcinski2015-11-053-7/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | - Add page titles to CI settings. - Fix CI admin navigation. - Remove duplicated scope. - Use monospace font for commit sha. - Add page title and header title to build page. - Proper authorization for cancel/retry builds. - Use gitlab pagination theme for builds and group members. - Don't paginate builds widget on build page. - Add badges to commit page Changes/Builds tabs. - Add "Builds" to commit Builds tab page title. - Add and use Ci::Build#retryable? method. - Add CI::Build#retried? method. - Allow all failed commit builds to be retried. - Proper authorization for cancel/retry all builds. - Remove unused param. - Use time_ago_with_tooltip where appropriate. - Tweak builds index text - Remove duplication between builds/build and commit_statuses/commit_status. - Use POST rather than GET for canceling and retrying builds. - Remove redundant URL helpers. - Add build ID to build page. - Link branch name on build page. - Move commit/:sha/ci to commit/:sha/builds.
* | | | Allow groups to appear in the search results if the group owner allows itValery Sizov2015-11-053-0/+39
| |/ / |/| |
* | | Merge branch 'rs-git-bin-path' into 'master' Douwe Maan2015-11-043-7/+7
|\ \ \ | |/ / |/| | | | | | | | | | | | | | Replace all usages of `git` command with configurable binary path Closes #3311 See merge request !1742
| * | Replace all usages of `git` command with configurable binary pathrs-git-bin-pathRobert Speicher2015-11-033-7/+7
| |/ | | | | | | Closes #3311
* | Merge branch 'create-project-performance' into 'master' Yorick Peterse2015-11-041-0/+28
|\ \ | | | | | | | | | | | | | | | | | | Improve performance of creating projects See merge request !1650
| * | Added benchmark for Projects::CreateServiceYorick Peterse2015-10-291-0/+28
| | | | | | | | | | | | | | | This benchmark currently runs at ~0.6 iterations per second and is unlikely to perform any better any time soon.
* | | Merge branch 'remove-duplicate-orders' into 'master' Yorick Peterse2015-11-042-6/+16
|\ \ \ | |_|/ |/| | | | | | | | | | | | | | | | | | | | Remove duplicate orders This changes the default order from `created_at DESC, id DESC` to just `id DESC` as this achieves the same results without the overhead of having to sort data twice (we've seen queries go from 200ms to just a few ms by removing the double sort). cc @jacobvosmaer @dzaporozhets @rspeicher @DouweM See merge request !1735
| * | Fixed User sorting specsremove-duplicate-ordersYorick Peterse2015-11-031-6/+6
| | | | | | | | | | | | | | | The descriptions were not accurate and one particular spec seemingly expected the wrong User row to be returned.
| * | Added benchmark for User.allYorick Peterse2015-11-031-0/+10
| | | | | | | | | | | | | | | This benchmark exists to test if ordering has any noticeable impact in the test environment.
* | | Merge branch 'drop-old-builds' into 'master' Kamil Trzciński2015-11-031-0/+44
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Cleanup stuck CI builds Fixes #3143 /cc @dzaporozhets @jacobvosmaer See merge request !1655
| * | | Cleanup stuck CI builds dailydrop-old-buildsKamil Trzcinski2015-11-031-0/+44
| | | |
* | | | Merge branch 'web_hook_repo_changes'Valery Sizov2015-11-031-0/+6
|\ \ \ \
| * | | | Add added, modified and removed properties to commit object in webhookweb_hook_repo_changesValery Sizov2015-11-031-0/+6
| | | | |
* | | | | Merge branch 'only-syntax' into 'master' Kamil Trzciński2015-11-031-79/+220
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Extend yml syntax for only and except to support specifying repository path This allows to limit execution of jobs to specific repository. For example: ```yaml job: only: - branches@gitlab-org/gitlab-ce except: - master@gitlab-org/gitlab-ce ``` The above will run `job` for all branches on `gitlab-org/gitlab-ce`, except master. @dzaporozhets @JobV @vsizov Please review. See merge request !1720
| * | | | | Fix testsonly-syntaxKamil Trzcinski2015-11-031-15/+11
| | | | | |
| * | | | | Extend yml syntax for only and except to support specifying repository pathKamil Trzcinski2015-11-021-79/+224
| | | | | |
* | | | | | Fix code that depends on incorrect inflector behaviorDmitriy Zaporozhets2015-11-031-1/+1
| |_|/ / / |/| | | | | | | | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
* | | | | Merge branch 'olhado/gitlab-ce-commit-search'Dmitriy Zaporozhets2015-11-031-0/+9
|\ \ \ \ \ | |_|/ / / |/| | | |
| * | | | Refactor search by commits messageolhado/gitlab-ce-commit-searchDmitriy Zaporozhets2015-11-021-2/+2
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
| * | | | Actually converted code to following suggestions.Michael Chmielewski2015-10-281-1/+1
| | | | |
| * | | | Fixed method to use git log via Popen as recommended, and made output match ↵Michael Chmielewski2015-10-281-1/+1
| | | | | | | | | | | | | | | | | | | | test (and thus system) expectations.
| * | | | Trying to incorporate suggestions from comments on Merge Request 1661Michael Chmielewski2015-10-281-2/+2
| | | | |
| * | | | Trailing new lines at ends of files are important.Michael Chmielewski2015-10-281-1/+1
| | | | |
| * | | | Make subject of spec line up on one line to satisfy rubocop requirementsMike Chmielewski2015-10-281-4/+2
| | | | |
| * | | | Add support for searching commit log messagesJonathan Schoeffling2015-10-281-0/+11
| | |_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | Include the log messages of recent commits in project-level search results, providing functionality similar to 'git log --grep'. Update repository model rspec tests to validate the output of Repository#commits_with_log_matching.
* | | | Merge branch 'api_file_touched_at' into 'master' Dmitriy Zaporozhets2015-11-021-0/+1
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add ability to fetch the commit ID of the last commit that actually touched a file https://dev.gitlab.org/gitlab/gitlabhq/issues/2564 See merge request !1718
| * | | | Add ability to fetch the commit ID of the last commit that actually touched ↵api_file_touched_atValery Sizov2015-11-021-0/+1
| | |_|/ | |/| | | | | | | | | | a file
* | | | Merge branch 'backup-improvements' into 'master' Kamil Trzciński2015-11-021-7/+8
|\ \ \ \ | |_|_|/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Reduce disk IO and space usage during backups This is based on improvements made to the GitLab CI 8.0 backup script. - Avoid creating many small intermediate files while backing up builds and uploads by using tar and light gzip compression - Use same backup/restore code for uploads and builds - Only store a compressed intermediate DB dump See merge request !1520