Skip to content

Commit

Permalink
Added tests for FieldAccessExpr#resolveAccessedValue()
Browse files Browse the repository at this point in the history
  • Loading branch information
malteskoruppa committed May 24, 2018
1 parent d6ba23f commit d772798
Show file tree
Hide file tree
Showing 2 changed files with 61 additions and 6 deletions.
Expand Up @@ -100,6 +100,10 @@ public com.github.javaparser.ast.body.FieldDeclaration getWrappedNode() {
return wrappedNode; return wrappedNode;
} }


public VariableDeclarator getVariableDeclarator() {
return variableDeclarator;
}

@Override @Override
public String toString() { public String toString() {
return "JPField{" + getName() + "}"; return "JPField{" + getName() + "}";
Expand Down
Expand Up @@ -16,8 +16,12 @@


package com.github.javaparser.symbolsolver.resolution; package com.github.javaparser.symbolsolver.resolution;


import com.github.javaparser.JavaParser;
import com.github.javaparser.ast.CompilationUnit; import com.github.javaparser.ast.CompilationUnit;
import com.github.javaparser.ast.body.ClassOrInterfaceDeclaration;
import com.github.javaparser.ast.body.EnumDeclaration;
import com.github.javaparser.ast.body.MethodDeclaration; import com.github.javaparser.ast.body.MethodDeclaration;
import com.github.javaparser.ast.body.VariableDeclarator;
import com.github.javaparser.ast.expr.AssignExpr; import com.github.javaparser.ast.expr.AssignExpr;
import com.github.javaparser.ast.expr.Expression; import com.github.javaparser.ast.expr.Expression;
import com.github.javaparser.ast.expr.FieldAccessExpr; import com.github.javaparser.ast.expr.FieldAccessExpr;
Expand All @@ -26,8 +30,10 @@
import com.github.javaparser.ast.stmt.ReturnStmt; import com.github.javaparser.ast.stmt.ReturnStmt;
import com.github.javaparser.resolution.declarations.ResolvedValueDeclaration; import com.github.javaparser.resolution.declarations.ResolvedValueDeclaration;
import com.github.javaparser.resolution.types.ResolvedType; import com.github.javaparser.resolution.types.ResolvedType;
import com.github.javaparser.symbolsolver.JavaSymbolSolver;
import com.github.javaparser.symbolsolver.javaparser.Navigator; import com.github.javaparser.symbolsolver.javaparser.Navigator;
import com.github.javaparser.symbolsolver.javaparsermodel.JavaParserFacade; import com.github.javaparser.symbolsolver.javaparsermodel.JavaParserFacade;
import com.github.javaparser.symbolsolver.javaparsermodel.declarations.JavaParserFieldDeclaration;
import com.github.javaparser.symbolsolver.model.resolution.SymbolReference; import com.github.javaparser.symbolsolver.model.resolution.SymbolReference;
import com.github.javaparser.symbolsolver.resolution.typesolvers.CombinedTypeSolver; import com.github.javaparser.symbolsolver.resolution.typesolvers.CombinedTypeSolver;
import com.github.javaparser.symbolsolver.resolution.typesolvers.JavaParserTypeSolver; import com.github.javaparser.symbolsolver.resolution.typesolvers.JavaParserTypeSolver;
Expand All @@ -44,7 +50,7 @@ public class FieldsResolutionTest extends AbstractResolutionTest {
@Test @Test
public void accessClassFieldThroughThis() { public void accessClassFieldThroughThis() {
CompilationUnit cu = parseSample("AccessClassMemberThroughThis"); CompilationUnit cu = parseSample("AccessClassMemberThroughThis");
com.github.javaparser.ast.body.ClassOrInterfaceDeclaration clazz = Navigator.demandClass(cu, "AccessClassMemberThroughThis"); ClassOrInterfaceDeclaration clazz = Navigator.demandClass(cu, "AccessClassMemberThroughThis");
MethodDeclaration method = Navigator.demandMethod(clazz, "getLabel2"); MethodDeclaration method = Navigator.demandMethod(clazz, "getLabel2");
ReturnStmt returnStmt = (ReturnStmt) method.getBody().get().getStatements().get(0); ReturnStmt returnStmt = (ReturnStmt) method.getBody().get().getStatements().get(0);
Expression expression = returnStmt.getExpression().get(); Expression expression = returnStmt.getExpression().get();
Expand All @@ -56,7 +62,7 @@ public void accessClassFieldThroughThis() {
@Test @Test
public void accessClassFieldThroughThisWithCompetingSymbolInParentContext() { public void accessClassFieldThroughThisWithCompetingSymbolInParentContext() {
CompilationUnit cu = parseSample("AccessClassMemberThroughThis"); CompilationUnit cu = parseSample("AccessClassMemberThroughThis");
com.github.javaparser.ast.body.ClassOrInterfaceDeclaration clazz = Navigator.demandClass(cu, "AccessClassMemberThroughThis"); ClassOrInterfaceDeclaration clazz = Navigator.demandClass(cu, "AccessClassMemberThroughThis");
MethodDeclaration method = Navigator.demandMethod(clazz, "setLabel"); MethodDeclaration method = Navigator.demandMethod(clazz, "setLabel");
ExpressionStmt expressionStmt = (ExpressionStmt) method.getBody().get().getStatements().get(0); ExpressionStmt expressionStmt = (ExpressionStmt) method.getBody().get().getStatements().get(0);
AssignExpr assignExpr = (AssignExpr) expressionStmt.getExpression(); AssignExpr assignExpr = (AssignExpr) expressionStmt.getExpression();
Expand All @@ -74,7 +80,7 @@ public void accessClassFieldThroughThisWithCompetingSymbolInParentContext() {
@Test @Test
public void accessEnumFieldThroughThis() { public void accessEnumFieldThroughThis() {
CompilationUnit cu = parseSample("AccessEnumMemberThroughThis"); CompilationUnit cu = parseSample("AccessEnumMemberThroughThis");
com.github.javaparser.ast.body.EnumDeclaration enumDecl = Navigator.demandEnum(cu, "AccessEnumMemberThroughThis"); EnumDeclaration enumDecl = Navigator.demandEnum(cu, "AccessEnumMemberThroughThis");
MethodDeclaration method = Navigator.demandMethod(enumDecl, "getLabel"); MethodDeclaration method = Navigator.demandMethod(enumDecl, "getLabel");
SimpleName expression = Navigator.findSimpleName(method, "label").get(); SimpleName expression = Navigator.findSimpleName(method, "label").get();


Expand All @@ -86,7 +92,7 @@ public void accessEnumFieldThroughThis() {
@Test @Test
public void accessEnumMethodThroughThis() { public void accessEnumMethodThroughThis() {
CompilationUnit cu = parseSample("AccessEnumMemberThroughThis"); CompilationUnit cu = parseSample("AccessEnumMemberThroughThis");
com.github.javaparser.ast.body.EnumDeclaration enumDecl = Navigator.demandEnum(cu, "AccessEnumMemberThroughThis"); EnumDeclaration enumDecl = Navigator.demandEnum(cu, "AccessEnumMemberThroughThis");
MethodDeclaration method = Navigator.demandMethod(enumDecl, "getLabel2"); MethodDeclaration method = Navigator.demandMethod(enumDecl, "getLabel2");
ReturnStmt returnStmt = (ReturnStmt) method.getBody().get().getStatements().get(0); ReturnStmt returnStmt = (ReturnStmt) method.getBody().get().getStatements().get(0);
Expression expression = returnStmt.getExpression().get(); Expression expression = returnStmt.getExpression().get();
Expand All @@ -96,14 +102,59 @@ public void accessEnumMethodThroughThis() {
} }


@Test @Test
public void accessFieldThroughSuper() { public void accessClassFieldThroughSuper() {
CompilationUnit cu = parseSample("AccessThroughSuper"); CompilationUnit cu = parseSample("AccessThroughSuper");
com.github.javaparser.ast.body.ClassOrInterfaceDeclaration clazz = Navigator.demandClass(cu, "AccessThroughSuper.SubClass"); ClassOrInterfaceDeclaration clazz = Navigator.demandClass(cu, "AccessThroughSuper.SubClass");
MethodDeclaration method = Navigator.demandMethod(clazz, "fieldTest"); MethodDeclaration method = Navigator.demandMethod(clazz, "fieldTest");
ReturnStmt returnStmt = (ReturnStmt) method.getBody().get().getStatements().get(0); ReturnStmt returnStmt = (ReturnStmt) method.getBody().get().getStatements().get(0);
Expression expression = returnStmt.getExpression().get(); Expression expression = returnStmt.getExpression().get();


ResolvedType ref = JavaParserFacade.get(new ReflectionTypeSolver()).getType(expression); ResolvedType ref = JavaParserFacade.get(new ReflectionTypeSolver()).getType(expression);
assertEquals("java.lang.String", ref.describe()); assertEquals("java.lang.String", ref.describe());
} }

@Test
public void resolveClassFieldThroughThis() {
// configure symbol solver before parsing
JavaParser.getStaticConfiguration().setSymbolResolver(new JavaSymbolSolver(new ReflectionTypeSolver()));

// parse compilation unit and get field access expression
CompilationUnit cu = parseSample("AccessClassMemberThroughThis");
ClassOrInterfaceDeclaration clazz = Navigator.demandClass(cu, "AccessClassMemberThroughThis");
MethodDeclaration method = Navigator.demandMethod(clazz, "getLabel2");
ReturnStmt returnStmt = (ReturnStmt) method.getBody().get().getStatements().get(0);
FieldAccessExpr expression = returnStmt.getExpression().get().asFieldAccessExpr();

// resolve field access expression
ResolvedValueDeclaration resolvedValueDeclaration = expression.resolveAccessedValue();

// get expected field declaration
VariableDeclarator variableDeclarator = Navigator.demandField(clazz, "label");

// check that the expected field declaration equals the resolved field declaration
assertEquals(variableDeclarator, ((JavaParserFieldDeclaration) resolvedValueDeclaration).getVariableDeclarator());
}

@Test
public void resolveClassFieldThroughSuper() {
// configure symbol solver before parsing
JavaParser.getStaticConfiguration().setSymbolResolver(new JavaSymbolSolver(new ReflectionTypeSolver()));

// parse compilation unit and get field access expression
CompilationUnit cu = parseSample("AccessThroughSuper");
ClassOrInterfaceDeclaration clazz = Navigator.demandClass(cu, "AccessThroughSuper.SubClass");
MethodDeclaration method = Navigator.demandMethod(clazz, "fieldTest");
ReturnStmt returnStmt = (ReturnStmt) method.getBody().get().getStatements().get(0);
FieldAccessExpr expression = returnStmt.getExpression().get().asFieldAccessExpr();

// resolve field access expression
ResolvedValueDeclaration resolvedValueDeclaration = expression.resolveAccessedValue();

// get expected field declaration
clazz = Navigator.demandClass(cu, "AccessThroughSuper.SuperClass");
VariableDeclarator variableDeclarator = Navigator.demandField(clazz, "field");

// check that the expected field declaration equals the resolved field declaration
assertEquals(variableDeclarator, ((JavaParserFieldDeclaration) resolvedValueDeclaration).getVariableDeclarator());
}
} }

0 comments on commit d772798

Please sign in to comment.