| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
| |
`from_config` (#2392)
|
| |
|
|
|
|
|
|
|
|
|
| |
This is an attempt to fix the flaky test
`test_cancel_merge_when_pipeline_succeeds`.
Were seeing a: 405 Method Not Allowed error when setting the MR to
merge_when_pipeline_succeeds.
Closes: #2383
|
| |
|
| |
|
| |
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
`cancel_merge_when_pipeline_succeeds()` (#2350)
* Call was incorrectly using a `PUT` method when should have used a
`POST` method.
* Changed return type to a `dict` as GitLab only returns
{'status': 'success'} on success. Since the function didn't work
previously, this should not impact anyone.
* Updated the test fixture `merge_request` to add ability to create
a pipeline.
* Added functional test for `mr.cancel_merge_when_pipeline_succeeds()`
Fixes: #2349
|
| |
|
|
|
|
| |
Make the `test_project_push_rules` test work.
|
| |
|
| |
|
| |
|
| |
|
|
|
|
|
|
|
|
|
|
| |
Have been seeing intermittent failures in the test:
tests/functional/api/test_merge_requests.py::test_merge_request_reset_approvals
Also saw a failure in:
tests/functional/cli/test_cli_v4.py::test_accept_request_merge[subprocess]
Add a call to `wait_for_sidekiq()` to hopefully resolve the issues.
|
|
|
|
|
| |
Simplify usage of `wait_for_sidekiq` by putting the assert if it timed
out inside the function rather than after calling it.
|
|
|
|
|
|
| |
Added the newly added reset_approvals merge request api.
Signed-off-by: Lucas Zampieri <lzampier@redhat.com>
|
|
|
|
|
|
|
|
|
| |
* Use `settings.delayed_group_deletion=False` as that is the
recommended method to turn off the delayed group deletion now.
* Change test to look for `default` as `pages` is not mentioned in
the docs[1]
[1] https://docs.gitlab.com/ee/api/sidekiq_metrics.html#get-the-current-queue-metrics
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The GitHub CI is showing a ResourceWarning, causing our test to fail.
Update test to only look for our UserWarning which should not appear.
What was seen when debugging the GitHub CI:
{message:
ResourceWarning(
"unclosed <socket.socket fd=12, family=AddressFamily.AF_INET, type=SocketKind.SOCK_STREAM, proto=6, laddr=('127.0.0.1', 50862), raddr=('127.0.0.1', 8080)>"
),
category: 'ResourceWarning',
filename: '/home/runner/work/python-gitlab/python-gitlab/.tox/api_func_v4/lib/python3.10/site-packages/urllib3/poolmanager.py',
lineno: 271,
line: None
}
|
|
|
|
|
|
|
| |
Make it so can run just the test_gitlab.py functional test.
For example:
$ tox -e api_func_v4 -- -k test_gitlab.py
|
| |
|
|\
| |
| | |
test(functional): bump GitLab docker image to 15.2.0-ee.0
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Cluster support was deprecated in GitLab 14.5 [1]. And disabled by
default in GitLab 15.0 [2]
* Update docs to mark clusters as deprecated
* Remove testing of clusters
[1] https://docs.gitlab.com/ee/api/project_clusters.html
[2] https://gitlab.com/groups/gitlab-org/configure/-/epics/8
|
| |
| |
| |
| | |
In GitLab >= 15.0 `title` is required when creating a topic.
|
| |
| |
| |
| |
| | |
https://github.com/python-gitlab/python-gitlab/issues/1698 has been
fixed. Add test to show that.
|
|/
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
1. If a value is of type ArrayAttribute then append '[]' to the name
of the value for query parameters (`params`).
This is step 3 in a series of steps of our goal to add full
support for the GitLab API data types[1]:
* array
* hash
* array of hashes
Step one was: commit 5127b1594c00c7364e9af15e42d2e2f2d909449b
Step two was: commit a57334f1930752c70ea15847a39324fa94042460
Fixes: #1698
[1] https://docs.gitlab.com/ee/api/#encoding-api-parameters-of-array-and-hash-types
|
| |
|
| |
|
|
|
|
|
| |
Catch the deprecation warning for the call to `gl.lint()`, so it won't
show up in the log.
|
| |
|
| |
|
|
|
|
| |
Enable the `mypy` `strict_equality` check.
|
| |
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* feat(downloads): allow streaming downloads access to response iterator
Allow access to the underlying response iterator when downloading in
streaming mode by specifying `iterator=True`.
Update type annotations to support this change.
* docs(api-docs): add iterator example to artifact download
Document the usage of the `iterator=True` option when downloading
artifacts
* test(packages): add tests for streaming downloads
|
| |
|
| |
|
| |
|
|
|
|
|
|
|
|
|
|
|
| |
`list()`
`as_list=False` is confusing as it doesn't explain what is being
returned. Replace it with `iterator=True` which more clearly explains
to the user that an iterator/generator will be returned.
This maintains backward compatibility with `as_list` but does issue a
DeprecationWarning if `as_list` is set.
|
| |
|
|\
| |
| | |
feat(objects): support getting project/group deploy tokens by id
|
| | |
|
|\ \
| | |
| | | |
feat(user): support getting user SSH key by id
|
| |/ |
|
|/
|
|
|
|
|
|
|
|
|
| |
A common cause of issues filed and questions raised is that a user
will call a `list()` method and only get 20 items. As this is the
default maximum of items that will be returned from a `list()` method.
To help with this we now emit a warning when the result from a
`list()` method is greater-than or equal to 20 (or the specified
`per_page` value) and the user is not using either `all=True`,
`all=False`, `as_list=False`, or `page=X`.
|
|
|
|
| |
This reverts commit e3035a799a484f8d6c460f57e57d4b59217cd6de.
|
|
|
|
|
|
|
|
|
|
|
|
| |
When pagination occurs python-gitlab uses the URL provided by the
GitLab server to use for the next request.
We had previously set the GitLab server configuraiton to say its URL
was `http://gitlab.test` which is not in DNS. Set the hostname
in the URL to `http://127.0.0.1:8080` which is the correct URL for the
GitLab server to be accessed while doing functional tests.
Closes: #1877
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
The usage of deleting was incorrect according to the current API.
Remove custom `delete()` method as not needed.
Add tests to show it works with labels needing to be encoded.
Also enable the test_group_labels() test function. Previously it was
disabled.
Add ability to do a `get()` for group labels.
Closes: #1867
|
| |
|