Improved handling of incomplete configs array

Author: cmaglie

Diff for: internal/go-configmap/configuration.go

@@ -97,7 +97,7 @@ func tryConversion(current any, desiredType reflect.Type) (any, error) {
 		}
 	case reflect.Array, reflect.Slice:
 		currentArray, ok := current.([]any)
-		if !ok {
+		if !ok && current != nil {
 			break
 		}
 		for i, elem := range currentArray {
@@ -108,6 +108,8 @@ func tryConversion(current any, desiredType reflect.Type) (any, error) {
 			currentArray[i] = newElem
 		}
 		return currentArray, nil
+	default:
+		panic("unhandled type in conversion: " + desiredType.Kind().String())
 	}
 
 	currentTypeString := currentType.String()

Diff for: internal/integrationtest/config/config_test.go

@@ -925,4 +925,12 @@ build.unk: 123
 	out, _, err = cli.Run("config", "get", "locale", "--config-file", invalidConfig.String())
 	require.NoError(t, err)
 	require.Equal(t, "en", strings.TrimSpace(string(out)))
+
+	// Run "config get" with a configuration contatinng a null array
+	nullArrayConfig := paths.New(filepath.Join(tmp, "null_array.yaml"))
+	nullArrayConfig.WriteFile([]byte(`board_manager.additional_urls:`))
+	t.Cleanup(func() { nullArrayConfig.Remove() })
+	out, _, err = cli.Run("config", "get", "locale", "--config-file", invalidConfig.String())
+	require.NoError(t, err)
+	require.Equal(t, "en", strings.TrimSpace(string(out)))
 }