| Commit message (Collapse) | Author | Age | Files | Lines |
|---|
| | |
|
| |\
| |
| |
| |
| | |
Fix hiding of quick action prompt in specs
See merge request gitlab-org/gitlab-ce!28285
|
| | | |
|
| |\ \
| | |
| | |
| | |
| | | |
EE Backport to CE of EE!12404
See merge request gitlab-org/gitlab-ce!28292
|
| | | | |
|
| |\ \ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Fix project visibility level validation
Closes #59379
See merge request gitlab-org/gitlab-ce!28305
|
| | |/ / |
|
| |\ \ \
| |/ /
|/| |
| | |
| | |
| | |
| | | |
Rename methods that conflict in Rails 5.2
Closes #55583
See merge request gitlab-org/gitlab-ce!28281
|
| | |/
| |
| |
| |
| | |
Adds suffix to enum methods and changes `in_groups` to
`of_groups`
|
| |\ \
| |/
|/|
| |
| |
| |
| | |
Eager load the GraphQL schema in specs
Closes gitlab-ee#11497
See merge request gitlab-org/gitlab-ce!28246
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Avoid counting on Rails eager loading the GraphQL types, but preload
them before the specs.
This would avoid loading the schema in 2 separate threads
concurrently (1 inside the specs, 1 inside the test-server that will
receive requests). Loading the schema in parallel like that could
cause duplicate definition errors.
|
| |\ \
| |/
|/|
| |
| | |
Fix unrelated errors when teardowning quanrantined specs
See merge request gitlab-org/gitlab-ce!28240
|
| | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Since we're skipping the example in a before hook, the @_subscribers
instance variable that is normally set in the setup phase of the test by
https://github.com/rails/rails-controller-testing/blob/21014e48be124dd8d0af250b79f682519d445015/lib/rails/controller/testing/template_assertions.rb#L21
isn't set but the teardown phase of the example still happens and tries
to iterate over the array of subcribers: https://github.com/rails/rails-controller-testing/blob/21014e48be124dd8d0af250b79f682519d445015/lib/rails/controller/testing/template_assertions.rb#L54
Using an around hook allows the @_subscribers instance variable to be
set.
Signed-off-by: Rémy Coutable <remy@rymai.me>
|
| |\ \
| | |
| | |
| | |
| | |
| | |
| | | |
Properly handle LFS Batch API response in project import
Closes #61624
See merge request gitlab-org/gitlab-ce!28223
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Project imports were failing with `undefined method each_with_object for
String` because the import was attempting to parse the LFS Batch API and
failing due to the fact that the Content-Type wasn't a supported format
(e.g. application/vnd.git-lfs+json instead of application/json). We now
parse the body as JSON.
Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/61624
|
| |\ \ \
| | | |
| | | |
| | | |
| | | | |
Simplify frontend test fixtures
See merge request gitlab-org/gitlab-ce!27531
|
| | | | | |
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
for f in $(find spec/javascripts/fixtures/ -name '*.rb')
do
sed -E -e 's/(, ){0,1}(fixture_file_name|
example\.description)(, ){0,1}//' < $f > $f.tmp; mv $f.tmp $f
done
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
for f in $(find spec/javascripts/fixtures/ -name '*.rb')
do
sed 's/do |example|/do/' < $f > $f.tmp; mv $f.tmp $f
done
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
for f in $(find spec/javascripts/fixtures/ -name '*.rb')
do
grep -v store_frontend_fixture < $f > $f.tmp; mv $f.tmp $f
done
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| | | | | |
|
| | |_|/
|/| |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
Pull mirrors would run the `Projects::AfterImportService`, which would
force a `git gc` each time it finished. This is overkill and not
necessary now that we have refs packed more frequently
(https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/27826).
Closes https://gitlab.com/gitlab-org/gitlab-ee/issues/11556
|
| | |/
|/|
| |
| |
| |
| |
| | |
A project admin attempting to invite an already-invited user
will see a confusing Error 500 message.
Closes https://gitlab.com/gitlab-org/gitlab-ce/issues/61574
|
| |\ \
| | |
| | |
| | |
| | |
| | |
| | | |
Change DetectRepositoryLanguagesWorker to not receive user
Closes #60425
See merge request gitlab-org/gitlab-ce!28091
|
| | | |
| | |
| | |
| | | |
Fixes #60425
|
| |\ \ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
'master'
Resolve "Flaky spec: "Dashboard > User filters projects with search bar Sorting Sorting by Stars sorts the project list" / spec/features/dashboard/user_filters_projects_spec.rb:257"
Closes #61635
See merge request gitlab-org/gitlab-ce!28224
|
| | | | |
| | | |
| | | |
| | | | |
Signed-off-by: Rémy Coutable <remy@rymai.me>
|
| | |/ /
|/| | |
|
| |/ /
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
This helper is right on the limit of the number of calls being made.
When the request store is enabled, and Sidekiq is running in inline
mode, just a couple of additional actions result in it being pushed
over the line.
The operation is entirely artificial, since in reality we'd perform
these requests in multiple separate processes, so just whitelist it
for now.
|
| | | |
|
| | | |
|
| | |
| |
| |
| |
| | |
This will avoid timestamp comparison issues later within equality check
with attributes
|
| | | |
|
| |\ \
| | |
| | |
| | |
| | |
| | |
| | | |
58404 - setup max depth for graphql
Closes #58404
See merge request gitlab-org/gitlab-ce!25737
|
| | | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
58404 - add change log
58404 - add spec
58404 - add more spec to test depth 2
58404 - fix spec
58404 - fix rubocop
58404 - refactor the code by Bob's advice
58404 - revert changes of all_graphql_fields_for
58404 - change text only
58404 - fix rspec according to gitlab's standard
58404 - revert previous spec
58404 - fix rubocop
|
| |\ \ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Adds header column to variables list
Closes #46806
See merge request gitlab-org/gitlab-ce!28060
|
| | | | |
| | | |
| | | |
| | | |
| | | | |
For the variables list in CI/CD settings
this commits adds an header row with titles
|
| |\ \ \ \
| | | | |
| | | | |
| | | | |
| | | | |
| | | | |
| | | | | |
Handle errors in successful notes reply
Closes #61377
See merge request gitlab-org/gitlab-ce!28082
|
| | | | | | |
|
| | | |_|/
| |/| | |
|
| |/ / /
| | |
| | |
| | |
| | |
| | | |
Udpated user_edit_profile_spec with time preferences
Minor update form fields
|
| | | |
| | |
| | |
| | | |
Use existing form to allow users to add custom metrics via the dashboard
|
| |\ \ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
Resolve "Add new Unicorn metrics"
Closes #56850
See merge request gitlab-org/gitlab-ce!27474
|
| | | | | |
|
| | | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
This updates monitor docs to reflect the new ruby and unicorn metrics as
well as making it so we fetch process start time via the proc table
instead of via CLOCK_BOOTTIME
|
| | | | |
| | | |
| | | |
| | | |
| | | | |
These metrics are not unicorn specific and can be used across ruby
processes
|
