Ensure CompilationUnitResolver.accept(...) uses right module

[mickaelistria]

Without this change, addType was always performed against the unnamed module (and some new package bindings & types were created) while it should look up on the current context module.

What it does

Fix CompilationUnitResolver so it properly resolve packages from the visible modules

How to test

I plan to provide a unit test reproducing it in a near future. Basically start from the same structure as ModuleBuilderTests.testAutoModule3 , but instead of calling getWorkingCopy("/mod.one/src/q/X.java", srcX, true), try calling ASTParser to generate the DOM CompilationUnit and check its getProblems().

Author checklist

• I have thoroughly tested my changes
• The change is following the coding conventions
• I have signed the Eclipse Contributor Agreement (ECA)

[mickaelistria]

Marking as draft as this PR can cause some StackOverflowError on LookupEnvironment.getResolvedType()

[mickaelistria]

This was an attempt to fix #2301 , which causes a StackOverflowError with the variation of ResolveTests9 as described on the issue

	public void testModuleInfo_ASTParser() throws CoreException {
		IFile providesFile = createFile("/Resolve/src/provides.java", "public class provides {}");
		IFile modInfo = null;
		try {
			getWorkingCopy(
					"/Resolve/src/test/ITest.java",
					"public interface ITest {}\n");
			getWorkingCopy(
					"/Resolve/src/test/TestClass.java",
					"public class TestClass implements ITest {}\n");

			this.wc = getWorkingCopy(
					"/Resolve/src/module-info.java",
					"module com.test {\n" +
					"  provides p1.Y with ResolveInterface;\n" +
					"}\n");

			ASTParser parser = ASTParser.newParser(AST.getJLSLatest());
			parser.setResolveBindings(true);
			parser.setProject(this.currentProject);
			parser.setSource(this.wc);
			CompilationUnit unit = (CompilationUnit) parser.createAST(null);
			assertArrayEquals(new IProblem[0], unit.getProblems());
		} finally {
			deleteResource(providesFile);
			if (modInfo != null)
				deleteResource(modInfo);
		}
	}