Skip to content

feat(pycharm): add configuration screen with variable, environment variables and testsuite file support #431

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

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

hasanalpzengin
Copy link

@hasanalpzengin hasanalpzengin commented Apr 18, 2025

Related with the Task #417

This update introduces a visible and functional configuration for test items, enhancing the overall usability of the PyCharm plugin:
Adds new fields to the configuration page for PyCharm:

  • environment variables: to pass into the commandline interface as environment variable if given.
  • variables: robot variables that will be passed to the execution with -v flag. Seperated with comma following the format:
    key1=value1,key2=value2 if given.
  • additional arguments: any arguments that can be passed to the execution if given.
  • suite path options: suite path to select. It will be converted to TestItem and added to the execution with --by-longname if given.

UI View:
image

- environment field
- variables
- additional arguments
- suite path

Signed-off-by: hasanalpzengin <[email protected]>
@d-biehl
Copy link
Member

d-biehl commented Apr 29, 2025

Sorry for the late reply, I needed some vacation from RobotCode ;-)
But, your PR looks really good!!! I need a little more time to take a closer look at it though.
What I do notice is that I would like to be able to execute individual test cases, not just one, and not just complete robot files. This already works, but it is not yet possible to select the test cases in the RunConfig. Maybe you can find a solution?

Translated with DeepL.com (free version)

@hasanalpzengin
Copy link
Author

@d-biehl multiple testcases would already be available with comma seperation in includedTestItems field. However maybe it's a problem of UX, therefore I have changed the field to a CheckBoxList from Jetbrains library for a modern look:
image

However I ask for your opinion about the test suite field. Would you like to keep it or remove it and leave all decision to the Test Items checkbox list ?
Currently both fields remains this means:
if you select a test suite path. We'll only run the tests from this file. However it makes the usage of Test Items list a bit obsolete because if the path is selected, all tests in the given suite path is appended to the selection no matter if they were selected or not.

Signed-off-by: Hasan Alp Zengin <[email protected]>
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