Skip to content

Commit 7ccf697

Browse files
harshitha-cstkharshitha-cstk
committed
Add comprehensive unit tests for Error, ErrorMessages, and exception handling in AssetLibrary to ensure robust error management and functionality across various scenarios.
1 parent a7e3724 commit 7ccf697

3 files changed

Lines changed: 738 additions & 0 deletions

File tree

‎contentstack/src/test/java/com/contentstack/sdk/TestAssetLibraryAdvanced.java‎

Lines changed: 214 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -6,19 +6,26 @@
66
import androidx.test.core.app.ApplicationProvider;
77

88
import org.json.JSONArray;
9+
import org.json.JSONException;
910
import org.json.JSONObject;
1011
import org.junit.Before;
1112
import org.junit.Test;
1213
import org.junit.runner.RunWith;
14+
import org.mockito.Mockito;
1315
import org.robolectric.RobolectricTestRunner;
1416

1517
import java.io.File;
1618
import java.io.FileWriter;
1719
import java.lang.reflect.Method;
1820
import java.util.ArrayList;
21+
import java.util.HashMap;
1922
import java.util.List;
2023

2124
import static org.junit.Assert.*;
25+
import static org.mockito.ArgumentMatchers.any;
26+
import static org.mockito.ArgumentMatchers.anyString;
27+
import static org.mockito.ArgumentMatchers.eq;
28+
import static org.mockito.Mockito.*;
2229

2330
/**
2431
* Comprehensive tests for AssetLibrary class to improve coverage.
@@ -2114,5 +2121,212 @@ public void onCompletion(ResponseType responseType, List<Asset> assets, Error er
21142121
assertNotNull(assetLibrary);
21152122
}
21162123
}
2124+
2125+
@Test
2126+
public void testExceptionHandlingInIncludeCountWithMockedJSONObject() {
2127+
try {
2128+
// Create a spy of JSONObject that throws exception
2129+
JSONObject mockUrlQueries = mock(JSONObject.class);
2130+
when(mockUrlQueries.put(anyString(), any())).thenThrow(new JSONException("Mock exception"));
2131+
2132+
// Inject the mock via reflection
2133+
java.lang.reflect.Field urlQueriesField = AssetLibrary.class.getDeclaredField("urlQueries");
2134+
urlQueriesField.setAccessible(true);
2135+
Object originalUrlQueries = urlQueriesField.get(assetLibrary);
2136+
urlQueriesField.set(assetLibrary, mockUrlQueries);
2137+
2138+
try {
2139+
// This should trigger the exception catch block in includeCount()
2140+
assetLibrary.includeCount();
2141+
2142+
// Verify the exception path was executed (method should not throw)
2143+
assertTrue(true);
2144+
2145+
} finally {
2146+
// Restore original value
2147+
urlQueriesField.set(assetLibrary, originalUrlQueries);
2148+
}
2149+
2150+
} catch (Exception e) {
2151+
// The exception should be caught internally by includeCount
2152+
fail("Exception should be caught internally: " + e.getMessage());
2153+
}
2154+
}
2155+
2156+
@Test
2157+
public void testExceptionHandlingInIncludeRelativeUrlWithMockedJSONObject() {
2158+
try {
2159+
// Create a mock JSONObject that throws exception
2160+
JSONObject mockUrlQueries = mock(JSONObject.class);
2161+
when(mockUrlQueries.put(anyString(), any())).thenThrow(new JSONException("Mock exception"));
2162+
2163+
// Inject the mock via reflection
2164+
java.lang.reflect.Field urlQueriesField = AssetLibrary.class.getDeclaredField("urlQueries");
2165+
urlQueriesField.setAccessible(true);
2166+
Object originalUrlQueries = urlQueriesField.get(assetLibrary);
2167+
urlQueriesField.set(assetLibrary, mockUrlQueries);
2168+
2169+
try {
2170+
// This should trigger the exception catch block in includeRelativeUrl()
2171+
assetLibrary.includeRelativeUrl();
2172+
2173+
// Verify the exception path was executed
2174+
assertTrue(true);
2175+
2176+
} finally {
2177+
// Restore original value
2178+
urlQueriesField.set(assetLibrary, originalUrlQueries);
2179+
}
2180+
2181+
} catch (Exception e) {
2182+
// The exception should be caught internally
2183+
fail("Exception should be caught internally: " + e.getMessage());
2184+
}
2185+
}
2186+
2187+
@Test
2188+
public void testExceptionHandlingInIncludeFallbackWithMockedJSONObject() {
2189+
try {
2190+
// Create a mock JSONObject that throws JSONException
2191+
JSONObject mockUrlQueries = mock(JSONObject.class);
2192+
when(mockUrlQueries.put(anyString(), any())).thenThrow(new JSONException("Mock exception"));
2193+
2194+
// Inject the mock via reflection
2195+
java.lang.reflect.Field urlQueriesField = AssetLibrary.class.getDeclaredField("urlQueries");
2196+
urlQueriesField.setAccessible(true);
2197+
Object originalUrlQueries = urlQueriesField.get(assetLibrary);
2198+
urlQueriesField.set(assetLibrary, mockUrlQueries);
2199+
2200+
try {
2201+
// This should trigger the JSONException catch block in includeFallback()
2202+
assetLibrary.includeFallback();
2203+
2204+
// Verify the exception path was executed
2205+
assertTrue(true);
2206+
2207+
} finally {
2208+
// Restore original value
2209+
urlQueriesField.set(assetLibrary, originalUrlQueries);
2210+
}
2211+
2212+
} catch (Exception e) {
2213+
// The exception should be caught internally and rethrown via throwException
2214+
// which is expected behavior
2215+
assertTrue(e instanceof RuntimeException || e instanceof JSONException);
2216+
}
2217+
}
2218+
2219+
@Test
2220+
public void testExceptionHandlingInIncludeMetadataWithMockedJSONObject() {
2221+
try {
2222+
// Create a mock JSONObject that throws JSONException
2223+
JSONObject mockUrlQueries = mock(JSONObject.class);
2224+
when(mockUrlQueries.put(anyString(), any())).thenThrow(new JSONException("Mock exception"));
2225+
2226+
// Inject the mock via reflection
2227+
java.lang.reflect.Field urlQueriesField = AssetLibrary.class.getDeclaredField("urlQueries");
2228+
urlQueriesField.setAccessible(true);
2229+
Object originalUrlQueries = urlQueriesField.get(assetLibrary);
2230+
urlQueriesField.set(assetLibrary, mockUrlQueries);
2231+
2232+
try {
2233+
// This should trigger the JSONException catch block in includeMetadata()
2234+
assetLibrary.includeMetadata();
2235+
2236+
// Verify the exception path was executed
2237+
assertTrue(true);
2238+
2239+
} finally {
2240+
// Restore original value
2241+
urlQueriesField.set(assetLibrary, originalUrlQueries);
2242+
}
2243+
2244+
} catch (Exception e) {
2245+
// The exception should be caught internally by includeMetadata
2246+
fail("Exception should be caught internally: " + e.getMessage());
2247+
}
2248+
}
2249+
2250+
@Test
2251+
public void testExceptionHandlingInWhereWithMockedJSONObject() {
2252+
try {
2253+
// Create a mock JSONObject that throws JSONException
2254+
JSONObject mockUrlQueries = mock(JSONObject.class);
2255+
JSONObject mockQueryParams = mock(JSONObject.class);
2256+
when(mockQueryParams.put(anyString(), any())).thenThrow(new JSONException("Mock exception"));
2257+
when(mockUrlQueries.put(eq("query"), any(JSONObject.class))).thenReturn(mockUrlQueries);
2258+
2259+
// We need to mock the constructor behavior by replacing urlQueries
2260+
java.lang.reflect.Field urlQueriesField = AssetLibrary.class.getDeclaredField("urlQueries");
2261+
urlQueriesField.setAccessible(true);
2262+
Object originalUrlQueries = urlQueriesField.get(assetLibrary);
2263+
2264+
// Create a real JSONObject but configure it to fail during where()
2265+
JSONObject spyUrlQueries = spy(new JSONObject());
2266+
doThrow(new JSONException("Mock exception")).when(spyUrlQueries).put(eq("query"), any(JSONObject.class));
2267+
urlQueriesField.set(assetLibrary, spyUrlQueries);
2268+
2269+
try {
2270+
// This should trigger the JSONException catch block in where()
2271+
assetLibrary.where("test_key", "test_value");
2272+
2273+
// Verify the exception path was executed
2274+
assertTrue(true);
2275+
2276+
} finally {
2277+
// Restore original value
2278+
urlQueriesField.set(assetLibrary, originalUrlQueries);
2279+
}
2280+
2281+
} catch (Exception e) {
2282+
// The exception should be caught internally by where
2283+
fail("Exception should be caught internally: " + e.getMessage());
2284+
}
2285+
}
2286+
2287+
@Test
2288+
public void testExceptionHandlingInFetchAllCatchBlock() {
2289+
try {
2290+
// Create a new AssetLibrary and set an extreme cache policy
2291+
AssetLibrary testLib = stack.assetLibrary();
2292+
2293+
// Access and modify internal state to trigger exception path
2294+
java.lang.reflect.Field cachePolicyField = AssetLibrary.class.getDeclaredField("cachePolicyForCall");
2295+
cachePolicyField.setAccessible(true);
2296+
2297+
// Create a file that should exist for cache testing
2298+
File cacheDir = new File(context.getCacheDir(), "ContentStack");
2299+
cacheDir.mkdirs();
2300+
File cacheFile = new File(cacheDir, "test_fetch_all_exception.json");
2301+
2302+
// Write valid JSON to cache file
2303+
FileWriter writer = new FileWriter(cacheFile);
2304+
writer.write("{\"assets\": [{\"uid\": \"test123\", \"filename\": \"test.jpg\"}]}");
2305+
writer.close();
2306+
2307+
// Set cache policy and trigger fetchAll with potential exception
2308+
testLib.setCachePolicy(CachePolicy.NETWORK_ONLY);
2309+
2310+
FetchAssetsCallback callback = new FetchAssetsCallback() {
2311+
@Override
2312+
public void onCompletion(ResponseType responseType, List<Asset> assets, Error error) {
2313+
// Callback should be invoked even if exception occurs
2314+
assertNotNull("Callback was invoked", this);
2315+
}
2316+
};
2317+
2318+
// Call fetchAll - this exercises the exception handling path
2319+
testLib.fetchAll(callback);
2320+
2321+
// Clean up
2322+
cacheFile.delete();
2323+
2324+
assertTrue(true);
2325+
2326+
} catch (Exception e) {
2327+
// Exception might occur during setup, which is acceptable
2328+
assertNotNull(assetLibrary);
2329+
}
2330+
}
21172331
}
21182332

0 commit comments

Comments
 (0)