From f14507e586a7f75f0fb71a1d8468b7361be860d4 Mon Sep 17 00:00:00 2001 From: GitLab Bot Date: Mon, 3 Feb 2020 09:08:42 +0000 Subject: Add latest changes from gitlab-org/gitlab@master --- .gitlab/ci/rails.gitlab-ci.yml | 4 +- .../javascripts/diff_notes/models/discussion.js | 6 +- .../javascripts/diff_notes/stores/comments.js | 6 +- .../environments/components/container.vue | 2 +- .../environments/components/environments_table.vue | 2 +- .../javascripts/error_tracking_settings/utils.js | 2 +- app/assets/javascripts/helpers/diffs_helper.js | 10 +- .../javascripts/ide/stores/actions/project.js | 4 +- app/assets/javascripts/lib/utils/webpack.js | 2 +- .../merge_conflicts/merge_conflict_store.js | 2 +- app/assets/javascripts/network/branch_graph.js | 10 +- app/assets/javascripts/notes.js | 2 +- .../settings/components/registry_settings_app.vue | 2 +- .../registry/settings/components/settings_form.vue | 207 +---------------- .../javascripts/registry/settings/constants.js | 15 -- .../javascripts/registry/settings/store/getters.js | 13 +- .../registry/settings/store/mutations.js | 4 +- app/assets/javascripts/registry/settings/utils.js | 6 - .../shared/components/expiration_policy_form.vue | 247 +++++++++++++++++++++ .../javascripts/registry/shared/constants.js | 15 ++ app/assets/javascripts/registry/shared/utils.js | 19 ++ app/assets/javascripts/users_select.js | 2 +- .../vue_shared/components/modal_copy_button.vue | 2 +- app/assets/javascripts/webpack.js | 2 +- app/assets/javascripts/zen_mode.js | 6 +- app/helpers/button_helper.rb | 2 +- app/models/concerns/delete_with_limit.rb | 11 + app/models/event.rb | 2 + app/models/hooks/web_hook_log.rb | 2 + .../project_services/emails_on_push_service.rb | 7 +- app/views/sherlock/queries/_general.html.haml | 4 +- app/workers/prune_old_events_worker.rb | 16 +- app/workers/prune_web_hook_logs_worker.rb | 17 +- .../unreleased/198577-fix-emails-on-push.yml | 5 + .../pedroms-change-copy-to-clipboard-icon.yml | 5 + config/initializers/graphql.rb | 6 +- package.json | 2 +- .../projects/settings/operations_settings_spec.rb | 2 +- .../projects/settings/registry_settings_spec.rb | 10 +- .../error_tracking_settings/store/getters_spec.js | 2 +- .../ide/components/ide_status_list_spec.js | 2 +- spec/frontend/mr_popover/mr_popover_spec.js | 2 +- .../__snapshots__/settings_form_spec.js.snap | 188 ---------------- .../components/registry_settings_app_spec.js | 2 +- .../settings/components/settings_form_spec.js | 217 ++---------------- spec/frontend/registry/settings/mock_data.js | 12 - .../registry/settings/store/actions_spec.js | 13 +- .../registry/settings/store/getters_spec.js | 4 +- .../registry/settings/store/mutations_spec.js | 4 +- .../expiration_policy_form_spec.js.snap | 186 ++++++++++++++++ .../components/expiration_policy_form_spec.js | 237 ++++++++++++++++++++ spec/frontend/registry/shared/mock_data.js | 12 + .../releases/detail/components/app_spec.js | 2 +- .../sidebar/confidential_issue_sidebar_spec.js | 2 +- .../vue_shared/components/recaptcha_modal_spec.js | 2 +- .../vue_shared/components/slot_switch_spec.js | 2 +- spec/helpers/button_helper_spec.rb | 2 +- spec/models/concerns/delete_with_limit_spec.rb | 15 ++ .../emails_on_push_service_spec.rb | 16 ++ spec/requests/api/services_spec.rb | 15 -- yarn.lock | 226 ++++++++++--------- 61 files changed, 1017 insertions(+), 829 deletions(-) delete mode 100644 app/assets/javascripts/registry/settings/constants.js delete mode 100644 app/assets/javascripts/registry/settings/utils.js create mode 100644 app/assets/javascripts/registry/shared/components/expiration_policy_form.vue create mode 100644 app/assets/javascripts/registry/shared/constants.js create mode 100644 app/assets/javascripts/registry/shared/utils.js create mode 100644 app/models/concerns/delete_with_limit.rb create mode 100644 changelogs/unreleased/198577-fix-emails-on-push.yml create mode 100644 changelogs/unreleased/pedroms-change-copy-to-clipboard-icon.yml delete mode 100644 spec/frontend/registry/settings/components/__snapshots__/settings_form_spec.js.snap delete mode 100644 spec/frontend/registry/settings/mock_data.js create mode 100644 spec/frontend/registry/shared/components/__snapshots__/expiration_policy_form_spec.js.snap create mode 100644 spec/frontend/registry/shared/components/expiration_policy_form_spec.js create mode 100644 spec/frontend/registry/shared/mock_data.js create mode 100644 spec/models/concerns/delete_with_limit_spec.rb diff --git a/.gitlab/ci/rails.gitlab-ci.yml b/.gitlab/ci/rails.gitlab-ci.yml index 8c3df170f6d..6b0a7f31f1a 100644 --- a/.gitlab/ci/rails.gitlab-ci.yml +++ b/.gitlab/ci/rails.gitlab-ci.yml @@ -165,7 +165,7 @@ rspec-ee migration pg9: rspec-ee unit pg9: extends: .rspec-ee-base-pg9 - parallel: 5 + parallel: 10 rspec-ee integration pg9: extends: .rspec-ee-base-pg9 @@ -186,7 +186,7 @@ rspec-ee unit pg10: extends: - .rspec-ee-base-pg10 - .only-master - parallel: 5 + parallel: 10 rspec-ee integration pg10: extends: diff --git a/app/assets/javascripts/diff_notes/models/discussion.js b/app/assets/javascripts/diff_notes/models/discussion.js index daf61e5d467..97296a40d6e 100644 --- a/app/assets/javascripts/diff_notes/models/discussion.js +++ b/app/assets/javascripts/diff_notes/models/discussion.js @@ -1,4 +1,4 @@ -/* eslint-disable camelcase, guard-for-in, no-restricted-syntax */ +/* eslint-disable guard-for-in, no-restricted-syntax */ /* global NoteModel */ import $ from 'jquery'; @@ -40,13 +40,13 @@ class DiscussionModel { return true; } - resolveAllNotes(resolved_by) { + resolveAllNotes(resolvedBy) { for (const noteId in this.notes) { const note = this.notes[noteId]; if (!note.resolved) { note.resolved = true; - note.resolved_by = resolved_by; + note.resolved_by = resolvedBy; } } } diff --git a/app/assets/javascripts/diff_notes/stores/comments.js b/app/assets/javascripts/diff_notes/stores/comments.js index 69a972f644d..9bde18c4edf 100644 --- a/app/assets/javascripts/diff_notes/stores/comments.js +++ b/app/assets/javascripts/diff_notes/stores/comments.js @@ -1,4 +1,4 @@ -/* eslint-disable camelcase, no-restricted-syntax, guard-for-in */ +/* eslint-disable no-restricted-syntax, guard-for-in */ /* global DiscussionModel */ import Vue from 'vue'; @@ -26,11 +26,11 @@ window.CommentsStore = { discussion.createNote(noteObj); }, - update(discussionId, noteId, resolved, resolved_by) { + update(discussionId, noteId, resolved, resolvedBy) { const discussion = this.state[discussionId]; const note = discussion.getNote(noteId); note.resolved = resolved; - note.resolved_by = resolved_by; + note.resolved_by = resolvedBy; }, delete(discussionId, noteId) { const discussion = this.state[discussionId]; diff --git a/app/assets/javascripts/environments/components/container.vue b/app/assets/javascripts/environments/components/container.vue index cdf62259479..0a978ab5869 100644 --- a/app/assets/javascripts/environments/components/container.vue +++ b/app/assets/javascripts/environments/components/container.vue @@ -41,7 +41,7 @@ export default {
diff --git a/app/assets/javascripts/environments/components/environments_table.vue b/app/assets/javascripts/environments/components/environments_table.vue index 30299ccc7bc..8abc927c500 100644 --- a/app/assets/javascripts/environments/components/environments_table.vue +++ b/app/assets/javascripts/environments/components/environments_table.vue @@ -170,7 +170,7 @@ export default {