Skip to content
This repository has been archived by the owner on Sep 2, 2024. It is now read-only.

Testing: MayaPy Default #5903

Closed

Conversation

tokejepsen
Copy link
Member

Changelog Description

This makes MayaPy the default application group to test when running tests for Maya.

Major change for testing is piping stdout and stderr to files, which should work for non-Python hosts as well.

Split from #5644

Testing notes:

openpype_console runtests C:\Users\tokejepsen\OpenPype\tests\integration\hosts\maya\test_publish_in_maya.py --mongo_url "mongodb://localhost:2707/"
openpype_console runtests C:\Users\tokejepsen\OpenPype\tests\integration\hosts\maya\test_publish_in_maya.py --app_group maya --mongo_url "mongodb://localhost:2707/"

@tokejepsen tokejepsen added the sponsored Client endorsed or requested label Nov 13, 2023
@ynbot ynbot added host: Maya size/S Denotes a PR changes 100-499 lines, ignoring general files type: enhancement Enhancements to existing functionality labels Nov 13, 2023
@iLLiCiTiT
Copy link
Member

iLLiCiTiT commented Nov 13, 2023

If changes in applications.py are only because of testing then I don't agree with them, until you explain them very clearly. The handling of stdout and stderr is there because of Windows UI build (openpype_gui) which has different handling and stdout/strderr of the launched subprocess.

@tokejepsen
Copy link
Member Author

@iLLiCiTiT I've reverted the logic of the stdout and stderr when sys.stdout is None now. Could you have another look?

@mkolar mkolar removed the sponsored Client endorsed or requested label Dec 7, 2023
@mkolar
Copy link
Member

mkolar commented Feb 9, 2024

Because we're splitting OpenPype into ayon-core and individual host addons, this PR would have to be re-created to target one of those.

We're closing it down, but we'll he happy for a new PR to ynput/ayon-core or the host addon repository once it's up.

@mkolar mkolar closed this Feb 9, 2024
@ynbot ynbot added this to the next-patch milestone Feb 9, 2024
@jakubjezek001 jakubjezek001 removed this from the next-patch milestone Feb 15, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
host: Maya port to AYON size/S Denotes a PR changes 100-499 lines, ignoring general files target: AYON type: enhancement Enhancements to existing functionality
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

None yet

5 participants