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

Add option robot-enabled (default *false*) to support robotframework-lsp #23

Merged
merged 3 commits into from
Jun 30, 2023

Conversation

datakurre
Copy link
Contributor

@datakurre datakurre commented Jun 29, 2023

What would it require to include this in the recipe?

From README:

robot-enabled
    Required: No

    Default: False

    Generate setting `robot.python.env` with buildout eggs for Robot Framework Language Server.
    Generate task **Start Plone Test Server** into `tasks.json`.
    Generate task **Robot Framework: Launch Template** into `launch.json` for Robot Framework Language Server.

How to try this out:

plone/buildout.coredev@ac93a45

  1. robotframework == 6.0.2 (or < 6.1) must be installed directly into virtualenv (robotframework-lsp supports robotframework 6.1 only by 1.11.0; see below)
  2. robotframework-lsp version 1.10.2 must be installed into VSCode (1.11.0 has a blocking issue latest version RF LS no longer respects the values of robot.pythonpath  robocorp/robotframework-lsp#947)
  3. add robot-enabled = true for this recipe configuration and run buildout

Now VSCode should be able to provide highlighting and code navigation for robot tests.

To run and debug Robot tests directly from VSCode:

  1. Start VSCode task Start Plone Test Server (configured by this recipe change) and type the full Python path for you Plone Robot test fixture (e.g. plone.app.discussion.testing.PLONE_APP_DISCUSSION_ROBOT_TESTING)
  2. Have fun by executing Robot tests directly from VSCode.

Right now this hardcodes port 55001 for Plone Test Server and 49999 for Plone Test Server listener for setting up and tearing down test fixtures.

@nazrulworld
Copy link
Collaborator

Thanks a lot @datakurre . I will do a release as soon as possible.

@nazrulworld nazrulworld merged commit fcf7fde into collective:master Jun 30, 2023
@mauritsvanrees
Copy link
Member

Could we have a release of this please?

Or could you invite the collective PyPI user as owner?

@nazrulworld
Copy link
Collaborator

@mauritsvanrees I think there is a new release already (associated with this PR).
And also I have sent an invitation for this user https://pypi.org/user/collective/

@mauritsvanrees
Copy link
Member

Thanks!

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.

3 participants