Skip to content

Conversation

JohnK1987
Copy link
Member

Summary of changes

This PR will solve issue USB HID Tests fails #437 - missing python modules for USB tests.
Before were important to install additional Python dependencies for Greentea test manually. The CMake will do it automatically now.
The list of Greentea dependencies were moved from ..tools/requirements-greentea.txt into ..tools/pyproject.toml as project optional dependencies and this part will be installed only with Greentea test.

Impact of changes

N/A

Migration actions required

N/A

Documentation

I did some modification of documentation at

  • drivers/usb/tests/TESTS/usb_device/README.md - instruction about manual installation was removed for windows (not sure what about Linux)
  • Wiki Running-the-Mbed-Greentea-Tests-Locally - some correction of links and removed point about manual installation.

Pull request type

[x] Patch update (Bug fix / Target update / Docs update / Test update / Refactor)
[] Feature update (New feature / Functionality change / New API)
[] Major update (Breaking change E.g. Return code change / API behaviour change)

Test results

[x] No Tests required for this change (E.g docs only update)
[] Covered by existing mbed-os tests (Greentea or Unittest)
[] Tests / results supplied as part of this PR

Add Greentea requirements for USB Device host tests
Refactor Python package installation logic based on test configuration.
Added support for greentea dependencies with a separate stamp file.
Copy link
Collaborator

@multiplemonomials multiplemonomials left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice fix! One tiny comment then good to merge

@JohnK1987 JohnK1987 merged commit 64f9b9d into master Sep 8, 2025
53 checks passed
@JohnK1987 JohnK1987 deleted the GT_req branch September 8, 2025 18:22
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