Skip to content

[BugFix] Fix suite filtering#116

Merged
andersson1234 merged 1 commit into
mainfrom
fix-filtering
Nov 11, 2025
Merged

[BugFix] Fix suite filtering#116
andersson1234 merged 1 commit into
mainfrom
fix-filtering

Conversation

@douglasdotc
Copy link
Copy Markdown
Collaborator

@douglasdotc douglasdotc commented Nov 11, 2025

Description

Recently there is an issue when we try to run a collection of suite with filtering:

$ VNC_PORT=XX VNC_HOST=duckstation2998 yarf --platform=Vnc ./tests/desktop-installer/ -- --suite "Noble.Entire-Disk"
INFO:yarf.rf_libraries.suite_parser:Selected assets:
  installer.resource
  common/common.resource
  ...
==============================================================================
desktop-installer                                                             
==============================================================================
desktop-installer                                                     | SKIP |
0 tests, 0 passed, 0 failed
==============================================================================
Output:  /tmp/yarf-outdir/output.xml
[ ERROR ] Suite 'desktop-installer' contains no tests.

After investigation, it looks like the way RobotFramework used to accept filters has changed and requires the parent directory.

This PR add processing to the suites passed by the user by adding the directory name (suite.name) to each of the suites before filtering them.

Resolved issues

closes #18

Documentation

Tests

Updated unit test.

Try to run yarf with --suite filter. The error above should be gone.

Signed-off-by: Douglas Chiang <douglas.chiang@canonical.com>
Copy link
Copy Markdown
Contributor

@andersson1234 andersson1234 left a comment

Choose a reason for hiding this comment

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

LGTM!

@andersson1234 andersson1234 merged commit 3e3a6a7 into main Nov 11, 2025
9 checks passed
@andersson1234 andersson1234 deleted the fix-filtering branch November 11, 2025 16:33
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.

--suite doesn't parse and run only the suite

2 participants