From 46de03a46a25567db62c316e6ea5af5cfdbfe4d9 Mon Sep 17 00:00:00 2001 From: Nicola Corti Date: Wed, 2 Nov 2022 16:02:53 -0700 Subject: [PATCH] Fix test_buck by not using lambdas inside ReactImagePropertyTest (#35181) Summary: Pull Request resolved: https://github.com/facebook/react-native/pull/35181 As the title says, this unblocks `test_buck` by removign the lambdas used inside test. Changelog: [Internal] [Changed] - Fix test_buck by not using lambdas inside ReactImagePropertyTest Reviewed By: cipolleschi Differential Revision: D40958412 fbshipit-source-id: 60b8609a25985230dfd6c4dcdf983dc2a8cfaf64 --- .../views/image/ReactImagePropertyTest.java | 17 +++++++++++++++-- 1 file changed, 15 insertions(+), 2 deletions(-) diff --git a/ReactAndroid/src/test/java/com/facebook/react/views/image/ReactImagePropertyTest.java b/ReactAndroid/src/test/java/com/facebook/react/views/image/ReactImagePropertyTest.java index 779b245d7a..a2e255f1e4 100644 --- a/ReactAndroid/src/test/java/com/facebook/react/views/image/ReactImagePropertyTest.java +++ b/ReactAndroid/src/test/java/com/facebook/react/views/image/ReactImagePropertyTest.java @@ -35,6 +35,7 @@ import org.junit.Rule; import org.junit.Test; import org.junit.runner.RunWith; import org.mockito.invocation.InvocationOnMock; +import org.mockito.stubbing.Answer; import org.powermock.api.mockito.PowerMockito; import org.powermock.core.classloader.annotations.PowerMockIgnore; import org.powermock.core.classloader.annotations.PrepareForTest; @@ -61,9 +62,21 @@ public class ReactImagePropertyTest { public void setup() { PowerMockito.mockStatic(Arguments.class); PowerMockito.when(Arguments.createArray()) - .thenAnswer((InvocationOnMock invocation) -> new JavaOnlyArray()); + .thenAnswer( + new Answer() { + @Override + public Object answer(InvocationOnMock invocation) { + return new JavaOnlyArray(); + } + }); PowerMockito.when(Arguments.createMap()) - .thenAnswer((InvocationOnMock invocation) -> new JavaOnlyMap()); + .thenAnswer( + new Answer() { + @Override + public Object answer(InvocationOnMock invocation) { + return new JavaOnlyMap(); + } + }); // RNLog is stubbed out and the whole class need to be mocked PowerMockito.mockStatic(RNLog.class);