summaryrefslogtreecommitdiff
path: root/spec
Commit message (Collapse)AuthorAgeFilesLines
* Merge branch ↵Filipa Lacerda2019-06-051-8/+8
|\ | | | | | | | | | | | | | | | | '61910-common_utils_spec-js-fails-locally-on-master-for-roundofffloat' into 'master' Resolve "common_utils_spec.js fails locally for roundOffFloat" Closes #61910 See merge request gitlab-org/gitlab-ce!28963
| * Fix broken floating point tests61910-common_utils_spec-js-fails-locally-on-master-for-roundofffloatjboyson12019-05-301-8/+8
| | | | | | | | | | | | | | | | Update tests to use toBeCloseTo instead of toBe for floating point checks. More info here: https://jestjs.io/docs/en/expect#tobeclosetonumber-numdigits
* | Merge branch 'jivanvl-add-single-stat-chart-component' into 'master'Filipa Lacerda2019-06-051-0/+28
|\ \ | | | | | | | | | | | | Add single_stat chart component to the monitoring bundle See merge request gitlab-org/gitlab-ce!28948
| * | Add single_stat chart component to the monitoring bundlejivanvl-add-single-stat-chart-componentJose Vargas2019-06-041-0/+28
| | | | | | | | | | | | | | | This merge requests just adds the component without integrating it to the dashboard, it adds tests as well
* | | Merge branch 'revert-git-depth-for-merge-request' into 'master'Kamil Trzciński2019-06-051-19/+3
|\ \ \ | | | | | | | | | | | | | | | | Revert a default GIT_DEPTH for MR pipeline See merge request gitlab-org/gitlab-ce!28926
| * | | Revert a default GIT_DEPTH for MR pipelinerevert-git-depth-for-merge-requestFabio Pitino2019-05-311-19/+3
| | | |
* | | | Add multiple extends supportWolphin2019-06-053-11/+39
| | | |
* | | | Fix whitespace changes visibility when the related file was initially collapsedOndřej Budai2019-06-051-5/+6
| | | |
* | | | Merge branch 'fix-external-dashboard-specs' into 'master'Clement Ho2019-06-041-3/+5
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | Fix "operation settings external dashboard component" Jest specs See merge request gitlab-org/gitlab-ce!29160
| * | | | Fix submit button selector in specsPaul Gascou-Vaillancourt2019-06-041-3/+5
| | | | |
* | | | | Merge branch 'ce-9578-adjust-opened-issues-count-and-completion-rate' into ↵Robert Speicher2019-06-042-80/+20
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 'master' Port: EE adjust burndown chart opened issues count See merge request gitlab-org/gitlab-ce!28777
| * | | | | Adjust burndown chart opened issues countce-9578-adjust-opened-issues-count-and-completion-rateAlexandru Croitor2019-06-042-80/+20
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Adjust opened issues count in the milestone burndown chart to match current opened issues in the milestone. * Fix completion rate calculation to include milestones only.
* | | | | | Merge branch 'fe-jestify-specs-starting-with-a-1' into 'master'Mike Greiling2019-06-044-14/+51
|\ \ \ \ \ \ | |_|/ / / / |/| | | | | | | | | | | | | | | | | Jestify some specs that start with "A" See merge request gitlab-org/gitlab-ce!28727
| * | | | | Jestify autosave spec and add localStorage helperfe-jestify-specs-starting-with-a-1Paul Slaughter2019-06-042-11/+49
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The helper was needed because `jest.spyOn` would not work on this special window object property. The only way we could successfully spy on `localStorage` was with this helper.
| * | | | | Jestify api specPaul Slaughter2019-05-301-1/+1
| | | | | |
| * | | | | Jestify activities specPaul Slaughter2019-05-301-2/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Also add `jQuery` to global in `test_setup`. This was needed because: - `vendor/jquery.endless-scroll` depends on it. - Adding it to `global` in the spec itself didn't work.
* | | | | | Merge branch '61964-unicorn-instrumentation' into 'master'Bob Van Landuyt2019-06-043-8/+71
|\ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Adds ruby and unicorn instrumentation Closes #61964 See merge request gitlab-org/gitlab-ce!28499
| * \ \ \ \ \ Merge branch 'master' into 61964-unicorn-instrumentationRyan Cobb2019-05-29155-7033/+3214
| |\ \ \ \ \ \
| * | | | | | | Adds ruby and unicorn instrumentationRyan Cobb2019-05-203-8/+71
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This adds ruby and unicorn instrumentation. This was originally intended in 11.11 but due to performance concerns it was reverted. This new commit foregoes the sys-proctable gem was causing performance issues previously.
* | | | | | | | Handle external dashboard form submissionAdriel Santiago2019-06-043-49/+124
| |_|_|_|_|/ / |/| | | | | | | | | | | | | | | | | | | | Connect frontend UI with backend api for external dashboard link
* | | | | | | Extract setIssueDetail() into issue boards storeWinnie Hellmann2019-06-041-0/+11
| | | | | | |
* | | | | | | Merge branch 'ce-fix-vue-spec-warning' into 'master'Fatih Acet2019-06-041-8/+0
|\ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix warning in spec from committing unused mutation See merge request gitlab-org/gitlab-ce!28815
| * | | | | | | Fix warning in spec from committing unused mutationce-fix-vue-spec-warningSimon Knox2019-05-291-8/+0
| | | | | | | |
* | | | | | | | Extract toggleFilter() into issue boards storeWinnie Hellmann2019-06-041-0/+34
| | | | | | | |
* | | | | | | | Merge branch 'bvl-design-diff-notes-ce' into 'master'Douwe Maan2019-06-041-0/+12
|\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Adjustments related to DiffNotes on diffs outside of a project's main repository See merge request gitlab-org/gitlab-ce!29023
| * | | | | | | | Make `resolvable_types` a class methodBob Van Landuyt2019-06-031-0/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This turns Notable::RESOLVABLE_TYPES into a `Notable.resolvable_types`. That allows us to override it in EE.
* | | | | | | | | Merge branch 'mr-widget-merge-error' into 'master'Filipa Lacerda2019-06-042-8/+8
|\ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Update rendering of merge request in merge request widget Closes #40246 See merge request gitlab-org/gitlab-ce!28238
| * | | | | | | | | Update rendering of merge request in merge request widgetPhil Hughes2019-06-042-8/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/40246
* | | | | | | | | | Expose wiki_size on GraphQL APIAlessio Caiazza2019-06-041-1/+2
| | | | | | | | | |
* | | | | | | | | | Merge branch '10685-remove-feature-flag-ce' into 'master'Nick Thomas2019-06-044-6/+0
|\ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Remove `approval_rules` feature flag and remove obsolete approvals implementation See merge request gitlab-org/gitlab-ce!28979
| * | | | | | | | | | Remove spec feature stubbing10685-remove-feature-flag-ceMark Chao2019-06-043-4/+0
| | | | | | | | | | |
| * | | | | | | | | | FE: Remove legacy approval feature in MR filesPaul Slaughter2019-06-041-2/+0
| | | | | | | | | | |
* | | | | | | | | | | Merge branch 'cancel-auto-merge-when-merge-request-is-closed' into 'master'Grzegorz Bizon2019-06-041-0/+8
|\ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Cancel auto merge when merge request is closed See merge request gitlab-org/gitlab-ce!28782
| * | | | | | | | | | | Cancel auto merge when merge request is closedcancel-auto-merge-when-merge-request-is-closedShinya Maeda2019-06-041-0/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We should cancel auto merge when merge request is closed.
* | | | | | | | | | | | Merge branch ↵Grzegorz Bizon2019-06-0412-205/+397
|\ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | '58941-use-gitlab-serverless-with-existing-knative-installation' into 'master' Resolve "Use GitLab serverless with existing Knative installation" Closes #58941 See merge request gitlab-org/gitlab-ce!27173
| * | | | | | | | | | | | Move file one folder level up to avoid namespace conflict58941-use-gitlab-serverless-with-existing-knative-installationJoão Cunha2019-06-032-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Clusters::Cluster::KnativeServicesFinder becomes Clusters::KnativeServicesFinder This is to avoid loading race condition between: ``` module Clusters class Cluster < ApplicationRecord ``` and ``` module Clusters class Cluster ```
| * | | | | | | | | | | | Adapt functions to work for external KnativeJoão Cunha2019-05-2912-205/+397
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Remove Kn services cache from Clusters::Application::Knative Knative function can exist even if user did not installed Knative via GitLab managed apps. -> Move responsibility of finding services into the Cluster -> Responsability is inside Clusters::Cluster::KnativeServiceFinder -> Projects::Serverless::FunctionsFinder now calls depends solely on a cluster to find the Kn services. -> Detect Knative by resource presence instead of service presence -> Mock knative_installed response temporarily for frontend to develop Display loader while `installed === 'checking'` Added frontend work to determine if Knative is installed Memoize with_reactive_cache(*args, &block) to avoid race conditions When calling with_reactive_cache more than once, it's possible that the second call will already have the value populated. Therefore, in cases where we need the sequential calls to have consistent results, we'd fall under a race condition. Check knative installation via Knative resource presence Only load pods if Knative is discovered Always return a response in FunctionsController#index - Always indicate if Knative is installed, not installed or checking - Always indicate the partial response for functions. Final response is guaranteed when knative_installed is either true | false. Adds specs for Clusters::Cluster#knative_services_finder Fix method name when calling on specs Add an explicit check for functions Added an explicit check to see if there are any functions available Fix Serverless feature spec - we don't find knative installation via database anymore, rather via Knative resource Display error message for request timeouts Display an error message if the request times out Adds feature specs for when functions exist Remove a test purposed hardcoded flag Add ability to partially load functions Added the ability to partially load functions on the frontend Add frontend unit tests Added tests for the new frontend additions Generate new translations Generated new frontend translations Address review comments Cleaned up the frontend unit test. Added computed prop for `isInstalled`. Move string to constant Simplify nil to array conversion Put knative_installed states in a frozen hash for better read Pluralize list of Knative states Quey services and pods filtering name This way we don't need to filter the namespace in memory. Also, the data we get from the network is much smaller. Simplify cache_key and fix bug - Simplifies the cache_key by removing namespace duplicate - Fixes a bug with reactive_cache memoization
* | | | | | | | | | | | | Merge branch 'introduce-auto-merge-process-worker' into 'master'Thong Kuah2019-06-043-27/+65
|\ \ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Introduce sidekiq worker for auto merge process See merge request gitlab-org/gitlab-ce!28780
| * | | | | | | | | | | | | Introduce sidekiq worker for auto merge processintroduce-auto-merge-process-workerShinya Maeda2019-06-043-27/+65
| | |/ / / / / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | As we have a central domain for auto merge process today, we should use a single worker for any auto merge process.
* | | | | | | | | | | | | Change s_() calls to _() callsMichał Zając2019-06-041-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There are no namespaces in the strings so we don't need those
* | | | | | | | | | | | | Ignore Puma empty worker statsJan Provaznik2019-06-041-0/+27
| |_|/ / / / / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In some cases (during worker start) it's possible that Puma.stats returns an empty hash for worker's last status. In that case we just skip sampling of the worker until these stats are available.
* | | | | | | | | | | | Merge branch ↵Kamil Trzciński2019-06-046-103/+171
|\ \ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 'set-real-next-run-at-for-preventing-duplciate-pipeline-creations' into 'master' Make pipeline schedule worker resilient Closes gitlab-com/gl-infra/production#805 and #61955 See merge request gitlab-org/gitlab-ce!28407
| * | | | | | | | | | | | Make pipeline schedule worker resilientset-real-next-run-at-for-preventing-duplciate-pipeline-creationsShinya Maeda2019-06-036-103/+171
| | |_|_|/ / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Currently, pipeline schedule worker is unstable because it's sometimes killed by excessive memory consumption. In order to improve the performance, we add the following fixes: 1. next_run_at is always real_next_run, which means the value always takes into account of worker's cron schedule 1. Remove exlusive lock. This is already covered by real_next_run change. 1. Use RunPipelineScheduleWorker for avoiding memory killer. Memory consumption is spread to the multiple sidekiq worker.
* | | | | | | | | | | | Add expand/collapse buttonAdriel Santiago2019-06-041-0/+8
| |/ / / / / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add ability to expand/collapse operation settings
* | | | | | | | | | | Merge branch 'sh-fix-import-url-update' into 'master'Thong Kuah2019-06-031-0/+12
|\ \ \ \ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Fix project settings not being able to update Closes #62708 See merge request gitlab-org/gitlab-ce!29097
| * | | | | | | | | | | Fix project settings not being able to updateStan Hu2019-06-031-0/+12
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Previously import_url would always be present in the update parameters, which would cause the validation to fail. We now only include this parameter only if there is URL given. Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/62708
* | | | | | | | | | | | Comply with `no-implicit-coercion` rule (CE)Nathan Friend2019-06-033-3/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This commit is the result of running `yarn eslint --fix` after enabling the `no-implicit-coercion` ESLint rule. This rule has been added to our ESLint config here: https://gitlab.com/gitlab-org/gitlab-eslint-config/merge_requests/14
* | | | | | | | | | | | Backporting EE fixrossfuhrman2019-06-031-36/+38
|/ / / / / / / / / / /
* | | | | | | | | | | Move boardsStore.moving to BoardList componentWinnie Hellmann2019-06-031-0/+12
| | | | | | | | | | |
* | | | | | | | | | | Merge branch 'master' of dev.gitlab.org:gitlab/gitlabhqRobert Speicher2019-06-035-2/+72
|\ \ \ \ \ \ \ \ \ \ \