summaryrefslogtreecommitdiff
path: root/spec/support
Commit message (Collapse)AuthorAgeFilesLines
* Merge branch '28695-move-all-associated-records-to-ghost-user' into 'master' Douwe Maan2017-04-061-0/+39
|\ | | | | | | | | | | | | Resolve "Deleting a user shouldn't delete associated records" Closes #28695 and #30514 See merge request !10467
| * Implement review comments from @DouweM for !10467.28695-move-all-associated-records-to-ghost-userTimothy Andrew2017-04-061-24/+4
| | | | | | | | | | | | | | | | | | | | 1. Have `MigrateToGhostUser` be a service rather than a mixed-in module, to keep things explicit. Specs testing the behavior of this class are moved into a separate service spec file. 2. Add a `user.reported_abuse_reports` association to make the `migrate_abuse_reports` method more consistent with the other `migrate_` methods.
| * Move a user's award emoji to the ghost userTimothy Andrew2017-04-061-1/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ... when the user is destroyed. 1. Normally, for a given awardable and award emoji name, a user is only allowed to create a single award emoji. 2. This validation needs to be removed for ghost users, since: - User A and User B have created award emoji - with the same name and against the same awardable - User A is deleted. Their award emoji is moved to the ghost user - User B is deleted. Their award emoji needs to be moved to the ghost user. However, this breaks the uniqueness validation, since the ghost user is only allowed to have one award emoji of a given name for a given awardable
| * Move a user's notes to the ghost userTimothy Andrew2017-04-061-12/+14
| | | | | | | | ... when the user is destroyed.
| * Move a user's merge requests to the ghost user.Timothy Andrew2017-04-061-0/+53
| | | | | | | | | | | | | | 1. When the user is deleted. 2. Refactor out code relating to "migrating records to the ghost user" into a `MigrateToGhostUser` concern, which is tested using a shared example.
* | Merge branch 'feature/multi-level-container-registry-images' into 'master' Kamil Trzciński2017-04-061-11/+28
|\ \ | | | | | | | | | | | | | | | | | | Multi-level container registry images Closes #17801 See merge request !10109
| * \ Merge branch 'master' into feature/multi-level-container-registry-imagesGrzegorz Bizon2017-04-066-21/+128
| |\ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * master: (94 commits) Merge branch 'open-redirect-fix-continue-to' into 'security' Merge branch 'open-redirect-host-fix' into 'security' Merge branch 'path-disclosure-proj-import-export' into 'security' Merge branch '29364-private-projects-mr-fix' Merge branch '30125-markdown-security' Issue title realtime Update CHANGELOG.md for 8.16.9 Update CHANGELOG.md for 8.17.5 Update CHANGELOG.md for 9.0.4 Add "search" optional param and docs for V4 Use PDFLab to render PDFs in GitLab Separate Scala from Java in CI examples Fix broken link Reorganize CI examples, add more links Refactor CI index page Remove deprecated field from workhorse response Use gitlab-workhorse 1.4.3 Document how ETag caching middleware handles query parameters Make group skip validation in the frontend Use NamespaceValidator::WILDCARD_ROUTES in ETag caching middleware ...
| * \ \ Merge branch 'master' into feature/multi-level-container-registry-imagesGrzegorz Bizon2017-04-046-22/+13
| |\ \ \ | | | |/ | | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * master: (57 commits) Ensure we generate unique usernames otherwise validations fail Fix a Knapsack issue that would load support/capybara.rb before support/env.rb Ensure users have a short username otherwise a click event is triggered ... Enable the `bullet_logger` setting; enable `raise` in test environment Fix Rubocop offenses Set the right timeout for Gitlab::Shell#fetch_remote Refactoring Projects::ImportService Move methods that are not related to mirroring to the repository model Fix GitHub pull request formatter spec Rename skip_metrics to imported on the importable concern Add CHANGELOG Remove unused include from RepositoryImportWorker Skip MR metrics when importing projects from GitHub Fetch GitHub project as a mirror to get all refs at once Make file templates easy to use and discover Ensure user has a unique username otherwise `user10` would match `user1` Ensure the AbuseReport fixtures create unique reported users Don't use FFaker in factories, use sequences instead Fix brittle specs Fix the AbuseReport seeder ... Conflicts: db/schema.rb
| * | | Refactor container registry repository tag stubsGrzegorz Bizon2017-04-031-6/+22
| | | |
| * | | Refactor feature specs for container registryGrzegorz Bizon2017-04-031-8/+8
| | | |
| * | | Merge branch 'master' into feature/multi-level-container-registry-imagesGrzegorz Bizon2017-04-039-20/+55
| |\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * master: (230 commits) Fix N+1 query in loading pipelines in merge requests Fix Spinach and Capybara dependencies Prevent users from disconnecting gitlab account from CAS 30276 Move issue, mr, todos next to profile dropdown in top nav Refactor SearchController#show Properly eagerly-load the Capybara server for JS feature specs only Updating documentation to include a missing step in the update procedure Eager-load the Capybara server to prevent timeouts Increase Capybara's timeout Add metrics button to Environment Overview page Fix link to Jira service documentation Handle parsing OpenBSD ps output properly to display sidekiq infos on ... Eliminate unnecessary queries that add ~500 ms of load time for a large issue 20914 Limits line length for project home page Allow users to import GitHub projects to subgroups Update dpl CI example Fix the docs:check:links job Don't clean up the gitlab-test-fork_bare repo Make GitLab use Gitaly for commit_is_ancestor Remove unnecessary ORDER BY clause from `forked_to_project_id` subquery ...
| * \ \ \ Merge branch 'master' into feature/multi-level-container-registry-imagesGrzegorz Bizon2017-03-277-83/+135
| |\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * master: (192 commits) Implement new service for creating user Update sentry-raven 2.0.2 -> 2.4.0 Update webmock 1.21.0 -> 1.24.6 Update spring 1.7.2 -> 2.0.1 Update simplecov 0.12.0 -> 0.14.1 Update pry-rails 0.3.4 -> 0.3.5 Update pry-byebug 3.4.1 -> 3.4.2 Update flay 2.6.1 -> 2.8.1 Remove Tags filter from Projects Explore dropdown Update capybara-screenshot 1.0.11 -> 1.0.14 Update bullet 5.2.0 -> 5.5.1 Update brakeman 3.4.1 -> 3.6.1 Remove web-console gem Update better_errors 1.0.1 -> 2.1.1 Display flash message to unauthenticated user when creating new issue Fix up emoji tests that should have failed :/ Fix RSpec/DescribeSymbol cop violations Add event limit warning all tabs Cycle Analytics Adding non_archived scope for counting projects Resolve "Gitlab administrator cannot create projects in every group" ... Conflicts: db/schema.rb
| * | | | | Fix specs for container repository destroy serviceGrzegorz Bizon2017-03-241-1/+2
| | | | | |
* | | | | | Recent search history for issuesEric Eastwood2017-04-061-2/+16
| |_|_|_|/ |/| | | | | | | | | | | | | | Fixes https://gitlab.com/gitlab-org/gitlab-ce/issues/27262
* | | | | Merge branch 'gitaly-refs' into 'master'Robert Speicher2017-04-052-16/+22
|\ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Implement Gitaly refs client See merge request !9291
| * | | | | Incorporate Gitaly client for refs serviceAlejandro Rodríguez2017-04-042-16/+22
| | | | | |
* | | | | | Create subgroups if they don't exist while importing projectsJames Lopez2017-04-051-3/+56
| | | | | |
* | | | | | Refactor test_utils bundleMike Greiling2017-04-041-2/+2
|/ / / / /
* | | | | Merge branch '29492-useless-queries' into 'master'Sean McGivern2017-04-042-0/+48
|\ \ \ \ \ | |_|_|_|/ |/| | | | | | | | | | | | | | | | | | | | | | | | remove useless queries with false conditions (e.g 1=0) Closes #29492 See merge request !10141
| * | | | Remove useless queries with false conditions (e.g 1=0)mhasbini2017-04-042-0/+48
| | |_|/ | |/| |
* | | | Fix brittle specsRémy Coutable2017-04-032-4/+4
| | | | | | | | | | | | | | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* | | | Don't use FFaker in factories, use sequences insteadRémy Coutable2017-04-034-19/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | FFaker can generate data that randomly break our test suite. This simplifies our factories and use sequences which are more predictive. Signed-off-by: Rémy Coutable <remy@rymai.me>
* | | | Fix warning when cloning repo that already existsbvl-cleanup-reposBob Van Landuyt2017-04-021-2/+4
|/ / / | | | | | | | | | | | | | | | fixes the warning: fatal: destination path '~/gitlab-ce/gitlab/tmp/tests/gitlab-test-fork_bare' already exists and is not an empty directory.
* | | Fix Spinach and Capybara dependenciesStan Hu2017-04-011-1/+1
| | | | | | | | | | | | Closes #30347
* | | Properly eagerly-load the Capybara server for JS feature specs onlyRémy Coutable2017-03-312-3/+8
| | | | | | | | | | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* | | Eager-load the Capybara server to prevent timeoutsRémy Coutable2017-03-312-2/+2
| | | | | | | | | | | | Signed-off-by: Rémy Coutable <remy@rymai.me>
* | | Increase Capybara's timeoutRémy Coutable2017-03-312-8/+2
| | | | | | | | | | | | | | | | | | | | | | | | Also disabled assets warming for Capybara/RSpec and remove a check to skip assets warming when ./tmp/cache/assets/test exists because assets are now served by webpack-dev-server. Signed-off-by: Rémy Coutable <remy@rymai.me>
* | | Allow users to import GitHub projects to subgroupsDouglas Barbosa Alexandre2017-03-301-0/+14
| | |
* | | Don't clean up the gitlab-test-fork_bare repoNick Thomas2017-03-301-2/+2
| | |
* | | group links select2 infinite scrollLuke "Jared" Bennett2017-03-301-0/+8
| | |
* | | Merge branch 'rs-empty-project-misc' into 'master' Rémy Coutable2017-03-301-1/+1
|\ \ \ | | | | | | | | | | | | | | | | Use `:empty_project` in worker, view, task, serializer, policy, migration, and mailer specs See merge request !10272
| * | | Use `:empty_project` where possible in mailer specsRobert Speicher2017-03-271-1/+1
| | |/ | |/|
* | | Merge branch '29843-project-subgroup-transfer' into 'security'Rémy Coutable2017-03-292-4/+14
| | | | | | | | | | | | | | | | | | Use full path for moving directories when changing namespace path See merge request !2078
* | | Merge branch '30172-fix-mattermost-enabled-setting-not-properly-stubbed' ↵Sean McGivern2017-03-281-0/+4
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | into 'master' Add a new `stub_mattermost_setting` stub helper to properly stub Mattermost settings Closes #30172 See merge request !10288
| * | | Add a new `stub_mattermost_setting` stub helper to properly stub Mattermost ↵Rémy Coutable2017-03-281-0/+4
| | | | | | | | | | | | | | | | | | | | | | | | settings Signed-off-by: Rémy Coutable <remy@rymai.me>
* | | | Support wider range of monitored environments with PrometheusJoshua Lambert2017-03-281-2/+2
|/ / /
* | | Use `:empty_project` where possible in service specsRobert Speicher2017-03-271-1/+1
|/ /
* | fix new MR form erroneously selects tag as source when branch of same name ↵Sanster2017-03-241-2/+3
| | | | | | | | exists #23237
* | Merge branch 'rs-notify-mailer-specs' into 'master' Rémy Coutable2017-03-231-78/+62
|\ \ | | | | | | | | | | | | Cleanup Notify spec and combine multiple examples into one See merge request !10129
| * | Miscellaneous cleanup of Notify shared examplesRobert Speicher2017-03-211-56/+64
| | |
| * | Remove the 'custom subject suffix' spec from a shared exampleRobert Speicher2017-03-211-10/+0
| | | | | | | | | | | | | | | | | | This one-off test case is covered in the Notify mailer spec, so there was no benefit in testing it every time this shared example was referenced.
| * | Move the `a new user email` shared example to the Emails::Profile specRobert Speicher2017-03-211-14/+0
| | | | | | | | | | | | | | | | | | | | | This shared example was only used by this spec so having it in a separate file provided no benefit, at the cost of clarity. This also reduces the three `it` blocks into a single test with `aggregate_failures`.
* | | Merge branch 'zj-chat-notification-default-branch' into 'master' Rémy Coutable2017-03-231-0/+19
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | Allow chat notifications only for the default branch Closes #28565 and #18088 See merge request !10158
| * | | Improve readability and add testZ.J. van de Weg2017-03-231-0/+19
| | | |
* | | | Don't navigate to about:blankRémy Coutable2017-03-221-8/+0
| | | | | | | | | | | | | | | | | | | | | | | | This turned out to prevent the browser from clearing the local storage. Signed-off-by: Rémy Coutable <remy@rymai.me>
* | | | Introduce a new middleware for the test environment that can block requestsRémy Coutable2017-03-221-0/+40
|/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The idea is that after each feature spec example, we block all incoming requests at the Rack level, go to the 'about:blank' page, and wait until the current requests reach 0. This should solve the problem where a request would end after database cleaner performed the database truncation. The problem was that a GET request can still lead to records creation (e.g. namespaces or routes). Signed-off-by: Rémy Coutable <remy@rymai.me>
* | | Merge branch 'resolve-global-vue' into 'master' Filipa Lacerda2017-03-211-1/+1
|\ \ \ | |/ / |/| | | | | | | | Remove Vue from the global space See merge request !10056
| * | fix wait_for_vue_resource to not rely on global Vue objectMike Greiling2017-03-201-1/+1
| |/
* | Resolve "Fix the Prometheus queries for multiple container environments"Jose Ivan Vargas Lopez2017-03-211-2/+2
| |
* | New directory from interface on existing branchJacopo2017-03-211-0/+16
|/ | | | | The user can create a new directory on a different branch than the source branch when the branch already exists.