Fix tests (#108)

Flagsmith · Mar 23, 2023 · 0191e6d · 0191e6d
1 parent f57c754
commit 0191e6d
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/src/main/java/com/flagsmith/config/FlagsmithCacheConfig.java b/src/main/java/com/flagsmith/config/FlagsmithCacheConfig.java
@@ -144,8 +144,8 @@ public Builder recordStats() {
      * @param envFlagsCacheKey key to use in the cache for environment level flags
      * @return the Builder
      */
-    public Builder enableEnvLevelCaching(@NonNull String envFlagsCacheKey) {
-      if (StringUtils.isBlank(envFlagsCacheKey)) {
+    public Builder enableEnvLevelCaching(String envFlagsCacheKey) {
+      if (envFlagsCacheKey == null || StringUtils.isBlank(envFlagsCacheKey)) {
         throw new IllegalArgumentException("Missing environment level cache key");
       }
       this.envFlagsCacheKey = envFlagsCacheKey;

diff --git a/src/main/java/com/flagsmith/interfaces/FlagsmithSdk.java b/src/main/java/com/flagsmith/interfaces/FlagsmithSdk.java
@@ -23,7 +23,7 @@ Flags identifyUserWithTraits(
   );
 
   FlagsmithConfig getConfig();
-  
+
   EnvironmentModel getEnvironment();
 
   RequestProcessor getRequestor();
@@ -43,8 +43,8 @@ default FlagsmithCache getCache() {
    * validate user has a valid identifier.
    * @param identifier user identifier
    */
-  default void assertValidUser(@NonNull String identifier) {
-    if (StringUtils.isBlank(identifier)) {
+  default void assertValidUser(String identifier) {
+    if (identifier == null || StringUtils.isBlank(identifier)) {
       throw new IllegalArgumentException("Missing user identifier");
     }
   }