Access ServiceReference bundle only once 44/110444/3
authorRobert Varga <robert.varga@pantheon.tech>
Sun, 3 Mar 2024 06:39:55 +0000 (07:39 +0100)
committerRobert Varga <robert.varga@pantheon.tech>
Sun, 3 Mar 2024 07:07:45 +0000 (08:07 +0100)
Cache the result of getBundle(), as we will be needing it anyway.

JIRA: ODLPARENT-262
Change-Id: Ic3f12503ec2b81ac20165a9555900ff783e48a0e
Signed-off-by: Robert Varga <robert.varga@pantheon.tech>
features-test-plugin/src/main/java/org/opendaylight/odlparent/features/test/plugin/TestProbe.java

index 7385307548765bd44aab24c940590794a16c14d4..8733f6d0cf1f068fbb326c99c26823accb3fb17a 100644 (file)
@@ -211,8 +211,8 @@ public final class TestProbe {
         // log services of NOK bundles
         try {
             for (var serviceRef : bundleContext.getAllServiceReferences(null, null)) {
-                if (serviceRef.getBundle() != null && nokBundles.contains(serviceRef.getBundle().getBundleId())) {
-                    final var bundle = serviceRef.getBundle();
+                final var bundle = serviceRef.getBundle();
+                if (bundle != null && nokBundles.contains(bundle.getBundleId())) {
                     final var usingBundles = serviceRef.getUsingBundles() == null ? List.of() :
                         Arrays.stream(serviceRef.getUsingBundles()).map(Bundle::getSymbolicName).toList();
                     final var propKeys = serviceRef.getPropertyKeys();