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

Refresh with one depth when calling JDTUtils.getFileOrFolder() #1207

Merged
merged 2 commits into from
Oct 14, 2019

Conversation

jdneo
Copy link
Contributor

@jdneo jdneo commented Oct 12, 2019

fix #1137

Signed-off-by: Sheng Chen [email protected]

Copy link
Contributor

@testforstephen testforstephen left a comment

Choose a reason for hiding this comment

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

This issue has big impact on the debug user, because the new package is not synced to jdt so that jdt builder will not generate .class for the Java files under the new package.

Looking forward this fix.

Copy link
Contributor

@fbricon fbricon left a comment

Choose a reason for hiding this comment

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

Tests ?

Signed-off-by: Sheng Chen <[email protected]>
final int originalMemberNum = ((IFolder) parent).members().length;

File newPackage = new File(project.getLocation().toString(), "/src/org/eclipse/testGetFileOrFolder");
newPackage.mkdirs();
Copy link
Contributor

Choose a reason for hiding this comment

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

will the new package be cleaned up after the test done?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@jdneo
Copy link
Contributor Author

jdneo commented Oct 14, 2019

test this please

@jdneo
Copy link
Contributor Author

jdneo commented Oct 14, 2019

I tried locally, the newly added test will fail if the refresh depth is DEPTH_ZERO. And the test will be success after the change.

@fbricon fbricon merged commit 6e83b9a into eclipse-jdtls:master Oct 14, 2019
@fbricon
Copy link
Contributor

fbricon commented Oct 14, 2019

Thnanks @jdneo !

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.

The project's package fragments are not update to date when adding a new one.
3 participants