code generated for `date-time` fromat can not parse null response data

Abierto

#997 abierto el 12/3/2024 2 comentarios 4 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Evaluación

Dificultad: 3/5
Tiempo estimado: 1-2 días
Aptitud para principiantes: 45/100

Tipo de issue: Error
Claridad: Bastante claro
Estado de actividad: Estancado
Stack tecnológico: python
Área: api

Línea de trabajo

Comienza localizando la lógica del generador que emite la deserialización de modelos para los campos de OpenAPI con el formato date-time y, a continuación, reproduce el problema utilizando el esquema nullable updateTime y los datos de respuesta proporcionados. Añade una prueba de regresión para los valores date-time null y verifica que los clientes generados acepten la respuesta sin generar TypeError.

Escrito por el modelo de indexación a partir del texto del issue.

Descripción

Describe the bug
code generated for date-time fromat in models file like this for now:

_update_time = d.pop("updateTime", UNSET)
update_time: Union[Unset, datetime.datetime]
if isinstance(_update_time, Unset):
    update_time = UNSET
else:
    update_time = isoparse(_update_time)

if response data of updateTime field is null, the _update_time variable will be set to None, so isoparse function will rasie TypeError except.

self = <dateutil.parser.isoparser.isoparser object at 0x7f94bf7ae950>, dt_str = None

    def _parse_isodate_common(self, dt_str):
>       len_str = len(dt_str)
E       TypeError: object of type 'NoneType' has no len()

../../../../.local/lib/python3.10/site-packages/dateutil/parser/isoparser.py:213: TypeError

OpenAPI Spec File

{
    "updateTime": {
        "type": "string",
        "description": "updateTime",
        "format": "date-time"
    }
}

Desktop (please complete the following information):

OS: windows wsl ubuntu 22.0
Python Version: 3.10.0
openapi-python-client version 0.17.2

Additional context
my suggestion is, adding branch check value is None before call isoparse function, generate code like this:

  _update_time = d.pop("updateTime", UNSET)
  update_time: Union[Unset, datetime.datetime]
  if isinstance(_update_time, Unset):
      update_time = UNSET
  elif _update_time is None:
      update_time = None
  else:
      update_time = isoparse(_update_time)

Lenguaje dominante: Python
Estrellas: 2k
Forks: 293
Merge medio: 34 min
PR fusionados (30 d): 1