improve test and add check for unused imports

Signed-off-by: Kavindu Dodanduwa <[email protected]>

Author: Kavindu-Dodan

checkstyle.xml

@@ -63,7 +63,8 @@
                         default="checkstyle-xpath-suppressions.xml" />
             <property name="optional" value="true"/>
         </module>
-    
+
+        <module name="UnusedImports"/>
         <module name="OuterTypeFilename"/>
         <module name="IllegalTokenText">
             <property name="tokens" value="STRING_LITERAL, CHAR_LITERAL"/>

src/test/java/dev/openfeature/sdk/OpenFeatureAPITest.java

@@ -46,16 +46,21 @@ void namedProviderOverwrittenTest() {
 
     @Test
     void providerToMultipleNames() {
-        FeatureProvider provider = new InMemoryProvider(Collections.EMPTY_MAP);
+        FeatureProvider inMemAsEventingProvider = new InMemoryProvider(Collections.EMPTY_MAP);
+        FeatureProvider noOpAsNonEventingProvider = new NoOpProvider();
 
         // register same provider for multiple names & as default provider
-        OpenFeatureAPI.getInstance().setProviderAndWait(provider);
-        OpenFeatureAPI.getInstance().setProviderAndWait("clientA", provider);
-        OpenFeatureAPI.getInstance().setProviderAndWait("clientB", provider);
-
-        assertEquals(provider, OpenFeatureAPI.getInstance().getProvider());
-        assertEquals(provider, OpenFeatureAPI.getInstance().getProvider("clientA"));
-        assertEquals(provider, OpenFeatureAPI.getInstance().getProvider("clientB"));
+        OpenFeatureAPI.getInstance().setProviderAndWait(inMemAsEventingProvider);
+        OpenFeatureAPI.getInstance().setProviderAndWait("clientA", inMemAsEventingProvider);
+        OpenFeatureAPI.getInstance().setProviderAndWait("clientB", inMemAsEventingProvider);
+        OpenFeatureAPI.getInstance().setProviderAndWait("clientC", noOpAsNonEventingProvider);
+        OpenFeatureAPI.getInstance().setProviderAndWait("clientD", noOpAsNonEventingProvider);
+
+        assertEquals(inMemAsEventingProvider, OpenFeatureAPI.getInstance().getProvider());
+        assertEquals(inMemAsEventingProvider, OpenFeatureAPI.getInstance().getProvider("clientA"));
+        assertEquals(inMemAsEventingProvider, OpenFeatureAPI.getInstance().getProvider("clientB"));
+        assertEquals(noOpAsNonEventingProvider, OpenFeatureAPI.getInstance().getProvider("clientC"));
+        assertEquals(noOpAsNonEventingProvider, OpenFeatureAPI.getInstance().getProvider("clientD"));
     }
 
     @Test