fix: support annotations for runtime package model

[SirYwell]

Previously, packages where only visited from type references. This means the package of the type given to the scan method isn't necessarily visited, and its annotations aren't visited either.

This fixes it. The test case assumes that the package has the @Deprecated annotation declared in the package-info.java:

spoon/src/test/java/spoon/test/pkg/package-info.java

Lines 5 to 6 in 63104c7

	@Deprecated
	package spoon.test.pkg;

When debugging, I saw that packages (java.util, java.lang) are visited multiple times, as they are always visited whenever a type (new) or type reference (before) is visited. As the values of a package don't change at runtime (well, at least within a module), this doesn't seem necessary. Should we skip that by making sure that packages are only visited once here:

spoon/src/main/java/spoon/support/visitor/java/JavaReflectionTreeBuilder.java

Lines 121 to 129 in 9ff8194

	public void visitPackage(Package aPackage) {
	final CtPackage ctPackage = factory.Package().getOrCreate(aPackage.getName());
	enter(new PackageRuntimeBuilderContext(ctPackage));
	super.visitPackage(aPackage);
	exit();
	contexts.peek().addPackage(ctPackage);
	}

? I'm not really sure if this would cause any issues.

[monperrus]

LGTM! Could you add the contract in the test? Thanks.

[SirYwell]

Could you add the contract in the test?

Oh yeah I forgot that.

[monperrus]

Thanks a lot @SirYwell