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

Commit 62daf107 authored by Yan Han's avatar Yan Han
Browse files

Remove <Report Audio Status> tests for TV 

The way we handle these messages changed in b/191230338. These tests no
longer make sense, as we no longer notify AudioService at all when
receiving a <Report Audio Status> message.

Test: atest com.android.server.hdmi
Bug: 195935398
Change-Id: I85ebc3984d8a641e7743d5b2358806def5cffc0f
parent 44414dd2

services/tests/servicestests/src/com/android/server/hdmi/HdmiCecLocalDeviceTvTest.java

+0 −66
Original line number Diff line number Diff line
@@ -29,7 +29,6 @@ import static org.junit.Assert.assertFalse; 
import static org.junit.Assert.assertTrue;

import static org.mockito.ArgumentMatchers.anyInt;

import static org.mockito.Mockito.never;

import static org.mockito.Mockito.times;

import static org.mockito.Mockito.verify;



import android.content.Context;
@@ -539,69 +538,4 @@ public class HdmiCecLocalDeviceTvTest { 


        verify(mAudioManager, never()).setStreamVolume(anyInt(), anyInt(), anyInt());

    }



    @Test

    public void handleReportAudioStatus_SamOnArcOn_setStreamVolumeCalled() {

        mNativeWrapper.setPortConnectionStatus(2, true);

        HdmiCecMessage hdmiCecMessage = HdmiCecMessageBuilder.buildReportPhysicalAddressCommand(

                ADDR_AUDIO_SYSTEM, 0x2000, HdmiDeviceInfo.DEVICE_AUDIO_SYSTEM);

        mNativeWrapper.onCecMessage(hdmiCecMessage);



        HdmiCecFeatureAction systemAudioAutoInitiationAction =

                new SystemAudioAutoInitiationAction(mHdmiCecLocalDeviceTv, ADDR_AUDIO_SYSTEM);

        mHdmiCecLocalDeviceTv.addAndStartAction(systemAudioAutoInitiationAction);



        HdmiCecMessage reportSystemAudioMode =

                HdmiCecMessageBuilder.buildReportSystemAudioMode(

                        ADDR_AUDIO_SYSTEM,

                        mHdmiCecLocalDeviceTv.getDeviceInfo().getLogicalAddress(),

                        true);

        mHdmiControlService.handleCecCommand(reportSystemAudioMode);



        HdmiCecMessage requestArcInitiation = HdmiCecMessageBuilder.buildInitiateArc(

                ADDR_AUDIO_SYSTEM,

                ADDR_TV);

        mNativeWrapper.onCecMessage(requestArcInitiation);



        mTestLooper.dispatchAll();



        // SAM and ARC must be on

        assertTrue(mHdmiCecLocalDeviceTv.isSystemAudioActivated());

        assertTrue(mHdmiCecLocalDeviceTv.isArcEstablished());



        HdmiCecMessage reportAudioStatus = HdmiCecMessageBuilder.buildReportAudioStatus(

                ADDR_AUDIO_SYSTEM,

                ADDR_TV,

                50, // Volume of incoming message does not affect HDMI-CEC logic

                false);

        mNativeWrapper.onCecMessage(reportAudioStatus);



        mTestLooper.dispatchAll();



        verify(mAudioManager, times(1)).setStreamVolume(anyInt(), anyInt(), anyInt());

    }



    @Test

    public void handleReportAudioStatus_SamOff_setStreamVolumeNotCalled() {

        // Emulate Audio device on port 0x1000 (does not support ARC)

        mNativeWrapper.setPortConnectionStatus(1, true);

        HdmiCecMessage hdmiCecMessage = HdmiCecMessageBuilder.buildReportPhysicalAddressCommand(

                ADDR_AUDIO_SYSTEM, 0x1000, HdmiDeviceInfo.DEVICE_AUDIO_SYSTEM);

        mNativeWrapper.onCecMessage(hdmiCecMessage);



        mTestLooper.dispatchAll();



        assertFalse(mHdmiCecLocalDeviceTv.isSystemAudioActivated());



        HdmiCecMessage reportAudioStatus = HdmiCecMessageBuilder.buildReportAudioStatus(

                ADDR_AUDIO_SYSTEM,

                ADDR_TV,

                50, // Volume of incoming message does not affect HDMI-CEC logic

                false);

        mNativeWrapper.onCecMessage(reportAudioStatus);



        mTestLooper.dispatchAll();



        verify(mAudioManager, never()).setStreamVolume(anyInt(), anyInt(), anyInt());

    }

}