From 3bd51ea4e4fdb2565f5a9b66ab6f76e878a0ba27 Mon Sep 17 00:00:00 2001
From: Ben Gruver <bgruv@google.com>
Date: Fri, 21 Mar 2014 20:19:37 -0700
Subject: [PATCH] Implement SmaliAnnotation.getQualifiedName() and related
 tests

---
 .../jf/smalidea/psi/impl/SmaliAnnotation.java |  6 +++++-
 .../smalidea/SmaliClassModifierListTest.java  | 21 ++++++++++++++++---
 2 files changed, 23 insertions(+), 4 deletions(-)

diff --git a/smalidea/src/main/java/org/jf/smalidea/psi/impl/SmaliAnnotation.java b/smalidea/src/main/java/org/jf/smalidea/psi/impl/SmaliAnnotation.java
index eee64b06..291f04c2 100644
--- a/smalidea/src/main/java/org/jf/smalidea/psi/impl/SmaliAnnotation.java
+++ b/smalidea/src/main/java/org/jf/smalidea/psi/impl/SmaliAnnotation.java
@@ -58,7 +58,11 @@ public class SmaliAnnotation extends SmaliStubBasedPsiElement<SmaliAnnotationStu
     }
 
     @Nullable @Override public String getQualifiedName() {
-        return null;
+        SmaliClassTypeElement classType = findChildByClass(SmaliClassTypeElement.class);
+        if (classType == null) {
+            return null;
+        }
+        return classType.getJavaType();
     }
 
     @Nullable @Override public PsiJavaCodeReferenceElement getNameReferenceElement() {
diff --git a/smalidea/src/test/java/org/jf/smalidea/SmaliClassModifierListTest.java b/smalidea/src/test/java/org/jf/smalidea/SmaliClassModifierListTest.java
index d88f5bf3..85d58efd 100644
--- a/smalidea/src/test/java/org/jf/smalidea/SmaliClassModifierListTest.java
+++ b/smalidea/src/test/java/org/jf/smalidea/SmaliClassModifierListTest.java
@@ -148,8 +148,7 @@ public class SmaliClassModifierListTest extends LightCodeInsightFixtureTestCase
         SmaliAnnotation[] annotations = modifierList.getAnnotations();
         Assert.assertEquals(1, annotations.length);
 
-        // TODO: use PsiAnnotation.getQualifiedName() instead, once it has been implemented
-        Assert.assertEquals(".annotation Lmy/pkg/anno; .end annotation", annotations[0].getText());
+        Assert.assertEquals("my.pkg.anno", annotations[0].getQualifiedName());
 
         SmaliAnnotation[] applicableAnnotations = modifierList.getApplicableAnnotations();
         Assert.assertEquals(1, applicableAnnotations.length);
@@ -168,6 +167,22 @@ public class SmaliClassModifierListTest extends LightCodeInsightFixtureTestCase
         Assert.assertEquals(0, modifierList.getApplicableAnnotations().length);
     }
 
-    // TODO: test modifierList.findAnnotation once PsiAnnotation.getQualifiedName has been implemented
+    public void testFindAnnotation() {
+        final SmaliFile file = (SmaliFile)myFixture.addFileToProject("my/pkg/blah.smali",
+                ".class public final Lmy/pkg/blah;\n" +
+                ".annotation Lanno; .end annotation\n" +
+                ".super Ljava/lang/Object;\n" +
+                ".annotation Lmy/pkg/anno; .end annotation\n" +
+                ".annotation Lmy/pkg/anno2; .end annotation\n" +
+                ".annotation Lmy/pkg/anno3; .end annotation\n");
+
+        SmaliClass smaliClass = file.getPsiClass();
+        SmaliModifierList modifierList = smaliClass.getModifierList();
+
+        SmaliAnnotation smaliAnnotation = modifierList.findAnnotation("my.pkg.anno2");
+        Assert.assertNotNull(smaliAnnotation);
+        Assert.assertEquals("my.pkg.anno2", smaliAnnotation.getQualifiedName());
+    }
+
     // TODO: test modifierList.addAnnotation once implemented
 }