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

Enhancement/unit tests #21

Closed
wants to merge 20 commits into from
Closed

Enhancement/unit tests #21

wants to merge 20 commits into from

Conversation

scottmcdonnell
Copy link
Contributor

@scottmcdonnell scottmcdonnell commented Feb 7, 2024 •
edited
Loading

Tests with mock Kitsu data for existing endpoints:

  • [GET] /addons/kitsu/{version}/pairing
  • [POST] /addons/kitsu/{version}/pairing
  • [POST] /addons/kitsu/{version}/push

Tests for:

  • services/processor.fullsync.py

tests can be added for new functions needed in push.py to handle the kitsu listener payloads.

scott added 19 commits December 19, 2023 20:49
kitsu edit tasks are currently unsupported, this fix allows those tas…
038bb9b 
…ks to be skipped, for the moment, to completed the syncing.
catch exceptions on sync_folder and sync_task so they can be logged a…
072bc47 
…nd the sync will continue otherwise the sync stops at the first exception.
add kitsu test folder and client for testing api
fa96ece
added tests for pairing endpoints.
eec94e9
mock kitsu data and tests for service fullsync
02154b4
swap client for ayon_api package
146833b
missing import
683d7b4
tests for push assets and episodes
059944a
test push sequences
39b1f20
test for push shots
1799a92
test push tasks
52560ac
test push update eipsode
4157bd4
Merge remote-tracking branch 'ayon/develop' into kitsu-tests
a904f5e 
# Conflicts:
#	server/kitsu/push.py
#	version.py
test label formatter
85ad5c8
wip
0932048
Merge remote-tracking branch 'ayon/develop' into enhancement/unit-tests
e90fdb3
push update tests
1db769e
test for shot attributes startFrame and endFrame
e745bda
Merge branch 'fix/attrib' into enhancement/unit-tests
79dd33e
@scottmcdonnell scottmcdonnell mentioned this pull request Feb 7, 2024
Closed
@martastain
Copy link
Member

Do you think you could change the structure, so tests would be separated from the rest of the package? Like:

.
├── client
├── create_package.py
├── LICENSE
├── README.md
├── server
├── services
├── tests
│   ├── common
│   │   ├── __init__.py
│   │   └── utils.py
│   ├── pyproject.toml
│   ├── pytest.ini
│   └── tests
│       ├── fixtures.py
│       ├── __init__.py
│       ├── mock_data.py
│       ├── test_fullsync.py
│       ├── test_pairing.py
│       ├── test_push_create.py
│       ├── test_push_update.py
│       └── test_utils.py
└── version.py

@martastain martastain mentioned this pull request Feb 19, 2024
Merged
@scottmcdonnell
Copy link
Contributor Author

superseded by #31

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@martastain martastain Awaiting requested review from martastain

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

Successfully merging this pull request may close these issues.

None yet
2 participants
@scottmcdonnell @martastain