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.

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

[🚧] new multi tests #418

Merged
merged 3 commits into from
Jan 5, 2024
Merged

[🚧] new multi tests #418

merged 3 commits into from
Jan 5, 2024

Conversation

wolfv
Copy link
Member

@wolfv wolfv commented Dec 13, 2023

  • decide on adding pip_check: True / False to PythonTest.
  • Reuse previous environments, ignoring pip
  • Decide on serialization format, should we keep run_test.py or serialize to a JSON?
  • For the Commands test, should we serialize (potential) if selectors so that people can have optional dependencies based on their test platform? E.g.
requirements:
  build:
     - if: build_platform == linux-64
       then:
         - arm-emulator

@wolfv wolfv force-pushed the new-tests branch 5 times, most recently from bacc78b to 7b1db2f Compare December 14, 2023 14:21
@wolfv
Copy link
Member Author

wolfv commented Dec 15, 2023

This is how we are currently serializing the 3 different test types:

PythonTest:

  • run_test.py (this contains the imports that are listed as import bla..., one per line)

CommandsTest:

  • test_time_dependencies.json that contains a dictionary with runandbuild` keys
  • run_test.sh / run_test.bat. If it's an arch'd package, either the .sh or .bat. If it is a noarch package we write both files.

DownstreamTest:

  • downstream_test.json: { "downstream": "package_name" }.

@wolfv
Copy link
Member Author

wolfv commented Jan 4, 2024

The Python tests are now serialized to a python_test.json file instead.

@wolfv wolfv merged commit b5ac7ef into prefix-dev:main Jan 5, 2024
13 checks passed
@wolfv wolfv deleted the new-tests branch January 5, 2024 09:50
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.

1 participant