fix: harOpen error if no har in zip file (#473)

Author: canstand

channel.go

@@ -1,9 +1,5 @@
 package playwright
 
-import (
-	"reflect"
-)
-
 type channel struct {
 	eventEmitter
 	guid       string
@@ -40,14 +36,11 @@ func (c *channel) innerSend(method string, returnAsDict bool, options ...interfa
 	if returnAsDict {
 		return result, nil
 	}
-	if reflect.TypeOf(result).Kind() == reflect.Map {
-		mapV := result.(map[string]interface{})
-		if len(mapV) == 0 {
-			return nil, nil
-		}
+	if mapV, ok := result.(map[string]interface{}); ok && len(mapV) <= 1 {
 		for key := range mapV {
 			return mapV[key], nil
 		}
+		return nil, nil
 	}
 	return result, nil
 }

local_utils.go

@@ -44,15 +44,21 @@ func (l *localUtilsImpl) Zip(options localUtilsZipOptions) (interface{}, error)
 }
 
 func (l *localUtilsImpl) HarOpen(file string) (string, error) {
-	harId, err := l.channel.Send("harOpen", []map[string]interface{}{
+	result, err := l.channel.SendReturnAsDict("harOpen", []map[string]interface{}{
 		{
 			"file": file,
 		},
 	})
-	if harId == nil {
-		return "", err
+	if err == nil {
+		value := result.(map[string]interface{})
+		if harId, ok := value["harId"]; ok {
+			return harId.(string), nil
+		}
+		if err, ok := value["error"]; ok {
+			return "", fmt.Errorf("%w:%v", ErrPlaywright, err)
+		}
 	}
-	return harId.(string), err
+	return "", err
 }
 
 func (l *localUtilsImpl) HarLookup(option harLookupOptions) (*harLookupResult, error) {

tests/har_test.go

@@ -786,3 +786,9 @@ func TestShouldUpdateExtractedHarZipForPage(t *testing.T) {
 	require.Contains(t, string(body), "hello, world!")
 	require.NoError(t, expect.Locator(page2.Locator("body")).ToHaveCSS("background-color", "rgb(255, 192, 203)"))
 }
+
+func TestShouldErrorWhenWrongZipFile(t *testing.T) {
+	BeforeEach(t)
+
+	require.Error(t, page.RouteFromHAR(Asset("chromium-linux.zip")))
+}