Skip to content

Remove duplicate tests#1237

Closed
robfalck wants to merge 23 commits into
OpenMDAO:masterfrom
robfalck:remove_duplicate_tests
Closed

Remove duplicate tests#1237
robfalck wants to merge 23 commits into
OpenMDAO:masterfrom
robfalck:remove_duplicate_tests

Conversation

@robfalck

Copy link
Copy Markdown
Contributor

Summary

The old doc testing integration tests stored in examples/*/doc directories are redundant with other tests and the jupyterbook docs. This PR removes the redundant ones and moves the others into test subdirectories.

Related Issues

  • Resolves #

Backwards incompatibilities

None

New Dependencies

None

robfalck added 23 commits May 11, 2026 13:32
…ff check command. (OpenMDAO#1235)

* Added dev-docs for claude and updated pre-commit hook to use local ruff check command.

* Remove some in-flux transcriptions descriptions.

* Removed dymos_2 from tests. Cleaned up some more DYMOS_INCUDE_CHECK_PARTIALS thigns

* test flying robot using IPOPT

* Edits to the test workflow to test openmdao master with the dymos-pixi environment

* run dev-local on macos also

* added ability to save pixi loc file for downloadå

* workflow cleanup

* more updates to action versions in the workflow.

* pixi lock update.

* more workflow cleanup

* actionlint

* edits after feedback from dingraha

* dont include dymos in pixi.toml environment, install explicitly with no-deps

* Address windows ci issues.

* Back to having the pixi environment in pyproject.toml

* More example cleanup to try to get more reliability on CI
@robfalck robfalck force-pushed the remove_duplicate_tests branch from 881aee1 to 586d6e6 Compare May 11, 2026 17:35
@robfalck robfalck closed this May 11, 2026
@coveralls

Copy link
Copy Markdown

Coverage Status

Coverage is 0.0%robfalck:remove_duplicate_tests into OpenMDAO:master. No base build found for OpenMDAO:master.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants