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

Commit 88bf26bf authored by Felipe Leme's avatar Felipe Leme Committed by Android (Google) Code Review
Browse files

Merge changes from topic "presubmit-am-67f791696fed41829026111e0282dce4" into sc-v2-dev-plus-aosp 

* changes:
  [automerge] Changes TimingsTraceLog and TimingsTraceAndSlog to be less spammy. 2p: bd3271d1
  Changes TimingsTraceLog and TimingsTraceAndSlog to be less spammy.
parents dcd66a37 6b38ba99

core/java/android/util/TimingsTraceLog.java

+1 −1
Original line number Diff line number Diff line
@@ -127,7 +127,7 @@ public class TimingsTraceLog { 
     * Logs a duration so it can be parsed by external tools for performance reporting.

     */

    public void logDuration(String name, long timeMs) {

        Slog.d(mTag, name + " took to complete: " + timeMs + "ms");

        Slog.v(mTag, name + " took to complete: " + timeMs + "ms");

    }



    /**

core/tests/mockingcoretests/src/android/util/TimingsTraceLogTest.java

+7 −7
Original line number Diff line number Diff line
@@ -123,7 +123,7 @@ public class TimingsTraceLogTest { 
    public void testLogDuration() throws Exception {

        TimingsTraceLog log = new TimingsTraceLog(TAG, TRACE_TAG_APP, 10);

        log.logDuration("logro", 42);

        verify((MockedVoidMethod) () -> Slog.d(eq(TAG), contains("logro took to complete: 42ms")));

        verify((MockedVoidMethod) () -> Slog.v(eq(TAG), contains("logro took to complete: 42ms")));

    }



    @Test
@@ -134,7 +134,7 @@ public class TimingsTraceLogTest { 


        verify((MockedVoidMethod) () -> Trace.traceBegin(TRACE_TAG_APP, "test"));

        verify((MockedVoidMethod) () -> Trace.traceEnd(TRACE_TAG_APP));

        verify((MockedVoidMethod) () -> Slog.d(eq(TAG), matches("test took to complete: \\dms")));

        verify((MockedVoidMethod) () -> Slog.v(eq(TAG), matches("test took to complete: \\dms")));

    }



    @Test
@@ -149,8 +149,8 @@ public class TimingsTraceLogTest { 
        verify((MockedVoidMethod) () -> Trace.traceBegin(TRACE_TAG_APP, "L2"));

        verify((MockedVoidMethod) () -> Trace.traceEnd(TRACE_TAG_APP), times(2)); // L1 and L2



        verify((MockedVoidMethod) () -> Slog.d(eq(TAG), matches("L2 took to complete: \\d+ms")));

        verify((MockedVoidMethod) () -> Slog.d(eq(TAG), matches("L1 took to complete: \\d+ms")));

        verify((MockedVoidMethod) () -> Slog.v(eq(TAG), matches("L2 took to complete: \\d+ms")));

        verify((MockedVoidMethod) () -> Slog.v(eq(TAG), matches("L1 took to complete: \\d+ms")));

    }



    @Test
@@ -170,9 +170,9 @@ public class TimingsTraceLogTest { 
        verify((MockedVoidMethod) () -> Trace.traceBegin(TRACE_TAG_APP, "L3"));

        verify((MockedVoidMethod) () -> Trace.traceEnd(TRACE_TAG_APP), times(3));



        verify((MockedVoidMethod) () -> Slog.d(eq(TAG), matches("L2 took to complete: \\d+ms")));

        verify((MockedVoidMethod) () -> Slog.d(eq(TAG), matches("L1 took to complete: \\d+ms")));

        verify((MockedVoidMethod) () -> Slog.d(eq(TAG), matches("L3 took to complete: \\d+ms")),

        verify((MockedVoidMethod) () -> Slog.v(eq(TAG), matches("L2 took to complete: \\d+ms")));

        verify((MockedVoidMethod) () -> Slog.v(eq(TAG), matches("L1 took to complete: \\d+ms")));

        verify((MockedVoidMethod) () -> Slog.v(eq(TAG), matches("L3 took to complete: \\d+ms")),

                never());



        verify((MockedVoidMethod) () -> Slog.w(TAG, "not tracing duration of 'L3' "

services/core/java/com/android/server/utils/TimingsTraceAndSlog.java

+1 −1
Original line number Diff line number Diff line
@@ -80,7 +80,7 @@ public final class TimingsTraceAndSlog extends TimingsTraceLog { 


    @Override

    public void traceBegin(@NonNull String name) {

        Slog.i(mTag, name);

        Slog.d(mTag, name);

        super.traceBegin(name);

    }

services/tests/mockingservicestests/src/com/android/server/utils/TimingsTraceAndSlogTest.java

0 → 100644
+165 −0
Original line number Diff line number Diff line 
/*

 * Copyright (C) 2022 The Android Open Source Project

 *

 * Licensed under the Apache License, Version 2.0 (the "License");

 * you may not use this file except in compliance with the License.

 * You may obtain a copy of the License at

 *

 *      http://www.apache.org/licenses/LICENSE-2.0

 *

 * Unless required by applicable law or agreed to in writing, software

 * distributed under the License is distributed on an "AS IS" BASIS,

 * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.

 * See the License for the specific language governing permissions and

 * limitations under the License.

 */

package com.android.server.utils;



import static android.os.Trace.TRACE_TAG_APP;



import static com.android.dx.mockito.inline.extended.ExtendedMockito.mockitoSession;

import static com.android.dx.mockito.inline.extended.ExtendedMockito.verify;



import static com.google.common.truth.Truth.assertThat;



import static org.mockito.Matchers.anyString;

import static org.mockito.Matchers.contains;

import static org.mockito.Matchers.eq;

import static org.mockito.Matchers.matches;

import static org.mockito.Mockito.never;

import static org.mockito.Mockito.times;



import android.os.Trace;

import android.util.Slog;



import androidx.test.filters.SmallTest;

import androidx.test.runner.AndroidJUnit4;



import com.android.dx.mockito.inline.extended.MockedVoidMethod;



import org.junit.After;

import org.junit.Before;

import org.junit.Test;

import org.junit.runner.RunWith;

import org.mockito.MockitoSession;



import java.util.ArrayList;

import java.util.List;



/**

 * Tests for {@link TimingsTraceAndSlog}.

 *

 * <p>Usage: {@code atest FrameworksMockingServicesTests:TimingsTraceAndSlogTest}

 */

@SmallTest

@RunWith(AndroidJUnit4.class)

public class TimingsTraceAndSlogTest {



    private static final String TAG = "TEST";



    private MockitoSession mSession;



    @Before

    public final void startMockSession() {

        mSession = mockitoSession()

                .spyStatic(Slog.class)

                .spyStatic(Trace.class)

                .startMocking();

    }



    @After

    public final void finishMockSession() {

        mSession.finishMocking();

    }



    @Test

    public void testDifferentThreads() throws Exception {

        TimingsTraceAndSlog log = new TimingsTraceAndSlog(TAG, TRACE_TAG_APP);

        // Should be able to log on the same thread

        log.traceBegin("test");

        log.traceEnd();

        final List<String> errors = new ArrayList<>();

        // Calling from a different thread should fail

        Thread t = new Thread(() -> {

            try {

                log.traceBegin("test");

                errors.add("traceBegin should fail on a different thread");

            } catch (IllegalStateException expected) {

            }

            try {

                log.traceEnd();

                errors.add("traceEnd should fail on a different thread");

            } catch (IllegalStateException expected) {

            }

            // Verify that creating a new log will work

            TimingsTraceAndSlog log2 = new TimingsTraceAndSlog(TAG, TRACE_TAG_APP);

            log2.traceBegin("test");

            log2.traceEnd();



        });

        t.start();

        t.join();

        assertThat(errors).isEmpty();

    }



    @Test

    public void testGetUnfinishedTracesForDebug() {

        TimingsTraceAndSlog log = new TimingsTraceAndSlog(TAG, TRACE_TAG_APP);

        assertThat(log.getUnfinishedTracesForDebug()).isEmpty();



        log.traceBegin("One");

        assertThat(log.getUnfinishedTracesForDebug()).containsExactly("One").inOrder();



        log.traceBegin("Two");

        assertThat(log.getUnfinishedTracesForDebug()).containsExactly("One", "Two").inOrder();



        log.traceEnd();

        assertThat(log.getUnfinishedTracesForDebug()).containsExactly("One").inOrder();



        log.traceEnd();

        assertThat(log.getUnfinishedTracesForDebug()).isEmpty();

    }



    @Test

    public void testLogDuration() throws Exception {

        TimingsTraceAndSlog log = new TimingsTraceAndSlog(TAG, TRACE_TAG_APP);

        log.logDuration("logro", 42);

        verify((MockedVoidMethod) () -> Slog.v(eq(TAG), contains("logro took to complete: 42ms")));

    }



    @Test

    public void testOneLevel() throws Exception {

        TimingsTraceAndSlog log = new TimingsTraceAndSlog(TAG, TRACE_TAG_APP);

        log.traceBegin("test");

        log.traceEnd();



        verify((MockedVoidMethod) () -> Trace.traceBegin(TRACE_TAG_APP, "test"));

        verify((MockedVoidMethod) () -> Trace.traceEnd(TRACE_TAG_APP));

        verify((MockedVoidMethod) () -> Slog.v(eq(TAG), matches("test took to complete: \\dms")));

    }



    @Test

    public void testMultipleLevels() throws Exception {

        TimingsTraceAndSlog log = new TimingsTraceAndSlog(TAG, TRACE_TAG_APP);

        log.traceBegin("L1");

        log.traceBegin("L2");

        log.traceEnd();

        log.traceEnd();



        verify((MockedVoidMethod) () -> Trace.traceBegin(TRACE_TAG_APP, "L1"));

        verify((MockedVoidMethod) () -> Trace.traceBegin(TRACE_TAG_APP, "L2"));

        verify((MockedVoidMethod) () -> Trace.traceEnd(TRACE_TAG_APP), times(2)); // L1 and L2



        verify((MockedVoidMethod) () -> Slog.v(eq(TAG), matches("L2 took to complete: \\d+ms")));

        verify((MockedVoidMethod) () -> Slog.v(eq(TAG), matches("L1 took to complete: \\d+ms")));

    }



    @Test

    public void testEndNoBegin() throws Exception {

        TimingsTraceAndSlog log = new TimingsTraceAndSlog(TAG, TRACE_TAG_APP);

        log.traceEnd();

        verify((MockedVoidMethod) () -> Trace.traceEnd(TRACE_TAG_APP));

        verify((MockedVoidMethod) () -> Slog.d(eq(TAG), anyString()), never());

        verify((MockedVoidMethod) () -> Slog.w(TAG, "traceEnd called more times than traceBegin"));

    }

}