Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 2bd65db1 authored by Deepanshu Gupta's avatar Deepanshu Gupta
Browse files

Layoutlib tests now check methods return value [DO NOT MERGE] 

The change is a cherry-pick from klp-dev.

The change also fixes the errors found by fixing the tests.

Change-Id: I87cef323c4eaee4e9f60475f01156b276593fa50
(cherry-picked from commit 89fdc0c5)
parent 6df2860b

tools/layoutlib/bridge/src/android/graphics/Bitmap_Delegate.java

+2 −1
Original line number Diff line number Diff line
@@ -253,8 +253,9 @@ public final class Bitmap_Delegate { 
    }



    @LayoutlibDelegate

    /*package*/ static void nativeRecycle(int nativeBitmap) {

    /*package*/ static boolean nativeRecycle(int nativeBitmap) {

        sManager.removeJavaReferenceFor(nativeBitmap);

        return true;

    }



    @LayoutlibDelegate

tools/layoutlib/bridge/tests/src/com/android/layoutlib/bridge/TestDelegates.java

+9 −0
Original line number Diff line number Diff line
@@ -129,6 +129,15 @@ public class TestDelegates extends TestCase { 
                                originalClass.getName()),

                        delegateMethod.getAnnotation(LayoutlibDelegate.class));



                // check the return type of the methods match.

                assertTrue(

                        String.format("Delegate method %1$s.%2$s does not match the corresponding " +

                                "framework method which returns %3$s",

                                delegateClass.getName(),

                                getMethodName(delegateMethod),

                                originalMethod.getReturnType().getName()),

                        delegateMethod.getReturnType() == originalMethod.getReturnType());



                // check that the method is static

                assertTrue(

                        String.format(