Enable unit tests

eclipse-ee4j · Feb 6, 2021 · 36cbe88 · 36cbe88
1 parent 16c1005
commit 36cbe88
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/.github/workflows/build.yml b/.github/workflows/build.yml
@@ -29,4 +29,4 @@ jobs:
       with:
         java-version: ${{ matrix.java }}
     - name: Build with Maven
-      run: mvn --no-transfer-progress -DskipTests -DskipITs install
+      run: mvn --no-transfer-progress install
diff --git a/.gitignore b/.gitignore
@@ -31,7 +31,7 @@ api/**/.project
 /dependencies
 glassfish-8.0/
 nbactions.xml
-/test/javaee8/passthrough/nb-configuration.xml
+nb-configuration.xml
 *.DS_Store
 .idea/
 Mojarra.iml

diff --git a/impl/src/main/java/jakarta/faces/component/ComponentStateHelper.java b/impl/src/main/java/jakarta/faces/component/ComponentStateHelper.java
@@ -188,7 +188,7 @@ public Object eval(Serializable key, Supplier<Object> defaultValueSupplier) {
 
         }
 
-        if (retVal == null) {
+        if (retVal == null && defaultValueSupplier != null) {
             retVal = defaultValueSupplier.get();
         }
-Original file line number
+Diff line change
@@ Expand Up @@
             }
-            if (retVal == null) {
+            if (retVal == null && defaultValueSupplier != null) {
                 retVal = defaultValueSupplier.get();
             }
@@ Expand Down @@