Merge pull request #179 from valfirst/keep-browser-optional-in-log

Keep `browser` optional in `log` object

Author: sdstoehr

src/main/java/de/sstoehr/harreader/model/HarLog.java

@@ -59,12 +59,9 @@ public void setCreator(HarCreatorBrowser creator) {
     }
 
     /**
-     * @return Information about the browser used.
+     * @return Information about the browser used, may be null.
      */
     public HarCreatorBrowser getBrowser() {
-        if (browser == null) {
-            browser = new HarCreatorBrowser();
-        }
         return browser;
     }
 

src/test/java/de/sstoehr/harreader/model/HarLogTest.java

@@ -4,6 +4,7 @@
 
 import static org.junit.jupiter.api.Assertions.assertEquals;
 import static org.junit.jupiter.api.Assertions.assertNotNull;
+import static org.junit.jupiter.api.Assertions.assertNull;
 
 import java.util.ArrayList;
 import java.util.List;
@@ -63,7 +64,7 @@ void testCreatorNull() {
     void testBrowserNull() {
         HarLog log = new HarLog();
         log.setBrowser(null);
-        assertNotNull(log.getBrowser());
+        assertNull(log.getBrowser());
     }
 
     @Override