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

Commit 431668ba authored by Treehugger Robot's avatar Treehugger Robot Committed by Android (Google) Code Review
Browse files

Merge "[audio] Fix audio permission fixed package names" into main

parents 0accb935 5f40594a

media/libaudiopermission/NativePermissionController.cpp

+2 −2
Original line number Diff line number Diff line
@@ -39,9 +39,9 @@ static std::optional<std::string> getFixedPackageName(uid_t uid) { 
        case AID_ROOT:

            return "root";

        case AID_SYSTEM:

            return "system";

            return "android";

        case AID_SHELL:

            return "shell";

            return "com.android.shell";

        case AID_MEDIA:

            return "media";

        case AID_AUDIOSERVER:

media/libaudiopermission/tests/NativePermissionControllerTest.cpp

+2 −2
Original line number Diff line number Diff line
@@ -57,7 +57,7 @@ TEST_F(NativePermissionControllerTest, getPackagesForUid_NotPopulated) { 
                IsErrorAnd(BinderStatusMatcher::hasException(EX_ILLEGAL_STATE)));



    // fixed uids should work

    EXPECT_THAT(controller_.getPackagesForUid(1000), IsOkAnd(ElementsAre(std::string{"system"})));

    EXPECT_THAT(controller_.getPackagesForUid(1000), IsOkAnd(ElementsAre(std::string{"android"})));

}



TEST_F(NativePermissionControllerTest, validateUidPackagePair_NotPopulated) {
@@ -66,7 +66,7 @@ TEST_F(NativePermissionControllerTest, validateUidPackagePair_NotPopulated) { 
                IsErrorAnd(BinderStatusMatcher::hasException(EX_ILLEGAL_STATE)));



    // fixed uids should work

    EXPECT_THAT(controller_.validateUidPackagePair(1000, "system"), IsOkAnd(IsTrue()));

    EXPECT_THAT(controller_.validateUidPackagePair(1000, "android"), IsOkAnd(IsTrue()));

}



// ---  Tests for populatePackagesForUids ----