summaryrefslogtreecommitdiff
path: root/spec/lib
Commit message (Collapse)AuthorAgeFilesLines
...
| * | | | Fix rest of rspec and spinach testsKamil Trzcinski2016-04-121-3/+12
| | | | |
| * | | | Merge remote-tracking branch 'origin/master' into ci-commit-as-pipelineKamil Trzcinski2016-04-112-11/+14
| |\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | # Conflicts: # app/views/projects/commits/_commit.html.haml
| * | | | | Fix specsKamil Trzcinski2016-04-111-3/+3
| | | | | |
* | | | | | Fix setting of "action" for Grape transactionsYorick Peterse2016-04-201-0/+24
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Merely setting the "action" tag will only result in the transaction itself containing a value for this tag. To ensure other metrics also contain this tag we must set the action using Transaction#action= instead.
* | | | | | Fix wiki testsSebastian Klier2016-04-201-0/+9
| |_|_|_|/ |/| | | |
* | | | | Merge branch 'make-before-after-overridable' into 'master' Kamil Trzciński2016-04-191-3/+70
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Make before_script and after_script overridable This is makes it possible to overwrite the before_script and after_script at job level. This is continuation of https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/3771 See merge request !3772
| * | | | | Make rubocop happymake-before-after-overridableKamil Trzcinski2016-04-181-8/+8
| | | | | |
| * | | | | Merge branch 'after-script' into make-before-after-overridableKamil Trzcinski2016-04-182-30/+109
| |\ \ \ \ \
| * \ \ \ \ \ Merge branch 'after-script' into make-before-after-overridableKamil Trzcinski2016-04-181-2/+2
| |\ \ \ \ \ \
| * \ \ \ \ \ \ Merge branch 'after-script' into make-before-after-overridableKamil Trzcinski2016-04-171-14/+14
| |\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * after-script: Add CHANGELOG and documentation Rename finally_script to after_script Conflicts: lib/ci/gitlab_ci_yaml_processor.rb spec/lib/ci/gitlab_ci_yaml_processor_spec.rb
| * | | | | | | | Allow the before_script and finally_script to be overwritten in context of jobKamil Trzcinski2016-04-161-3/+70
| | | | | | | | |
* | | | | | | | | Merge branch 'after-script' into 'master' Kamil Trzciński2016-04-191-0/+29
|\ \ \ \ \ \ \ \ \ | | |_|_|/ / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Implement after_script which allows to do cleanups as part of the build process This implements `after_script` in global context. The `after_script` will be executed always after the job, even if the job were canceled. This requires changes on Runner side that will be implemented in 1.2. cc @tmaczukin @grzesiek See merge request !3771
| * | | | | | | | Merge remote-tracking branch 'origin/master' into after-scriptKamil Trzcinski2016-04-181-16/+72
| |\ \ \ \ \ \ \ \
| * \ \ \ \ \ \ \ \ Merge remote-tracking branch 'origin/master' into after-scriptKamil Trzcinski2016-04-181-14/+37
| |\ \ \ \ \ \ \ \ \ | | |_|_|_|/ / / / / | |/| | | | | | | |
| * | | | | | | | | Fix rubocop complainsKamil Trzcinski2016-04-181-2/+2
| | |_|_|/ / / / / | |/| | | | | | |
| * | | | | | | | Rename finally_script to after_scriptKamil Trzcinski2016-04-171-7/+7
| | |_|/ / / / / | |/| | | | | |
| * | | | | | | Implement finally_script which allows to do cleanups as part of the build ↵Kamil Trzcinski2016-04-161-0/+29
| | |_|_|_|_|/ | |/| | | | | | | | | | | | | | | | | | | process
* | | | | | | Using single builder for push and tag eventsGabriel Mazetto2016-04-191-22/+6
| | | | | | |
* | | | | | | Added System Hooks for push and tag_pushGabriel Mazetto2016-04-191-3/+28
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Code is based on Project Webhooks, removing deprecations and without commits listing.
* | | | | | | Use double instead of OpenStruct in the Issue/PR formatters specsDouglas Barbosa Alexandre2016-04-182-32/+34
| | | | | | |
* | | | | | | Set GitHub milestones to Issue/Merge Request that were importedDouglas Barbosa Alexandre2016-04-182-0/+35
| | | | | | |
* | | | | | | Import milestones from GitHubDouglas Barbosa Alexandre2016-04-181-0/+82
| | | | | | |
* | | | | | | Import labels from GitHubDouglas Barbosa Alexandre2016-04-181-0/+19
| | | | | | |
* | | | | | | Use GitHub Issue/PR number as iid to keep referencesDouglas Barbosa Alexandre2016-04-182-0/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | With these changes we don’t lost the issue/pr references when importing them to GitLab.
* | | | | | | Count the number of SQL queries per transactionYorick Peterse2016-04-181-0/+3
| |_|/ / / / |/| | | | | | | | | | | | | | | | | Fixes gitlab-org/gitlab-ce#15335
* | | | | | Minor refactorings in CI configfeature/ci-job-env-variablesGrzegorz Bizon2016-04-181-3/+3
| | | | | |
* | | | | | Validate job-level variables in YAML config fileGrzegorz Bizon2016-04-181-13/+32
| | | | | |
* | | | | | Minor refactoring in code related to job variablesGrzegorz Bizon2016-04-181-19/+24
| | | | | |
* | | | | | Make CI config return empty array if no job variablesGrzegorz Bizon2016-04-181-0/+12
| | | | | |
* | | | | | Rename method that returns global envs in CI confGrzegorz Bizon2016-04-181-1/+1
| | | | | |
* | | | | | Read job variables directly from gitlab CI configGrzegorz Bizon2016-04-181-2/+2
| | | | | |
* | | | | | Make it possible to override build variablesGrzegorz Bizon2016-04-181-1/+1
| | | | | |
* | | | | | Add `variables` keyword to job in CI config YAMLGrzegorz Bizon2016-04-181-12/+32
| |/ / / / |/| | | |
* | | | | Use Module#prepend for method instrumentationfix-method-instrumentationYorick Peterse2016-04-181-14/+37
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | By using Module#prepend we can define a Module containing all proxy methods. This removes the need for setting up crazy method alias chains and in turn prevents us from having to deal with all that madness (e.g. methods calling each other recursively). Fixes gitlab-org/gitlab-ce#15281
* | | | Merge branch 'master' of gitlab.com:gitlab-org/gitlab-ce into ↵James Lopez2016-04-151-0/+7
|\ \ \ \ | | | | | | | | | | | | | | | fix/project-import_url
| * | | | Emoji categories fixValery Sizov2016-04-141-0/+7
| | | | |
* | | | | Merge branch 'master' of gitlab.com:gitlab-org/gitlab-ce into ↵James Lopez2016-04-148-77/+359
|\ \ \ \ \ | |/ / / / | | | | | | | | | | fix/project-import_url
| * | | | Merge branch 'simplify-gitlab-url_builder-15202' into 'master' Rémy Coutable2016-04-142-57/+89
| |\ \ \ \ | | |_|_|/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Refactor and expose only Gitlab::UrlBuilder.build(record) ``` $ git grep Gitlab::UrlBuilder app/models/commit.rb: url: Gitlab::UrlBuilder.build(self), app/services/issues/base_service.rb: issue_url = Gitlab::UrlBuilder.build(issue) app/services/merge_requests/base_service.rb: hook_data[:object_attributes][:url] = Gitlab::UrlBuilder.build(merge_request) app/views/search/results/_note.html.haml:- note_url = Gitlab::UrlBuilder.build(note) lib/gitlab/note_data_builder.rb: base_data[:object_attributes][:url] = Gitlab::UrlBuilder.build(note) spec/lib/gitlab/note_data_builder_spec.rb: expect(data[:object_attributes][:url]).to eq(Gitlab::UrlBuilder.build(note)) spec/lib/gitlab/url_builder_spec.rb:describe Gitlab::UrlBuilder, lib: true do ``` Fixes #15202. See merge request !3696
| | * | | Refactor and expose only Gitlab::UrlBuilder.build(record)simplify-gitlab-url_builder-15202Rémy Coutable2016-04-132-57/+89
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
| * | | | Merge branch 'external-omniauth-providers' into 'master' Marin Jankovski2016-04-131-9/+28
| |\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Allow Omniauth providers to be marked as external Related to #4009 With this MR we will be able to allow the user to set which Omniauth Providers they would like to have as external. All users login in via these providers will be marked as external, even if they already had an account before. If the provider is removed form the list of external providers, the users will be marked as internal at their next login. MR for Omnibus: gitlab-org/omnibus-gitlab!727 /cc @dblessing @DouweM See merge request !3657
| | * | | | Added default setting for `external_providers`Patricio Cano2016-04-111-9/+28
| | | |_|/ | | |/| |
| * | | | Merge branch 'rs-gitlab-com-check' into 'master' Dmitriy Zaporozhets2016-04-131-0/+17
| |\ \ \ \ | | |_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add `Gitlab.com?` method To be used as a feature flag for GitLab.com-only features, such as welcome emails. We will be careful to only use this to disable features or functionality that do not make sense for any installations that aren't GitLab.com. We will not use this to restrict features from other installations or keep them "exclusive" to GitLab.com. See merge request !3682
| | * | | Add `Gitlab.com?` methodrs-gitlab-com-checkRobert Speicher2016-04-121-0/+17
| | |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | To be used as a feature flag for GitLab.com-only features, such as welcome emails. We will be careful to only use this to disable features or functionality that do not make sense for any installations that aren't GitLab.com. We will not use this to restrict features from other installations or keep them "exclusive" to GitLab.com.
| * | | Added ability to add custom tags to transactionsYorick Peterse2016-04-131-0/+25
| |/ / | | | | | | | | | | | | | | | | | | One use case for this is manually setting the "action" tag for Grape API calls. Due to Grape running blocks there are no human readable method names that can be used for the "action" tag, thus we have to set these manually on a case by case basis.
| * | Fixed stubbing for Gitlab::Metrics specsYorick Peterse2016-04-111-1/+1
| | | | | | | | | | | | | | | | | | If the measure method uses Transaction.current directly the SQL subscriber (Subscribers::ActiveRecord) will add timings of queries triggered by DB cleaner.
| * | Track call counts in Gitlab::Metrics.measure_blockmetrics-measure-block-transactionYorick Peterse2016-04-111-0/+3
| | |
| * | Added specs for Gitlab::Metrics::System.cpu_timeYorick Peterse2016-04-111-0/+6
| | |
| * | Store block timings as transaction valuesYorick Peterse2016-04-111-11/+5
| |/ | | | | | | | | | | | | | | | | | | This makes it easier to query, simplifies the code, and makes it possible to figure out what transaction the data belongs to (simply because it's now stored _in_ the transaction). This new setup keeps track of both the real/wall time _and_ CPU time spent in a block, both measured using milliseconds (to keep all units the same).
| * Instrument Rails cache codeinstrument-rails-cacheYorick Peterse2016-04-081-0/+71
| | | | | | | | | | This allows us to track how much time of a transaction is spent in dealing with cached data.
| * Remove dumb debug statement and add many tests.Jacob Schatz2016-04-071-0/+115
| |