Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[Dependent] [WIP] REST API tests for IAM #4064

Closed
wants to merge 297 commits into from
Closed

[Dependent] [WIP] REST API tests for IAM #4064

wants to merge 297 commits into from

Conversation

sizov-kirill
Copy link
Contributor

@sizov-kirill sizov-kirill commented Dec 22, 2021 •
edited
Loading

Motivation and context

Functionality that was made in PR #3788 has no REST API tests, so this PR should create such tests, is expected that each resource will be have some basic tests:

  • auth
  • cloudstorages
  • comments
  • invitations
  • issues
  • jobs
  • lambda
  • memberships
  • organizations
  • projects
  • restrictions
  • server
  • tasks
  • users

How has this been tested?

Checklist

License

  • I submit my code changes under the same MIT License that covers the project.
    Feel free to contact the maintainers if that's a concern.
  • I have updated the license header for each file (see an example below)
# Copyright (C) 2021 Intel Corporation
#
# SPDX-License-Identifier: MIT

Boris Sekachev and others added 30 commits October 1, 2021 13:02
Create form
6b1b660
Merge branch 'nm/opa_integration' into bs/organizations
d45bc9d
Now ?org=id and ?org=slug works, changed opa rules for organization
8b659e9
Added missed files
04ef1d6
Fix organizations.rego
9a77c8e
Remove PUT method from the organization view
097d6ab
Played with contacts form
e3a9106
Refactoring: replaced method + path by scope
87f4347
@bsekachev
Linked with cvat-core, added actions and reducers
159d3b6
Merge remote-tracking branch 'origin/develop' into nm/opa_integration
9606090
@bsekachev
Added organization page
b769d00
@bsekachev
Merge branch 'nm/opa_integration' into bs/organizations
acbbe2d
@bsekachev
Minor design fixes
2883dc6
@bsekachev
Added activate/deactivate
6a13af0
Refactored permissions system and rego rules
0522383 
+ org=slug and org_id=id
+ disable pagination for organization
@bsekachev
Merge branch 'nm/opa_integration' of github.com:openvinotoolkit/cvat …
98634a7 
…into bs/organizations
Refactoring and fixes for Organization, Membership, Invitation (in
15b7eb4 
progress)
Add role 'Owner' for the organization.
53783fe
Add tests for organizations.rego
55e1bd5
Updated rego files, upgrade django, django-rest, drf, use JSONField for
485a2d2 
Organization.contact (now supported in django, django-rest)
Use "worker" instead of "W" for roles in DB to simplify usage in REST
efbcda5 
framework (otherwise need to implement methods to translate "worker" <->
"W")
@bsekachev
Merge branch 'nm/opa_integration' into bs/organizations
943799d
@bsekachev
Removed 'results' from server response
7fd9c43
Merge remote-tracking branch 'origin/develop' into nm/opa_integration
229a069
Don't add default groups for superuser
857fe3e
Don't assign default groups if a user is updated. Corectly handle users
7dbc141 
without groups.
@bsekachev
Added a couple of modals
730f555
@bsekachev
Merge branch 'nm/opa_integration' into bs/organizations
ccfeecf
Return Organization.owner with nickname and other fields in case of
8a7234c 
SAFE_METHODS.
@bsekachev
Added organization removing, minor fixes
a05fe98
Nikita Manovich and others added 23 commits December 15, 2021 21:11
Make output from psql and tar less verbose
ec7e436
Add one more test for REST API (users_api)
2841672
Improve REST API tests
9a0063f
@bsekachev
Updated client versions and changelog
aab4c8d
@bsekachev
Updated canvas version
59a4686
Merge branch 'develop' into nm/opa_integration
9558d3d
Merge remote-tracking branch 'origin/develop' into nm/opa_integration
4394e22
Cypress. Add test for check issue deleting feature. (#4031)
6c2a352
Merge remote-tracking branch 'origin/develop' into nm/opa_integration
35e2743
Merge branch 'nm/opa_integration' of github.com:openvinotoolkit/cvat …
b553c41 
…into nm/opa_integration
Fix server tests for import/export dataset for a project.
4436536
Fixed linter errors from Bandit
ca26ac1
Fixed warnings from PyLint
ba4cd2b
Fix PyLint warnings
23b2122
Fix pylint warnings
5890e3b
Fix ESLint error and warnings
7937ae6
Fix StyleLinter warnings
c4a8a0a
@bsekachev
Merged develop
1b0a484
@bsekachev
Minor fixes
5990a54
@bsekachev
Removed unused import
5e7d8a8
@bsekachev
Aborted change
3c7b463
@bsekachev
Removed unused import
266dbcc
Update users api tests
d19a1de
@sizov-kirill sizov-kirill changed the title [Dependent] [WIP] REST API test for IAM [Dependent] [WIP] REST API tests for IAM Dec 22, 2021
@sizov-kirill sizov-kirill deleted the sk/rest-api-tests-for-iam branch December 22, 2021 15:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@azhavoro azhavoro Awaiting requested review from azhavoro azhavoro is a code owner

@bsekachev bsekachev Awaiting requested review from bsekachev bsekachev is a code owner

@nmanovic nmanovic Awaiting requested review from nmanovic

@zhiltsov-max zhiltsov-max Awaiting requested review from zhiltsov-max zhiltsov-max is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@sizov-kirill @bsekachev