Skip to content

Refactor test-module workflow to run tests directly#180

Open
stephdl wants to merge 1 commit into
mainfrom
stephdl-patch-5
Open

Refactor test-module workflow to run tests directly#180
stephdl wants to merge 1 commit into
mainfrom
stephdl-patch-5

Conversation

@stephdl

@stephdl stephdl commented Jun 18, 2026

Copy link
Copy Markdown
Collaborator

This pull request simplifies the workflow configuration in .github/workflows/test-module.yml by consolidating multiple jobs into a single job and updating the way tests are run. The most important changes are:

Workflow simplification:

  • Removed the separate module and run_tests jobs, replacing them with a single run job that handles running tests directly using the test-module.yml@v1 workflow.

Configuration updates:

  • Updated the job inputs to use ui_tests_strategy: on_renovate_ui_change and simplified the debug_shell input assignment.

These changes streamline the CI process, making the workflow easier to maintain and less error-prone.

@stephdl stephdl requested a review from andre8244 June 18, 2026 13:05

@andre8244 andre8244 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

ui_tests_strategy makes sense only if the repository contains UI tests. If you want to add them you can use the changes of this PR as reference: https://github.com/NethServer/ns8-roundcubemail/pull/100/changes

Otherwise, let's replace ui_tests_strategy: on_renovate_ui_change with:

# UI test are not currently implemented
ui_tests_strategy: never

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