test: Added Adaptor integration tests to validate opening and closing of Houdini.

[erico-aws]

What was the problem/requirement? (What/Why)

There is no integration test for checking that the adaptor can start and stop Houdini. The minimal render tests partially cover this by starting Houdini but we never check that the adaptor closes Houdini when complete.

What was the solution? (How)

Added a few tests to check starting Houdini and closing it. Cancel should close Houdini and the same behaviour should be scene when running as a daemon.

What is the impact of this change?

More confidence in the Adaptors control over the Houdini process.

How was this change tested?

Ran tests:

Please run the integration tests and paste the results below.

bash-5.2$ hatch run integ:test
================================================================================================================ test session starts =================================================================================================================
platform darwin -- Python 3.10.8, pytest-8.4.2, pluggy-1.6.0 -- /Users/erico/Library/Application Support/hatch/env/virtual/deadline-cloud-for-houdini/RJSCNW7O/integ/bin/python
cachedir: .pytest_cache
rootdir: /Volumes/workplace/forks/deadline-cloud-for-houdini
configfile: pyproject.toml
plugins: xdist-3.8.0, cov-7.0.0
1 worker [9 items]     
scheduling tests via LoadScheduling

test/integ/test_houdini_adaptors.py::TestAdaptors::test_minimal_scene_adaptor 
[gw0] [ 11%] PASSED test/integ/test_houdini_adaptors.py::TestAdaptors::test_minimal_scene_adaptor 
test/integ/test_houdini_adaptors.py::TestAdaptors::test_wedge_node_adaptor 
[gw0] [ 22%] PASSED test/integ/test_houdini_adaptors.py::TestAdaptors::test_wedge_node_adaptor 
test/integ/test_houdini_adaptors.py::TestAdaptors::test_render_dependencies_adaptor 
[gw0] [ 33%] PASSED test/integ/test_houdini_adaptors.py::TestAdaptors::test_render_dependencies_adaptor 
test/integ/test_houdini_adaptors.py::TestAdaptors::test_adaptor_open_close 
[gw0] [ 44%] PASSED test/integ/test_houdini_adaptors.py::TestAdaptors::test_adaptor_open_close 
test/integ/test_houdini_adaptors.py::TestAdaptors::test_adaptor_cancel_closes_houdini 
[gw0] [ 55%] PASSED test/integ/test_houdini_adaptors.py::TestAdaptors::test_adaptor_cancel_closes_houdini 
test/integ/test_houdini_adaptors.py::TestAdaptors::test_adaptor_daemon_open_close 
[gw0] [ 66%] PASSED test/integ/test_houdini_adaptors.py::TestAdaptors::test_adaptor_daemon_open_close 
test/integ/test_houdini_submitters.py::TestSubmitters::test_minimal_scene_submitter 
[gw0] [ 77%] PASSED test/integ/test_houdini_submitters.py::TestSubmitters::test_minimal_scene_submitter 
test/integ/test_houdini_submitters.py::TestSubmitters::test_wedge_node_submitter 
[gw0] [ 88%] PASSED test/integ/test_houdini_submitters.py::TestSubmitters::test_wedge_node_submitter 
test/integ/test_houdini_submitters.py::TestSubmitters::test_render_dependencies_submitter 
[gw0] [100%] PASSED test/integ/test_houdini_submitters.py::TestSubmitters::test_render_dependencies_submitter 

================================================================================================================ slowest 5 durations =================================================================================================================
140.66s call     test/integ/test_houdini_adaptors.py::TestAdaptors::test_render_dependencies_adaptor
94.70s call     test/integ/test_houdini_adaptors.py::TestAdaptors::test_wedge_node_adaptor
81.38s call     test/integ/test_houdini_adaptors.py::TestAdaptors::test_minimal_scene_adaptor
37.52s call     test/integ/test_houdini_adaptors.py::TestAdaptors::test_adaptor_daemon_open_close
32.49s call     test/integ/test_houdini_adaptors.py::TestAdaptors::test_adaptor_open_close
=========================================================================================================== 9 passed in 479.34s (0:07:59) ============================================================================================================

Was this change documented?

No

Is this a breaking change?

No

---

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

[miabatta]

should be houdini-openjd

[miabatta]

Should we include some kind of process cleanup if this assert fails?

[erico-aws]

I don't think OpenJD gives us the daemon's process id, so I'm not sure that we can clean it up if this fails.

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud