operx-cli 0.7.3__tar.gz → 0.7.4__tar.gz

{operx_cli-0.7.3 → operx_cli-0.7.4}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: operx-cli
-Version: 0.7.3
+Version: 0.7.4
 Summary: La línea de comandos de OperX: tu empresa desde la terminal, con tu token de API.
 Requires-Python: >=3.10
 Description-Content-Type: text/markdown
@@ -174,7 +174,9 @@ operx personas ausencia "Ana Ruiz" --tipo permiso --desde 09/11/2026 --horas 08:
 
 operx rutinas listar
 operx rutinas crear "Contar la bodega" --rol Bodeguero      # sin --rol ni --persona, es para todos
+operx rutinas crear "Revisar el taller" --dia lunes --dia jueves  # sin --dia, va todos los días
 operx rutinas cambiar "Contar la bodega" --persona "Ana Ruiz"
+operx rutinas cambiar "Revisar el taller" --todos-los-dias
 operx rutinas quitar "Contar la bodega"                    # sale de los planes desde ahora
 operx rutinas cumplimiento --mes 10/2026                    # quién hizo sus rutinas (administradores)
 

{operx_cli-0.7.3 → operx_cli-0.7.4}/README.md

@@ -160,7 +160,9 @@ operx personas ausencia "Ana Ruiz" --tipo permiso --desde 09/11/2026 --horas 08:
 
 operx rutinas listar
 operx rutinas crear "Contar la bodega" --rol Bodeguero      # sin --rol ni --persona, es para todos
+operx rutinas crear "Revisar el taller" --dia lunes --dia jueves  # sin --dia, va todos los días
 operx rutinas cambiar "Contar la bodega" --persona "Ana Ruiz"
+operx rutinas cambiar "Revisar el taller" --todos-los-dias
 operx rutinas quitar "Contar la bodega"                    # sale de los planes desde ahora
 operx rutinas cumplimiento --mes 10/2026                    # quién hizo sus rutinas (administradores)
 

{operx_cli-0.7.3 → operx_cli-0.7.4}/operx_cli/AGENTES.md

@@ -69,7 +69,8 @@ llamada queda en el historial del token, marcada como hecha desde la CLI.
   - Equipos: `equipos parar`, `reanudar`, `hallazgo` y `hallazgo-a-orden`;
     `lecturas registrar` y `anular`.
   - Personas: `personas importar --confirmar` y `personas ausencia`;
-    `rutinas crear`, `cambiar` y `quitar`.
+    `rutinas crear`, `cambiar` y `quitar`; `campos crear`, `cambiar`,
+    `quitar` y `guardar`.
   - SG-SST y Ambiental: `sst nueva-accion` y `accion-hecha`;
     `ambiental nueva-accion`, `accion-hecha`, `residuo` y `lectura`.
   - `avisos --marcar-leidos`, y `operx api` con POST, PUT, PATCH o DELETE.
@@ -79,7 +80,18 @@ llamada queda en el historial del token, marcada como hecha desde la CLI.
   que ya la tenían la siguen contando.
 - Una rutina es para todos, para algunos roles (`--rol`, por su nombre) o
   para algunas personas (`--persona`); `rutinas cambiar --rol` reemplaza la
-  lista entera de roles.
+  lista entera de roles. Va todos los días o solo algunos de la semana
+  (`--dia lunes --dia jueves`); `rutinas cambiar --dia` reemplaza la lista
+  entera de días y `--todos-los-dias` la deja yendo todos los días.
+- Lo que la ficha de una persona no tiene (fecha de nacimiento, hijos, estado
+  civil…) va en un campo de información extra: `operx campos listar` dice
+  cuáles hay y, si falta, `campos crear "Hijos" --tipo numero` lo agrega para
+  todas las personas (tipos: texto, texto-largo, numero, fecha, si-no y lista,
+  con `--opcion` por cada opción). El tipo no se cambia después. Los valores
+  van por persona: `campos guardar "Ana Ruiz" --valor "Hijos=2"` (solo cambian
+  los campos que digas; `--vaciar Hijos` deja uno vacío) y `campos ver`. Antes
+  de crear un campo, mira que no exista ya. `campos quitar` lo oculta de todas
+  las personas y deja los valores guardados.
 - SG-SST y Ambiental van por empresa: sin `--empresa`, la de la cuenta. Una
   cuenta que lleva el SG-SST o lo ambiental de sus clientes ve las demás con
   `operx sst empresas` o `operx ambiental empresas`.

{operx_cli-0.7.3 → operx_cli-0.7.4}/operx_cli/__init__.py

@@ -2,4 +2,4 @@
 API (backend/users/api_allowlist.py) from a terminal, with a personal API
 token."""
 
-__version__ = "0.7.3"
+__version__ = "0.7.4"

{operx_cli-0.7.3 → operx_cli-0.7.4}/operx_cli/app.py

@@ -14,8 +14,8 @@ from . import __version__, settings, shell
 from .agent_guide import guide_text
 from .client import ApiError
 from .commands import (
-    account, ambiental, articulos, avisos, completar, inventario, lecturas, ordenes, personas, planta, preventivos, raw,
-    registros, reportes, rutinas, search, sst, update,
+    account, ambiental, articulos, avisos, campos, completar, inventario, lecturas, ordenes, personas, planta,
+    preventivos, raw, registros, reportes, rutinas, search, sst, update,
 )
 from .commands.common import CliError, Session
 from .lookup import NoSingleMatch
@@ -156,6 +156,7 @@ app.add_typer(articulos.app, name="articulos")
 app.add_typer(lecturas.app, name="lecturas")
 app.add_typer(personas.app, name="personas")
 app.add_typer(rutinas.app, name="rutinas")
+app.add_typer(campos.app, name="campos")
 app.add_typer(reportes.app, name="reportes")
 app.add_typer(sst.app, name="sst")
 app.add_typer(ambiental.app, name="ambiental")

operx_cli-0.7.4/operx_cli/commands/campos.py

@@ -0,0 +1,249 @@
+"""`operx campos`: Información extra, the fields a company adds to the record
+of its people (a birthday, the children, the marital status...) for what the
+record doesn't have, and their values on each person
+(backend/extra_fields, backend/people/extra_info.py). The fields are
+company-wide; each person has a value or none for every one."""
+
+from decimal import Decimal
+from enum import Enum
+from typing import Annotated, Optional
+
+import typer
+
+from ..lookup import NoSingleMatch, find_in, find_one
+from ..output import print_record, print_rows
+from ..values import decimal_text, iso_date
+from .common import CliError, CsvOption, JsonOption, YesOption, client_of, confirm_write, output_format, print_result
+
+app = typer.Typer(
+    help="Información extra: los campos que tu empresa agrega a la ficha de las personas, y sus valores.",
+    no_args_is_help=True,
+)
+
+# The kinds of record that can have extra fields; for now, people.
+ENTITY = "employee"
+
+FIELD_COLUMNS = [("id", "Id"), ("name", "Campo"), ("type", "Tipo"), ("options", "Opciones")]
+VALUE_COLUMNS = [("name", "Campo"), ("value", "Valor")]
+
+
+class Tipo(str, Enum):
+    texto = "texto"
+    texto_largo = "texto-largo"
+    numero = "numero"
+    fecha = "fecha"
+    si_no = "si-no"
+    lista = "lista"
+
+
+# backend/extra_fields/models.py TYPE_CHOICES
+TYPE_KEYS = {
+    "texto": "text", "texto-largo": "long_text", "numero": "number", "fecha": "date", "si-no": "yes_no", "lista": "choice",
+}
+TYPE_LABELS = {
+    "text": "Texto", "long_text": "Texto largo", "number": "Número", "date": "Fecha", "yes_no": "Sí / No", "choice": "Lista",
+}
+YES_WORDS = {"si", "sí", "s", "true", "1"}
+NO_WORDS = {"no", "n", "false", "0"}
+
+OpcionOption = Annotated[
+    Optional[list[str]],
+    typer.Option("--opcion", help="Una opción de la lista; repítela para varias. Solo para el tipo lista."),
+]
+
+
+def _fields(client):
+    return client.get("extra-fields", {"entity": ENTITY})
+
+
+def _find_field(fields, text):
+    """A field by its id or its exact name."""
+    try:
+        return find_in(fields, text, ["id", "name"], "campo")
+    except NoSingleMatch as error:
+        raise CliError(str(error)) from None
+
+
+def _person(client, text):
+    return find_one(client, "employees", text, ["user_name", "employee_code"], "persona")
+
+
+def _split_pair(pair):
+    name, separator, value = pair.partition("=")
+    if not separator or not name.strip():
+        raise CliError(f"«{pair}» no se entiende: escríbelo como Campo=valor, por ejemplo Hijos=2.")
+    return name.strip(), value.strip()
+
+
+def _number(text):
+    """The API takes a JSON number: 2, 2.5 ("2,5" and "1.234,5" work too)."""
+    try:
+        value = Decimal(decimal_text(text))
+    except ValueError as error:
+        raise CliError(str(error)) from None
+    return int(value) if value == value.to_integral_value() else float(value)
+
+
+def _value_for(field, text):
+    """What the API stores for the words typed, by the field's type."""
+    kind = field["field_type"]
+    if kind == "number":
+        return _number(text)
+    if kind == "date":
+        return iso_date(text)
+    if kind == "yes_no":
+        word = text.strip().casefold()
+        if word in YES_WORDS:
+            return True
+        if word in NO_WORDS:
+            return False
+        raise CliError(f"«{text}» no es sí ni no, en «{field['name']}».")
+    if kind == "choice":
+        wanted = " ".join(text.split()).casefold()
+        for option in field["options"]:
+            if " ".join(option.split()).casefold() == wanted:
+                return option
+        raise CliError(f"«{text}» no es una opción de «{field['name']}». Hay: {', '.join(field['options'])}.")
+    return text
+
+
+def _shown(field):
+    """A value as a person reads it."""
+    value = field["value"]
+    if value is None:
+        return "Sin registrar"
+    if field["field_type"] == "yes_no":
+        return "Sí" if value else "No"
+    if field["field_type"] == "date":
+        year, month, day = str(value).split("-")
+        return f"{day}/{month}/{year}"
+    return str(value)
+
+
+@app.command()
+def listar(
+    ctx: typer.Context,
+    as_json: JsonOption = False,
+    as_csv: CsvOption = False,
+):
+    """Los campos de información extra de las personas: su nombre, su tipo y las opciones de las listas."""
+    output = output_format(as_json, as_csv)
+    fields = _fields(client_of(ctx))
+    if output != "json":
+        fields = [
+            {**field, "type": TYPE_LABELS.get(field["field_type"], field["field_type"]), "options": ", ".join(field["options"])}
+            for field in fields
+        ]
+    print_rows(FIELD_COLUMNS, fields, output)
+
+
+@app.command()
+def crear(
+    ctx: typer.Context,
+    nombre: Annotated[str, typer.Argument(help="El nombre del campo: «Fecha de nacimiento».")],
+    tipo: Annotated[Tipo, typer.Option("--tipo", help="Qué guarda el campo; no se cambia después.")] = Tipo.texto,
+    opcion: OpcionOption = None,
+    si: YesOption = False,
+    as_json: JsonOption = False,
+):
+    """Agrega un campo a la ficha de las personas. Todas las personas lo tienen, vacío hasta que se llene."""
+    if tipo != Tipo.lista and opcion:
+        raise CliError("--opcion solo va con --tipo lista.")
+    if tipo == Tipo.lista and not opcion:
+        raise CliError("Una lista necesita sus opciones: --opcion «Soltero/a» --opcion «Casado/a».")
+    body = {"entity": ENTITY, "name": nombre, "field_type": TYPE_KEYS[tipo.value], "options": opcion or []}
+    confirm_write(f"Agregar a las personas el campo «{nombre}» ({TYPE_LABELS[body['field_type']]}).", si)
+    created = client_of(ctx).send("POST", "extra-fields", json=body).json()
+    print_result(created, as_json, f"Campo {created['id']} creado: {created['name']}.")
+
+
+@app.command()
+def cambiar(
+    ctx: typer.Context,
+    campo: Annotated[str, typer.Argument(help="El id o el nombre exacto del campo.")],
+    nombre: Annotated[Optional[str], typer.Option("--nombre", help="El nombre nuevo.")] = None,
+    opcion: OpcionOption = None,
+    si: YesOption = False,
+    as_json: JsonOption = False,
+):
+    """Cambia el nombre de un campo o las opciones de una lista. --opcion reemplaza la lista entera."""
+    if not (nombre or opcion):
+        raise CliError("Di qué cambiar: --nombre o --opcion.")
+    client = client_of(ctx)
+    found = _find_field(_fields(client), campo)
+    body = {}
+    if nombre:
+        body["name"] = nombre
+    if opcion:
+        body["options"] = opcion
+    confirm_write(f"Cambiar el campo «{found['name']}».", si)
+    changed = client.send("PATCH", f"extra-fields/{found['id']}", json=body).json()
+    print_result(changed, as_json, f"Campo {changed['id']} cambiado: {changed['name']}.")
+
+
+@app.command()
+def quitar(
+    ctx: typer.Context,
+    campo: Annotated[str, typer.Argument(help="El id o el nombre exacto del campo.")],
+    si: YesOption = False,
+    as_json: JsonOption = False,
+):
+    """Quita un campo de la ficha de todas las personas. Los valores ya guardados quedan en la base de datos."""
+    client = client_of(ctx)
+    found = _find_field(_fields(client), campo)
+    confirm_write(f"Quitar el campo «{found['name']}» de todas las personas.", si)
+    client.send("DELETE", f"extra-fields/{found['id']}")
+    print_result({"id": found["id"], "removed": True}, as_json, f"Campo {found['id']} quitado: {found['name']}.")
+
+
+@app.command()
+def ver(
+    ctx: typer.Context,
+    persona: Annotated[str, typer.Argument(help="El nombre exacto, el código o el id de la persona.")],
+    as_json: JsonOption = False,
+    as_csv: CsvOption = False,
+):
+    """La información extra de una persona: cada campo con su valor."""
+    output = output_format(as_json, as_csv)
+    client = client_of(ctx)
+    person = _person(client, persona)
+    answer = client.get(f"people/employees/{person['id']}/extra-fields")
+    if output == "json":
+        print_record(answer, "json")
+        return
+    rows = [{"name": field["name"], "value": _shown(field)} for field in answer["fields"]]
+    print_rows(VALUE_COLUMNS, rows, output)
+
+
+@app.command()
+def guardar(
+    ctx: typer.Context,
+    persona: Annotated[str, typer.Argument(help="El nombre exacto, el código o el id de la persona.")],
+    valor: Annotated[
+        Optional[list[str]],
+        typer.Option("--valor", help="Campo=valor, por ejemplo «Hijos=2» o «Fecha de nacimiento=17/05/1990»; repítelo para varios."),
+    ] = None,
+    vaciar: Annotated[
+        Optional[list[str]], typer.Option("--vaciar", help="El campo que se deja vacío; repítelo para varios."),
+    ] = None,
+    si: YesOption = False,
+    as_json: JsonOption = False,
+):
+    """Guarda valores en la información extra de una persona. Solo cambian los campos que digas."""
+    if not (valor or vaciar):
+        raise CliError("Di qué guardar: --valor Campo=valor o --vaciar Campo.")
+    client = client_of(ctx)
+    fields = _fields(client)
+    values = {}
+    for pair in valor or []:
+        name, text = _split_pair(pair)
+        field = _find_field(fields, name)
+        values[str(field["id"])] = _value_for(field, text)
+    for name in vaciar or []:
+        values[str(_find_field(fields, name)["id"])] = None
+    person = _person(client, persona)
+    who = person.get("user_name") or persona
+    changes = list(valor or []) + [f"vaciar {name}" for name in vaciar or []]
+    confirm_write(f"Guardar en la información extra de {who}: {'; '.join(changes)}.", si)
+    saved = client.send("PATCH", f"people/employees/{person['id']}/extra-fields", json={"values": values}).json()
+    print_result(saved, as_json, f"Información extra de {who} guardada.")

{operx_cli-0.7.3 → operx_cli-0.7.4}/operx_cli/commands/rutinas.py

@@ -1,7 +1,7 @@
 """`operx rutinas`: Personas › Rutinas, what goes on the day's plan of
 everyone, of some roles or of some people, and whether each person did it
 (backend/people/routine_views.py). A routine with no roles and no people is
-for everyone."""
+for everyone; one with no days goes every day."""
 
 from datetime import date
 from typing import Annotated, Optional
@@ -22,8 +22,12 @@ app = typer.Typer(
 )
 
 ROUTINE_COLUMNS = [
-    ("id", "Id"), ("text", "Rutina"), ("roles", "Roles"), ("employees", "Personas"), ("people_count", "Les llega a"),
+    ("id", "Id"), ("text", "Rutina"), ("roles", "Roles"), ("employees", "Personas"), ("days", "Días"),
+    ("people_count", "Les llega a"),
 ]
+# The API numbers the days 0 Monday ... 6 Sunday.
+WEEKDAY_NAMES = ["lunes", "martes", "miércoles", "jueves", "viernes", "sábado", "domingo"]
+WEEKDAY_ALIASES = {"miercoles": 2, "sabado": 5}
 COMPLIANCE_COLUMNS = [
     ("name", "Persona"), ("position", "Cargo"), ("done", "Hechas"), ("due", "Debidas"), ("days_without_plan", "Días sin plan"),
 ]
@@ -34,6 +38,31 @@ RolOption = Annotated[
 PersonaOption = Annotated[
     Optional[list[str]], typer.Option("--persona", help="El nombre o código de una persona que la recibe; repítelo para varias.")
 ]
+DiaOption = Annotated[
+    Optional[list[str]],
+    typer.Option("--dia", help="Un día de la semana en que va (lunes ... domingo); repítelo para varios. Sin --dia, va todos los días."),
+]
+
+
+def _weekdays(names):
+    """The API's day numbers for the names typed ("lunes", "Sábado"), sorted."""
+    numbers = set()
+    for name in names or []:
+        wanted = " ".join(name.split()).casefold()
+        if wanted in WEEKDAY_NAMES:
+            numbers.add(WEEKDAY_NAMES.index(wanted))
+        elif wanted in WEEKDAY_ALIASES:
+            numbers.add(WEEKDAY_ALIASES[wanted])
+        else:
+            raise CliError(f"«{name}» no es un día de la semana. Usa: {', '.join(WEEKDAY_NAMES)}.")
+    return sorted(numbers)
+
+
+def _days_text(weekdays):
+    """The days a routine goes, for a person: "todos los días" or "lunes, jueves"."""
+    if not weekdays or len(weekdays) == len(WEEKDAY_NAMES):
+        return "todos los días"
+    return ", ".join(WEEKDAY_NAMES[number] for number in sorted(weekdays))
 
 
 def _role_ids(client, names):
@@ -70,15 +99,25 @@ def _who(roles, people):
     return "para " + " y ".join(parts)
 
 
+def _only_days(weekdays):
+    """What a confirmation adds when the routine goes on some days only."""
+    if not weekdays or len(weekdays) == len(WEEKDAY_NAMES):
+        return ""
+    return f" (solo: {_days_text(weekdays)})"
+
+
 @app.command()
 def listar(
     ctx: typer.Context,
     as_json: JsonOption = False,
     as_csv: CsvOption = False,
 ):
-    """Las rutinas de la empresa y a cuántas personas les llega cada una."""
+    """Las rutinas de la empresa, los días en que van y a cuántas personas les llega cada una."""
     output = output_format(as_json, as_csv)
-    print_rows(ROUTINE_COLUMNS, client_of(ctx).get("people/routines"), output)
+    routines = client_of(ctx).get("people/routines")
+    if output != "json":
+        routines = [{**routine, "days": _days_text(routine.get("weekdays"))} for routine in routines]
+    print_rows(ROUTINE_COLUMNS, routines, output)
 
 
 @app.command()
@@ -87,13 +126,17 @@ def crear(
     texto: Annotated[str, typer.Argument(help="Lo que va en el plan: «Llenar la bitácora».")],
     rol: RolOption = None,
     persona: PersonaOption = None,
+    dia: DiaOption = None,
     si: YesOption = False,
     as_json: JsonOption = False,
 ):
-    """Crea una rutina. Sin --rol ni --persona, es para todos."""
+    """Crea una rutina. Sin --rol ni --persona, es para todos; sin --dia, va todos los días."""
     client = client_of(ctx)
+    weekdays = _weekdays(dia)
     body = {"text": texto, "role_ids": _role_ids(client, rol), "employee_ids": _employee_ids(client, persona)}
-    confirm_write(f"Crear la rutina «{texto}» {_who(rol, persona)}.", si)
+    if weekdays:
+        body["weekdays"] = weekdays
+    confirm_write(f"Crear la rutina «{texto}» {_who(rol, persona)}{_only_days(weekdays)}.", si)
     created = client.send("POST", "people/routines", json=body).json()
     print_result(
         created, as_json,
@@ -109,17 +152,26 @@ def cambiar(
     rol: RolOption = None,
     persona: PersonaOption = None,
     para_todos: Annotated[bool, typer.Option("--para-todos", help="La deja para todos, sin roles ni personas.")] = False,
+    dia: DiaOption = None,
+    todos_los_dias: Annotated[bool, typer.Option("--todos-los-dias", help="La deja yendo todos los días.")] = False,
     si: YesOption = False,
     as_json: JsonOption = False,
 ):
-    """Cambia una rutina. --rol y --persona reemplazan la lista entera; lo que no digas queda como estaba."""
+    """Cambia una rutina. --rol, --persona y --dia reemplazan la lista entera; lo que no digas queda como estaba."""
     if para_todos and (rol or persona):
         raise CliError("--para-todos no va con --rol ni --persona.")
-    if not (texto or rol or persona or para_todos):
-        raise CliError("Di qué cambiar: --texto, --rol, --persona o --para-todos.")
+    if todos_los_dias and dia:
+        raise CliError("--todos-los-dias no va con --dia.")
+    if not (texto or rol or persona or para_todos or dia or todos_los_dias):
+        raise CliError("Di qué cambiar: --texto, --rol, --persona, --para-todos, --dia o --todos-los-dias.")
+    weekdays = _weekdays(dia)
     client = client_of(ctx)
     found = _routine(client, rutina)
     body = without_empty({"text": texto})
+    if todos_los_dias:
+        body["weekdays"] = []
+    if weekdays:
+        body["weekdays"] = weekdays
     if para_todos:
         body["role_ids"] = []
         body["employee_ids"] = []

{operx_cli-0.7.3 → operx_cli-0.7.4}/operx_cli.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: operx-cli
-Version: 0.7.3
+Version: 0.7.4
 Summary: La línea de comandos de OperX: tu empresa desde la terminal, con tu token de API.
 Requires-Python: >=3.10
 Description-Content-Type: text/markdown
@@ -174,7 +174,9 @@ operx personas ausencia "Ana Ruiz" --tipo permiso --desde 09/11/2026 --horas 08:
 
 operx rutinas listar
 operx rutinas crear "Contar la bodega" --rol Bodeguero      # sin --rol ni --persona, es para todos
+operx rutinas crear "Revisar el taller" --dia lunes --dia jueves  # sin --dia, va todos los días
 operx rutinas cambiar "Contar la bodega" --persona "Ana Ruiz"
+operx rutinas cambiar "Revisar el taller" --todos-los-dias
 operx rutinas quitar "Contar la bodega"                    # sale de los planes desde ahora
 operx rutinas cumplimiento --mes 10/2026                    # quién hizo sus rutinas (administradores)
 

{operx_cli-0.7.3 → operx_cli-0.7.4}/operx_cli.egg-info/SOURCES.txt

@@ -28,6 +28,7 @@ operx_cli/commands/account.py
 operx_cli/commands/ambiental.py
 operx_cli/commands/articulos.py
 operx_cli/commands/avisos.py
+operx_cli/commands/campos.py
 operx_cli/commands/common.py
 operx_cli/commands/completar.py
 operx_cli/commands/empresas.py
@@ -48,6 +49,7 @@ tests/test_agentes.py
 tests/test_api_campos.py
 tests/test_app.py
 tests/test_articulos.py
+tests/test_campos.py
 tests/test_cards.py
 tests/test_client.py
 tests/test_completar.py

{operx_cli-0.7.3 → operx_cli-0.7.4}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "operx-cli"
-version = "0.7.3"
+version = "0.7.4"
 description = "La línea de comandos de OperX: tu empresa desde la terminal, con tu token de API."
 readme = "README.md"
 requires-python = ">=3.10"

operx_cli-0.7.4/tests/test_campos.py

@@ -0,0 +1,165 @@
+import json
+
+import pytest
+
+from .fake_api import FakeApi, page, run
+
+# extra_fields/serializers.py ExtraFieldSerializer
+HIJOS = {"id": 1, "entity": "employee", "name": "Hijos", "field_type": "number", "options": [], "position": 0}
+NACIMIENTO = {"id": 2, "entity": "employee", "name": "Fecha de nacimiento", "field_type": "date", "options": [], "position": 1}
+ESTADO = {
+    "id": 3, "entity": "employee", "name": "Estado civil", "field_type": "choice",
+    "options": ["Soltero/a", "Casado/a"], "position": 2,
+}
+CONDUCE = {"id": 4, "entity": "employee", "name": "Conduce", "field_type": "yes_no", "options": [], "position": 3}
+FIELDS = [HIJOS, NACIMIENTO, ESTADO, CONDUCE]
+# employees/serializers.py: the staff list
+MARIA = {"id": 7, "user_name": "María Torres", "employee_code": "E-007"}
+# people/extra_info.py extra_payload
+MARIA_VALUES = {"fields": [
+    {"id": 1, "name": "Hijos", "field_type": "number", "options": [], "value": 2},
+    {"id": 2, "name": "Fecha de nacimiento", "field_type": "date", "options": [], "value": "1990-05-17"},
+    {"id": 3, "name": "Estado civil", "field_type": "choice", "options": ["Soltero/a", "Casado/a"], "value": None},
+]}
+
+
+@pytest.fixture(autouse=True)
+def signed_in(monkeypatch):
+    monkeypatch.setenv("OPERX_TOKEN", "opx_test")
+
+
+def saving_fake(answer=None):
+    return FakeApi({
+        ("GET", "extra-fields/"): FIELDS,
+        ("GET", "employees/"): page(MARIA),
+        ("PATCH", "people/employees/7/extra-fields/"): answer or MARIA_VALUES,
+    })
+
+
+def test_listar_shows_each_field_with_its_type_and_options():
+    fake = FakeApi({("GET", "extra-fields/"): FIELDS})
+    result = run(fake, "campos", "listar")
+    assert result.exit_code == 0, result.output
+    assert "Fecha de nacimiento" in result.output and "Número" in result.output
+    assert "Soltero/a, Casado/a" in result.output
+    assert fake.requests[0].url.params["entity"] == "employee"
+
+
+def test_listar_json_is_the_api_s_answer_as_it_is():
+    fake = FakeApi({("GET", "extra-fields/"): FIELDS})
+    result = run(fake, "campos", "listar", "--json")
+    assert json.loads(result.output) == FIELDS
+
+
+def test_crear_sends_the_type_the_api_knows():
+    fake = FakeApi({("POST", "extra-fields/"): (201, HIJOS)})
+    result = run(fake, "campos", "crear", "Hijos", "--tipo", "numero", "--si")
+    assert result.exit_code == 0, result.output
+    assert fake.sent("POST", "extra-fields/") == {"entity": "employee", "name": "Hijos", "field_type": "number", "options": []}
+
+
+def test_crear_a_list_sends_its_options_in_order():
+    fake = FakeApi({("POST", "extra-fields/"): (201, ESTADO)})
+    run(fake, "campos", "crear", "Estado civil", "--tipo", "lista", "--opcion", "Soltero/a", "--opcion", "Casado/a", "--si")
+    assert fake.sent("POST", "extra-fields/")["options"] == ["Soltero/a", "Casado/a"]
+
+
+def test_crear_refuses_a_list_without_options_and_options_without_a_list():
+    fake = FakeApi({})
+    without = run(fake, "campos", "crear", "Estado civil", "--tipo", "lista", "--si")
+    stray = run(fake, "campos", "crear", "Hijos", "--tipo", "numero", "--opcion", "1", "--si")
+    assert without.exit_code == 1 and "necesita sus opciones" in without.output
+    assert stray.exit_code == 1 and "solo va con --tipo lista" in stray.output
+    assert fake.requests == []
+
+
+def test_crear_without_si_outside_a_terminal_changes_nothing():
+    fake = FakeApi({("POST", "extra-fields/"): (201, HIJOS)})
+    result = run(fake, "campos", "crear", "Hijos")
+    assert result.exit_code == 1
+    assert not fake.called("POST", "extra-fields/")
+
+
+def test_cambiar_renames_a_field_found_by_its_name():
+    fake = FakeApi({("GET", "extra-fields/"): FIELDS, ("PATCH", "extra-fields/1/"): HIJOS})
+    result = run(fake, "campos", "cambiar", "hijos", "--nombre", "Número de hijos", "--si")
+    assert result.exit_code == 0, result.output
+    assert fake.sent("PATCH", "extra-fields/1/") == {"name": "Número de hijos"}
+
+
+def test_cambiar_with_nothing_to_change_says_so():
+    fake = FakeApi({})
+    result = run(fake, "campos", "cambiar", "Hijos", "--si")
+    assert result.exit_code == 1 and "Di qué cambiar" in result.output
+
+
+def test_quitar_hides_the_field():
+    fake = FakeApi({("GET", "extra-fields/"): FIELDS, ("DELETE", "extra-fields/1/"): (204, None)})
+    result = run(fake, "campos", "quitar", "Hijos", "--si")
+    assert result.exit_code == 0, result.output
+    assert fake.called("DELETE", "extra-fields/1/")
+
+
+def test_a_name_that_is_no_field_lists_nothing_to_guess_from():
+    fake = FakeApi({("GET", "extra-fields/"): FIELDS})
+    result = run(fake, "campos", "quitar", "Mascotas", "--si")
+    assert result.exit_code == 1 and "Mascotas" in result.output
+    assert not fake.called("DELETE", "extra-fields/1/")
+
+
+def test_ver_shows_each_field_with_its_value_as_a_person_reads_it():
+    fake = FakeApi({("GET", "employees/"): page(MARIA), ("GET", "people/employees/7/extra-fields/"): MARIA_VALUES})
+    result = run(fake, "campos", "ver", "María Torres")
+    assert result.exit_code == 0, result.output
+    assert "17/05/1990" in result.output and "Sin registrar" in result.output
+
+
+def test_ver_json_is_the_api_s_answer_as_it_is():
+    fake = FakeApi({("GET", "employees/"): page(MARIA), ("GET", "people/employees/7/extra-fields/"): MARIA_VALUES})
+    result = run(fake, "campos", "ver", "E-007", "--json")
+    assert json.loads(result.output) == MARIA_VALUES
+
+
+def test_guardar_turns_what_is_typed_into_what_each_type_takes():
+    fake = saving_fake()
+    result = run(
+        fake, "campos", "guardar", "María Torres",
+        "--valor", "Hijos=2,0", "--valor", "Fecha de nacimiento=17/05/1990", "--valor", "estado civil=casado/a",
+        "--valor", "Conduce=sí", "--si",
+    )
+    assert result.exit_code == 0, result.output
+    assert fake.sent("PATCH", "people/employees/7/extra-fields/") == {
+        "values": {"1": 2, "2": "1990-05-17", "3": "Casado/a", "4": True},
+    }
+
+
+def test_guardar_a_decimal_number_stays_a_number():
+    fake = saving_fake()
+    run(fake, "campos", "guardar", "María Torres", "--valor", "Hijos=2,5", "--si")
+    assert fake.sent("PATCH", "people/employees/7/extra-fields/") == {"values": {"1": 2.5}}
+
+
+def test_guardar_vaciar_sends_null_for_that_field():
+    fake = saving_fake()
+    run(fake, "campos", "guardar", "María Torres", "--vaciar", "Hijos", "--si")
+    assert fake.sent("PATCH", "people/employees/7/extra-fields/") == {"values": {"1": None}}
+
+
+@pytest.mark.parametrize("pair, complaint", [
+    ("Hijos=dos", "no es un número"),
+    ("Estado civil=Viudo/a", "no es una opción de «Estado civil». Hay: Soltero/a, Casado/a"),
+    ("Conduce=quizás", "no es sí ni no"),
+    ("Mascotas=2", "Mascotas"),
+    ("Hijos", "Campo=valor"),
+])
+def test_guardar_refuses_a_value_that_does_not_fit_before_calling_the_api(pair, complaint):
+    fake = saving_fake()
+    result = run(fake, "campos", "guardar", "María Torres", "--valor", pair, "--si")
+    assert result.exit_code == 1 and complaint in result.output
+    assert not fake.called("PATCH", "people/employees/7/extra-fields/")
+
+
+def test_guardar_with_nothing_to_save_says_so():
+    fake = FakeApi({})
+    result = run(fake, "campos", "guardar", "María Torres", "--si")
+    assert result.exit_code == 1 and "Di qué guardar" in result.output

{operx_cli-0.7.3 → operx_cli-0.7.4}/tests/test_rutinas.py

@@ -5,10 +5,10 @@ import pytest
 from .fake_api import FakeApi, page, run
 
 # people/routine_views.py routines_payload
-BITACORA = {"id": 3, "text": "Llenar la bitácora", "roles": [], "employees": [], "people_count": 12}
+BITACORA = {"id": 3, "text": "Llenar la bitácora", "roles": [], "employees": [], "weekdays": [], "people_count": 12}
 HERRAMIENTAS = {
     "id": 4, "text": "Recoger herramientas", "roles": [{"id": 8, "name": "Bodeguero"}],
-    "employees": [{"id": 7, "name": "María Torres"}], "people_count": 2,
+    "employees": [{"id": 7, "name": "María Torres"}], "weekdays": [0, 3], "people_count": 2,
 }
 # people/routine_views.py RoutineRolesView
 ROLES = [{"id": 8, "name": "Bodeguero"}, {"id": 9, "name": "Técnico"}]
@@ -38,6 +38,55 @@ def test_listar_shows_who_gets_each_routine():
     assert "Bodeguero" in result.output and "María Torres" in result.output
 
 
+def test_listar_says_on_which_days_each_routine_goes():
+    fake = FakeApi({("GET", "people/routines/"): [BITACORA, HERRAMIENTAS]})
+    result = run(fake, "rutinas", "listar")
+    assert result.exit_code == 0, result.output
+    assert "todos los días" in result.output and "lunes, jueves" in result.output
+
+
+def test_listar_json_is_the_api_s_answer_as_it_is():
+    fake = FakeApi({("GET", "people/routines/"): [BITACORA, HERRAMIENTAS]})
+    result = run(fake, "rutinas", "listar", "--json")
+    assert json.loads(result.output) == [BITACORA, HERRAMIENTAS]
+
+
+def test_crear_with_days_sends_them_as_numbers_whatever_the_order_or_accents():
+    fake = FakeApi({("POST", "people/routines/"): (201, HERRAMIENTAS)})
+    result = run(fake, "rutinas", "crear", "Recoger herramientas", "--dia", "Jueves", "--dia", "lunes", "--dia", "miercoles", "--si")
+    assert result.exit_code == 0, result.output
+    assert fake.sent("POST", "people/routines/") == {
+        "text": "Recoger herramientas", "role_ids": [], "employee_ids": [], "weekdays": [0, 2, 3],
+    }
+
+
+def test_crear_with_a_day_that_is_not_one_changes_nothing():
+    fake = FakeApi({})
+    result = run(fake, "rutinas", "crear", "Recoger herramientas", "--dia", "lunez", "--si")
+    assert result.exit_code == 1
+    assert "lunez" in result.output and "lunes, martes" in result.output
+    assert not fake.called("POST", "people/routines/")
+
+
+def test_cambiar_replaces_the_days_or_goes_back_to_every_day():
+    fake = FakeApi({
+        ("GET", "people/routines/"): [HERRAMIENTAS],
+        ("PATCH", "people/routines/4/"): HERRAMIENTAS,
+    })
+    run(fake, "rutinas", "cambiar", "4", "--dia", "viernes", "--si")
+    assert fake.sent("PATCH", "people/routines/4/") == {"weekdays": [4]}
+    run(fake, "rutinas", "cambiar", "4", "--todos-los-dias", "--si")
+    assert fake.sent("PATCH", "people/routines/4/") == {"weekdays": []}
+
+
+def test_cambiar_refuses_days_and_every_day_together():
+    fake = FakeApi({})
+    result = run(fake, "rutinas", "cambiar", "4", "--dia", "viernes", "--todos-los-dias", "--si")
+    assert result.exit_code == 1
+    assert "--todos-los-dias no va con --dia" in result.output
+    assert fake.requests == []
+
+
 def test_crear_names_the_roles_and_people_and_sends_their_ids():
     created = {**HERRAMIENTAS, "id": 5, "text": "Contar la bodega"}
     fake = FakeApi({