GeezerVison

The World Wide Web, restored to 800 pixels of glory.

GeezerVison is showing github.com — Integration test marks by RussTorres · Pull Request #180 · AllenInstitute/render-python

Conversation

RussTorres commented Nov 12, 2026

Copy link

Copy Markdown

Collaborator

It is useful to represent different levels of integration when testing in different environments.

This PR adds three pytest marks that can be applied to methods in integration tests to include/exclude based on the available level of integration:

render_service : There is a running render webservice defined to use
java_client : There is a functioning java client specified
shared_data : data specified in testing is in a location available to both the render webservice and the client tests

The PR also officially adds stable test files to the repo that have previously been external. These are small enough that they can be tracked in the git repo and can make it easier to test in varied environments.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions

RussTorres added 8 commits

November 6, 2026 15:45


          pyproject: remove pytest-pep8

7815f8f


          test_data: include small test data that hasn't changed in repo

4cb8d9a


          integration tests: add pytest marks for different levels of integration

7dc1b1a


          integration: add ws connection smoke test

4c878c0


          client integration: add pytest marks and use conftest render

0c6cb85


          coordinate integration: fix match definition and add pytest integrati…

a2b2676

…on marks


          stack integration: add integration pytest marks

65ae8fc


          pointmatch integration: fix test pointmatch format and add pytest int…

ea453c6

…egration marks

RussTorres changed the base branch from main to develop

November 12, 2026 10:02

github-code-quality Bot found potential problems

Nov 12, 2026

View reviewed changes

Comment thread

integration_tests/test_multichannel.py

Comment on lines +22 to +23

		# def render():
		# return renderapi.connect(**render_params)

Comment thread

integration_tests/test_coordinate_integrated.py

Comment on lines +24 to +30

		# def render():
		# render_test_parameters = {
		# 'host': render_host,
		# 'port': render_port,
		# 'owner': 'test_coordinate',
		# 'project': 'test_coordinate_project',
		# 'client_scripts': client_script_location

Comment thread

integration_tests/test_pointmatch_integrated.py

Comment on lines +91 to +97

		# def render():
		# render_test_parameters = {
		# 'host': render_host,
		# 'port': render_port,
		# 'owner': 'test',
		# 'project': 'test_pointmatch_project',
		# 'client_scripts': client_script_location

Comment thread

integration_tests/test_client_integrated.py

Comment on lines +29 to +36

		# def render():
		# render_test_parameters = {
		# 'host': render_host,
		# 'port': render_port,
		# 'owner': 'test',
		# 'project': 'test_project',
		# 'client_scripts': client_script_location}
		# return renderapi.render.connect(**render_test_parameters)

Comment thread

integration_tests/test_stack_integrated.py

Comment on lines +23 to +30

		# def render():
		# render_test_parameters = {
		# 'host': render_host,
		# 'port': render_port,
		# 'owner': 'test',
		# 'project': 'test_project',
		# 'client_scripts': client_script_location}
		# return renderapi.render.connect(**render_test_parameters)

Reviewers

1 more reviewer

github-code-quality[bot]

github-code-quality[bot] left review comments

Reviewers whose approvals may not affect merge requirements

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

1 participant

Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.

You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.