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

Fix Issue #399 - fix JumpToStepDefinition and StepHyperlinkDetector for multi-project #411

Open
wants to merge 3 commits into
base: v1.x
Choose a base branch
from

Conversation

nddipiazza
Copy link

@nddipiazza nddipiazza commented Jun 5, 2020

when doing the StepHyperlinkDetector and JumpToStepDefinition , expand to fetch from all
projects in the Eclipse workspace, not just the one the gherkin file is from.

Nicholas DiPiazza added 2 commits June 5, 2020 13:52
when doing the StepHyperlinkDetector, expand to fetch from all
projects, not just the one the gherkin file is from.
@nddipiazza nddipiazza changed the title fix step hyperlink detector for multi-project setup fix JumpToStepDefinition and StepHyperlinkDetector for multi-project Jun 5, 2020
@nddipiazza nddipiazza changed the title fix JumpToStepDefinition and StepHyperlinkDetector for multi-project Fix Issue #399 - fix JumpToStepDefinition and StepHyperlinkDetector for multi-project Jun 5, 2020
don't look for stepdefs in closed projects
and ignore failures when trying in case something unexpected happens.
Copy link
Contributor

@laeubi laeubi left a comment

Choose a reason for hiding this comment

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

It is intentional that only in the project and its references a search is performed to prevent exhaustive search in large workspaces.

@nddipiazza
Copy link
Author

@laeubi in our use case, that totally blows up our use of the plugin because our stepdefs span several projects. did you have a situation where this was actually causing a great amount of lag? as far as I can tell, even a 40 sub-project workspace performs fairly well.

@laeubi
Copy link
Contributor

laeubi commented Jun 16, 2020

Cucumber only uses classes on the classpath to search for step-defs and thus we decided to only search step-defs on project and reference. Simply add the dependent projects as project reference or maven dependencies and it should work.

See #302 for example, in fact the current cucumber plugin behaves wrong as it scans all class (that's not why cucumber does) but that's a different story, enhancing this to the whole workspace would not make it any better.

@nddipiazza
Copy link
Author

Yeah I saw that code but unfortunately it does not work for references via maven dependency. No one I work with uses the Eclipse project reference stuff, they use maven dependencies and Eclipse automatically handles project references through that. So perhaps that is the real issue here - it's failing to find the project references via maven deps.

@laeubi
Copy link
Contributor

laeubi commented Jun 16, 2020

Then this must be fixed instead of iterate over all projects in the workspace :-)

@laeubi
Copy link
Contributor

laeubi commented Jan 30, 2021

@nddipiazza I'm currently completely overhaul the whole plugin and like to consider your problem there, can you share a minimal example that illustrates the problem?

@nddipiazza
Copy link
Author

@laeubi laeubi changed the base branch from main to v1.x September 14, 2021 08:01
@laeubi laeubi added the v1.x label Sep 14, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants