Modified preference initialization mechanism to ignore the initialization if the preference is already available within the underlying shared preference.

Author: buggysofts-com

PreferenceStore/build.gradle

@@ -35,6 +35,6 @@ dependencies {
     implementation 'com.google.code.gson:gson:2.9.1'
 
     testImplementation 'junit:junit:4.13.2'
-    androidTestImplementation 'androidx.test.ext:junit:1.1.3'
-    androidTestImplementation 'androidx.test.espresso:espresso-core:3.4.0'
+    androidTestImplementation 'androidx.test.ext:junit:1.1.4'
+    androidTestImplementation 'androidx.test.espresso:espresso-core:3.5.0'
 }

PreferenceStore/src/androidTest/java/com/buggysofts/preferencestore/ExampleInstrumentedTest.java

@@ -1,15 +1,15 @@
 package com.buggysofts.preferencestore;
 
+import static org.junit.Assert.assertEquals;
+
 import android.content.Context;
 
-import androidx.test.platform.app.InstrumentationRegistry;
 import androidx.test.ext.junit.runners.AndroidJUnit4;
+import androidx.test.platform.app.InstrumentationRegistry;
 
 import org.junit.Test;
 import org.junit.runner.RunWith;
 
-import static org.junit.Assert.*;
-
 /**
  * Instrumented test, which will execute on an Android device.
  *

PreferenceStore/src/main/AndroidManifest.xml

@@ -1,4 +1,4 @@
 <?xml version="1.0" encoding="utf-8"?>
-<manifest xmlns:android="http://schemas.android.com/apk/res/android">
+<manifest>
 
 </manifest>

PreferenceStore/src/main/java/com/buggysofts/preferencestore/BoundedPreference.java

@@ -5,7 +5,7 @@
 
 /**
  * A preference which with a predefined set of non-null values, and a default value.
- * */
+ */
 
 public class BoundedPreference<T> {
     private final String key;
@@ -16,13 +16,13 @@ public class BoundedPreference<T> {
     /**
      * Create a bounded preference.
      * <br>
-     * @param keyName Name of the preference. Should be of the form <b>pref_key_*</b>.
-     * @param desc Description of the preference, its context, purpose & usability, and any other important information. This is not going to be used anywhere, the purpose of this is to remind the user what it is for and how to use it.
-     * @param allValues All the supported values (by this preference). Each of the items must not be null.
-     * @param defaultValueIndex Index of the value that should be used as default.
      *
+     * @param keyName           Name of the preference. Should be of the form <b>pref_key_*</b>.
+     * @param desc              Description of the preference, its context, purpose & usability, and any other important information. This is not going to be used anywhere, the purpose of this is to remind the user what it is for and how to use it.
+     * @param allValues         All the supported values (by this preference). Each of the items must not be null.
+     * @param defaultValueIndex Index of the value that should be used as default.
      * @throws RuntimeException if any contract violation is found.
-     * */
+     */
     public BoundedPreference(@NonNull String keyName,
                              @NonNull String desc,
                              @NonNull T[] allValues,
@@ -32,62 +32,63 @@ public BoundedPreference(@NonNull String keyName,
         this.allValues = allValues;
         this.defaultValueIndex = defaultValueIndex;
 
-        if(key == null){
+        if (key == null) {
             throw new RuntimeException("Key must not be null.");
         }
-        if(allValues == null || allValues.length == 0){
+        if (allValues == null || allValues.length == 0) {
             throw new RuntimeException("Empty value list is not allowed.");
         } else {
             for (int i = 0; i < allValues.length; i++) {
-                if(allValues[i] == null){
+                if (allValues[i] == null) {
                     throw new RuntimeException("Preference values can not be null.");
                 }
             }
         }
-        if(!(defaultValueIndex >= 0 && defaultValueIndex < allValues.length)){
+        if (!(defaultValueIndex >= 0 && defaultValueIndex < allValues.length)) {
             throw new RuntimeException("Invalid default value index. Make sure the default value index lies between 0 and allValue.length - 1.");
         }
     }
 
     /**
      * Get value at the specified index.
+     *
      * @throws RuntimeException if index is invalid.
-     * */
+     */
     @NonNull
-    public T getValueAtIndex(@IntRange(from = 0) int index){
-        if(index > 0 && index < allValues.length){
+    public T getValueAtIndex(@IntRange(from = 0) int index) {
+        if (index > 0 && index < allValues.length) {
             return allValues[index];
         }
         throw new RuntimeException("Invalid value index. Make sure the default value index lies between 0 and allValue.length - 1");
     }
 
     /**
      * Get all the available values.
-     * */
+     */
     @NonNull
     public T[] getAllValues() {
         return allValues;
     }
 
     /**
      * Get the default value of this preference.
-     * */
+     */
     @NonNull
     public T getDefaultValue() {
         return allValues[defaultValueIndex];
     }
 
     /**
      * Get key string.
-     * */
+     */
     @NonNull
     public String getKey() {
         return key;
     }
 
     /**
      * Get key string.
-     * */
+     */
     @NonNull
     public String getDescription() {
         return desc;