[CTS15_R4] About case android.net.cts.NetworkStatsBinderTest#testAccessUidStatsFromBinder [404427110]

Assigned

Bug

[AOSP] assigned

Status Update

No update yet.

Description

ya...@lenovo.corp-partner.google.com

created issue #1

Mar 18, 2025 09:05AM

Dear Google:

We meet a CTS issue, please help to check.

Steps to reproduce the problem (including sample code if appropriate).

run cts -m CtsNetTestCases -t android.net.cts.NetworkStatsBinderTest#testAccessUidStatsFromBinder

What happened.

The test is failed. Error trace:

03-18 12:07:46.835 10261 13478 13497 E TestRunner: ----- begin exception -----
03-18 12:07:46.835 10261 13478 13497 E TestRunner: java.lang.NoSuchMethodError: No interface method getUidStats(II)J in class Landroid/net/INetworkStatsService; or its super classes (declaration of 'android.net.INetworkStatsService' appears in /apex/com.android.tethering/javalib/framework-connectivity-t.jar)
03-18 12:07:46.835 10261 13478 13497 E TestRunner: 	at android.net.cts.NetworkStatsBinderTest.getUidStatsFromBinder(NetworkStatsBinderTest.java:81)
03-18 12:07:46.835 10261 13478 13497 E TestRunner: 	at android.net.cts.NetworkStatsBinderTest.testAccessUidStatsFromBinder(NetworkStatsBinderTest.java:114)
03-18 12:07:46.835 10261 13478 13497 E TestRunner: 	at java.lang.reflect.Method.invoke(Native Method)
03-18 12:07:46.835 10261 13478 13497 E TestRunner: 	at org.junit.runners.model.FrameworkMethod$1.runReflectiveCall(FrameworkMethod.java:59)
03-18 12:07:46.835 10261 13478 13497 E TestRunner: 	at org.junit.internal.runners.model.ReflectiveCallable.run(ReflectiveCallable.java:12)
03-18 12:07:46.835 10261 13478 13497 E TestRunner: 	at org.junit.runners.model.FrameworkMethod.invokeExplosively(FrameworkMethod.java:61)
03-18 12:07:46.835 10261 13478 13497 E TestRunner: 	at org.junit.internal.runners.statements.InvokeMethod.evaluate(InvokeMethod.java:17)
03-18 12:07:46.835 10261 13478 13497 E TestRunner: 	at org.junit.internal.runners.statements.FailOnTimeout$CallableStatement.call(FailOnTimeout.java:148)
03-18 12:07:46.835 10261 13478 13497 E TestRunner: 	at org.junit.internal.runners.statements.FailOnTimeout$CallableStatement.call(FailOnTimeout.java:142)
03-18 12:07:46.835 10261 13478 13497 E TestRunner: 	at java.util.concurrent.FutureTask.run(FutureTask.java:264)
03-18 12:07:46.835 10261 13478 13497 E TestRunner: 	at java.lang.Thread.run(Thread.java:1012)
03-18 12:07:46.835 10261 13478 13497 E TestRunner: ----- end exception -----

What you think the correct behavior should be.

Recently Google submit the below change, the params and return value of getUidStats API in INetworkStatsService is changed.

https://android-review.googlesource.com/c/platform/packages/modules/Connectivity/+/3313055

Before: long getUidStats(int uid, int type);

After: StatsResult getTotalStats();

The latest CTS tool not contain this submit.

We have tried CTS15_R3, R4 and R4 daily build(2025/3/15), all of them are the same failure. Which CTS tool version can pass this test?

Best Regards, Wenjun Yang

CtsNetTestCases.zip

Restricted

1.1 MB

Download

bugreport-TB571FU_EEA-AP3A.240905.015.A2-2025-03-18-12-10-15.zip

Restricted

5.8 MB

Download

Comments

ya...@lenovo.corp-partner.google.com <ya...@lenovo.corp-partner.google.com> #2Mar 18, 2025 09:15AM

Triage notes: Assigning for a response.

vi...@google.com <vi...@google.com> #3Mar 18, 2025 07:12PM

Assigned to vi...@google.com.

When updating the providers, startProvider() is comparing the new scope produces with the previous scope produced, instead of the current parent scope, to determine if the content of the provider needs to change, ignoring if the parent changed. That has the effect of, if a provider provides a value that is identical to the parent value, the composer thinks that none of the the static composition locals changed and it doesn't need to force updates of the content of the provider.

The temporary work-around for this is to use a compositionLocalOf instead. Using compositionLocalOf is recommended for composition locals that can change and static should are not recommended for values that can change.

However, in the above example, LocalOtherValue only changes once so using a staticCompositionLocal is recommended as it avoids the overhead for tracking reads of a value that rarely changes, making this work-around temporary for LocalOtherValue or similar locals.

ya...@lenovo.corp-partner.google.com <ya...@lenovo.corp-partner.google.com> #4Mar 20, 2025 10:24AM

deleted

ya...@lenovo.corp-partner.google.com <ya...@lenovo.corp-partner.google.com> #5Mar 24, 2025 02:40AM

This issue also affects Material theming changes because they use staticCompositionLocalOf under the hood. And those cannot be changed by a work-around.

Issue 404427110

Description

Issue summary

Comments

ya...@lenovo.corp-partner.google.com <ya...@lenovo.corp-partner.google.com> #2Mar 18, 2025 09:15AM

vi...@google.com <vi...@google.com> #3Mar 18, 2025 07:12PM

ya...@lenovo.corp-partner.google.com <ya...@lenovo.corp-partner.google.com> #4Mar 20, 2025 10:24AM

ya...@lenovo.corp-partner.google.com <ya...@lenovo.corp-partner.google.com> #5Mar 24, 2025 02:40AM

Add comment

Issue metadata