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

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

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: operx-cli
-Version: 0.7.1
+Version: 0.7.3
 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
@@ -172,6 +172,12 @@ operx personas importar empleados.xlsx --confirmar   # guarda si ninguna fila ti
 operx personas ausencia "Ana Ruiz" --tipo vacaciones --desde 02/11/2026 --hasta 06/11/2026
 operx personas ausencia "Ana Ruiz" --tipo permiso --desde 09/11/2026 --horas 08:00-10:00
 
+operx rutinas listar
+operx rutinas crear "Contar la bodega" --rol Bodeguero      # sin --rol ni --persona, es para todos
+operx rutinas cambiar "Contar la bodega" --persona "Ana Ruiz"
+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 sst acciones --estado abiertas
 operx sst nueva-accion --titulo "Señalizar la salida" --para 15/11/2026 --responsable "Ana Ruiz"
 operx sst accion-hecha 7 --evidencia "Se instalaron dos señales"

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

@@ -158,6 +158,12 @@ operx personas importar empleados.xlsx --confirmar   # guarda si ninguna fila ti
 operx personas ausencia "Ana Ruiz" --tipo vacaciones --desde 02/11/2026 --hasta 06/11/2026
 operx personas ausencia "Ana Ruiz" --tipo permiso --desde 09/11/2026 --horas 08:00-10:00
 
+operx rutinas listar
+operx rutinas crear "Contar la bodega" --rol Bodeguero      # sin --rol ni --persona, es para todos
+operx rutinas cambiar "Contar la bodega" --persona "Ana Ruiz"
+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 sst acciones --estado abiertas
 operx sst nueva-accion --titulo "Señalizar la salida" --para 15/11/2026 --responsable "Ana Ruiz"
 operx sst accion-hecha 7 --evidencia "Se instalaron dos señales"

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

@@ -68,12 +68,18 @@ llamada queda en el historial del token, marcada como hecha desde la CLI.
     `traslado` y `revertir`.
   - Equipos: `equipos parar`, `reanudar`, `hallazgo` y `hallazgo-a-orden`;
     `lecturas registrar` y `anular`.
-  - Personas: `personas importar --confirmar` y `personas ausencia`.
+  - Personas: `personas importar --confirmar` y `personas ausencia`;
+    `rutinas crear`, `cambiar` y `quitar`.
   - 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.
 - Un error se corrige con su contrario, nunca borrando: `inventario revertir`
-  deshace un movimiento, `lecturas anular` una lectura.
+  deshace un movimiento, `lecturas anular` una lectura. La excepción es
+  `rutinas quitar`, que saca una rutina de los planes desde ahora; los días
+  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.
 - 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.1 → operx_cli-0.7.3}/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.1"
+__version__ = "0.7.3"

{operx_cli-0.7.1 → operx_cli-0.7.3}/operx_cli/api_fields.py

@@ -2,15 +2,21 @@
 (auth/api-tokens/schema/, drf-spectacular): the values in its address, its
 filters, the fields of its body and what it answers. `operx api campos`
 prints it, so an AI agent can call any route with `operx api` without
-guessing; no network here, so the MCP server can use it as it is.
+guessing; no network here, so the MCP server can use it as it is: its
+image copies this one file (mcp/Dockerfile), so keep it free of imports
+beyond the standard library.
 
 The schema's shapes this follows: a field's type behind a $ref; an allOf
 of one $ref (a nested record); a oneOf of an enum with BlankEnum and
 NullEnum (a choice that may be "" or null); each enum's labels in its
 description, one "* `value` - Label" a line.
+
+It also lists the routes and finds them by words (`operx api rutas`, the
+MCP's operx_rutas).
 """
 
 import re
+import unicodedata
 
 METHODS = ("get", "post", "put", "patch", "delete")
 # A field inside a list inside a list… is shown this many levels deep.
@@ -230,3 +236,40 @@ def _answer(components, operation):
             record = _resolve(components, record.get("items"))
         return {"estado": int(status), "pagina": paged, "campos": list(record.get("properties", {}))}
     return {"estado": None, "pagina": False, "campos": []}
+
+
+def routes_of(schema):
+    """Every route of the schema, one row per method: its address, its
+    product (the docs' section, in Spanish) and its description."""
+    routes = []
+    for path, operations in schema.get("paths", {}).items():
+        for method, operation in operations.items():
+            if method not in METHODS:
+                continue
+            # The first paragraph: the docs add a note on the token a write needs.
+            description = (operation.get("description") or "").split("\n\n")[0].strip()
+            if description.startswith("Necesita un token"):
+                description = ""
+            routes.append({
+                "metodo": method.upper(), "ruta": path.removeprefix("/api/v1/"),
+                # The docs' section: the product, in Spanish ("SG-SST", "Inventario").
+                "producto": ", ".join(operation.get("tags", [])), "descripcion": description,
+            })
+    return routes
+
+
+def plain(text):
+    """'Órdenes' → 'ordenes': no case, no accents."""
+    return "".join(char for char in unicodedata.normalize("NFKD", text.casefold()) if not unicodedata.combining(char))
+
+
+def mentions(where, text):
+    """Whether every word of `text` is in `where`, a plural also as its
+    singular: an agent asks for "repuestos" where the docs say "repuesto",
+    and "ordenes" for "órdenes"."""
+    where = plain(where)
+    for word in plain(text).split():
+        singular = word[:-2] if word.endswith("es") and len(word) > 5 else word.rstrip("s") if len(word) > 4 else word
+        if word not in where and singular not in where:
+            return False
+    return True

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

@@ -15,7 +15,7 @@ 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, search, sst, update,
+    registros, reportes, rutinas, search, sst, update,
 )
 from .commands.common import CliError, Session
 from .lookup import NoSingleMatch
@@ -155,6 +155,7 @@ app.add_typer(inventario.app, name="inventario")
 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(reportes.app, name="reportes")
 app.add_typer(sst.app, name="sst")
 app.add_typer(ambiental.app, name="ambiental")

{operx_cli-0.7.1 → operx_cli-0.7.3}/operx_cli/commands/raw.py

@@ -2,7 +2,6 @@
 lists them), for what has no command of its own yet."""
 
 import json
-import unicodedata
 from contextlib import ExitStack
 from pathlib import Path
 from typing import Annotated, Optional
@@ -10,7 +9,7 @@ from typing import Annotated, Optional
 import typer
 from rich.text import Text
 
-from ..api_fields import describe, find_route, methods_of
+from ..api_fields import describe, find_route, mentions, methods_of, routes_of
 from ..client import ApiError, api_path
 from ..output import json_text, note, print_rows, screen_console
 from .common import AllOption, CliError, YesOption, client_of, confirm_write
@@ -60,9 +59,9 @@ def _print_routes(client, text, as_json):
     """Every call the token may make, from the API's own docs
     (users/api_docs.py): the method, the address as `operx api` takes it,
     and what the docs say. `text` keeps the ones that mention it."""
-    routes = _routes(client.get("auth/api-tokens/schema"))
+    routes = routes_of(client.get("auth/api-tokens/schema"))
     if text:
-        routes = [route for route in routes if _mentions(f"{route['ruta']} {route['producto']} {route['descripcion']}", text)]
+        routes = [route for route in routes if mentions(f"{route['ruta']} {route['producto']} {route['descripcion']}", text)]
     print_rows(ROUTE_COLUMNS, routes, "json" if as_json else "tabla")
     if not as_json and routes:
         note("Lo que lleva cada una: operx api campos RUTA")
@@ -169,41 +168,6 @@ def _print_fields(client, ruta, method, as_json):
             console.print(Text(f"Responde ({answer['estado']}) {what}: {', '.join(answer['campos'])}".replace(" :", ":"), style="dim"))
 
 
-def _plain(text):
-    """'Órdenes' → 'ordenes': no case, no accents."""
-    return "".join(char for char in unicodedata.normalize("NFKD", text.casefold()) if not unicodedata.combining(char))
-
-
-def _mentions(where, text):
-    """Whether every word of `text` is in `where`, a plural also as its
-    singular: an agent asks for "repuestos" where the docs say "repuesto",
-    and "ordenes" for "órdenes"."""
-    where = _plain(where)
-    for word in _plain(text).split():
-        singular = word[:-2] if word.endswith("es") and len(word) > 5 else word.rstrip("s") if len(word) > 4 else word
-        if word not in where and singular not in where:
-            return False
-    return True
-
-
-def _routes(schema):
-    routes = []
-    for path, operations in schema.get("paths", {}).items():
-        for method, operation in operations.items():
-            if method.upper() not in METHODS:
-                continue
-            # The first paragraph: the docs add a note on the token a write needs.
-            description = (operation.get("description") or "").split("\n\n")[0].strip()
-            if description.startswith("Necesita un token"):
-                description = ""
-            routes.append({
-                "metodo": method.upper(), "ruta": path.removeprefix("/api/v1/"),
-                # The docs' section: the product, in Spanish ("SG-SST", "Inventario").
-                "producto": ", ".join(operation.get("tags", [])), "descripcion": description,
-            })
-    return routes
-
-
 def _show(response, salida):
     if salida:
         salida.write_bytes(response.content)

operx_cli-0.7.3/operx_cli/commands/rutinas.py

@@ -0,0 +1,169 @@
+"""`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."""
+
+from datetime import date
+from typing import Annotated, Optional
+
+import typer
+
+from ..lookup import NoSingleMatch, find_in, find_one
+from ..output import note, print_record, print_rows
+from ..values import shown_moment
+from .common import (
+    CliError, CsvOption, JsonOption, YesOption, client_of, confirm_write, month, output_format, print_result,
+    without_empty,
+)
+
+app = typer.Typer(
+    help="Rutinas (Personas): lo que va en el plan del día de todos, de algunos roles o de algunas personas.",
+    no_args_is_help=True,
+)
+
+ROUTINE_COLUMNS = [
+    ("id", "Id"), ("text", "Rutina"), ("roles", "Roles"), ("employees", "Personas"), ("people_count", "Les llega a"),
+]
+COMPLIANCE_COLUMNS = [
+    ("name", "Persona"), ("position", "Cargo"), ("done", "Hechas"), ("due", "Debidas"), ("days_without_plan", "Días sin plan"),
+]
+
+RolOption = Annotated[
+    Optional[list[str]], typer.Option("--rol", help="El nombre de un rol que la recibe; repítelo para varios.")
+]
+PersonaOption = Annotated[
+    Optional[list[str]], typer.Option("--persona", help="El nombre o código de una persona que la recibe; repítelo para varias.")
+]
+
+
+def _role_ids(client, names):
+    if not names:
+        return []
+    roles = client.get("people/routines/roles")
+    ids = []
+    for name in names:
+        try:
+            ids.append(find_in(roles, name, ["name"], "rol")["id"])
+        except NoSingleMatch as error:
+            # A short list: saying which there are saves a trip to the settings.
+            raise CliError(f"{error} Hay: {', '.join(role['name'] for role in roles)}.") from None
+    return ids
+
+
+def _employee_ids(client, names):
+    return [find_one(client, "employees", name, ["user_name", "employee_code"], "persona")["id"] for name in names or []]
+
+
+def _routine(client, text):
+    """A routine by its id or its exact text."""
+    return find_in(client.get("people/routines"), text, ["id", "text"], "rutina")
+
+
+def _who(roles, people):
+    if not roles and not people:
+        return "para todos"
+    parts = []
+    if roles:
+        parts.append(f"los roles {', '.join(roles)}")
+    if people:
+        parts.append(f"las personas {', '.join(people)}")
+    return "para " + " y ".join(parts)
+
+
+@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."""
+    output = output_format(as_json, as_csv)
+    print_rows(ROUTINE_COLUMNS, client_of(ctx).get("people/routines"), output)
+
+
+@app.command()
+def crear(
+    ctx: typer.Context,
+    texto: Annotated[str, typer.Argument(help="Lo que va en el plan: «Llenar la bitácora».")],
+    rol: RolOption = None,
+    persona: PersonaOption = None,
+    si: YesOption = False,
+    as_json: JsonOption = False,
+):
+    """Crea una rutina. Sin --rol ni --persona, es para todos."""
+    client = client_of(ctx)
+    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)
+    created = client.send("POST", "people/routines", json=body).json()
+    print_result(
+        created, as_json,
+        f"Rutina {created['id']} creada: {created['text']}. Les llega a {created['people_count']} personas.",
+    )
+
+
+@app.command()
+def cambiar(
+    ctx: typer.Context,
+    rutina: Annotated[str, typer.Argument(help="El id o el texto exacto de la rutina.")],
+    texto: Annotated[Optional[str], typer.Option("--texto", help="El texto nuevo.")] = None,
+    rol: RolOption = None,
+    persona: PersonaOption = None,
+    para_todos: Annotated[bool, typer.Option("--para-todos", help="La deja para todos, sin roles ni personas.")] = 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."""
+    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.")
+    client = client_of(ctx)
+    found = _routine(client, rutina)
+    body = without_empty({"text": texto})
+    if para_todos:
+        body["role_ids"] = []
+        body["employee_ids"] = []
+    if rol:
+        body["role_ids"] = _role_ids(client, rol)
+    if persona:
+        body["employee_ids"] = _employee_ids(client, persona)
+    confirm_write(f"Cambiar la rutina «{found['text']}».", si)
+    changed = client.send("PATCH", f"people/routines/{found['id']}", json=body).json()
+    print_result(
+        changed, as_json,
+        f"Rutina {changed['id']} cambiada: {changed['text']}. Les llega a {changed['people_count']} personas.",
+    )
+
+
+@app.command()
+def quitar(
+    ctx: typer.Context,
+    rutina: Annotated[str, typer.Argument(help="El id o el texto exacto de la rutina.")],
+    si: YesOption = False,
+    as_json: JsonOption = False,
+):
+    """Quita una rutina de los planes desde ahora; los días que ya la tenían la siguen contando."""
+    client = client_of(ctx)
+    found = _routine(client, rutina)
+    confirm_write(f"Quitar la rutina «{found['text']}» de los planes.", si)
+    client.send("DELETE", f"people/routines/{found['id']}")
+    print_result({"id": found["id"], "removed": True}, as_json, f"Rutina {found['id']} quitada: {found['text']}.")
+
+
+@app.command()
+def cumplimiento(
+    ctx: typer.Context,
+    mes: Annotated[Optional[str], typer.Option("--mes", help="El mes: 11/2026; sin él, el actual.")] = None,
+    as_json: JsonOption = False,
+    as_csv: CsvOption = False,
+):
+    """Quién hizo sus rutinas en un mes: por persona, las que traía su plan y las que marcó. Solo administradores."""
+    output = output_format(as_json, as_csv)
+    month_text = month(mes) if mes else date.today().strftime("%Y-%m")
+    answer = client_of(ctx).get("people/routines/compliance", {"month": month_text})
+    if output == "json":
+        print_record(answer, "json")
+        return
+    if answer.get("through"):
+        note(f"Cuenta hasta el {shown_moment(answer['through'])}: hoy no cuenta.")
+    print_rows(COMPLIANCE_COLUMNS, answer.get("people", []), output)

{operx_cli-0.7.1 → operx_cli-0.7.3}/operx_cli/lookup.py

@@ -3,6 +3,9 @@
 Only an exact match counts, ignoring case and extra spaces: a search's first
 hit is never taken, so "Bodega" can't move stock into "Bodega central norte".
 A number nobody is called is taken as the record's id.
+
+The MCP server uses find_in as it is: its image copies this one file
+(mcp/Dockerfile), so keep it free of imports.
 """
 
 

{operx_cli-0.7.1 → operx_cli-0.7.3}/operx_cli/shell.py

@@ -10,16 +10,19 @@ terminal (Git Bash, a pipe)."""
 import os
 import re
 import shlex
+import traceback
 
 import typer
 from rich.console import Console
 from rich.text import Text
 
-from . import settings
+from . import __version__, settings
 from .output import print_aligned
 from .record_completion import RecordNames, completion_text
 
 PROMPT = "operx › "
+# Where the CLI's own files are: a crash names its line closest to the error.
+PACKAGE_FOLDER = os.path.normcase(os.path.dirname(os.path.abspath(__file__)))
 STOP_WORDS = ("salir", "exit", "quit")
 HELP_WORDS = ("/ayuda", "ayuda", "/help", "help", "?")
 CLEAR_WORDS = ("limpiar", "/limpiar", "clear", "cls")
@@ -54,6 +57,7 @@ COMMAND_PRODUCTS = {
     "lecturas": ("equipment",),
     "personas importar": ("people",),
     "personas ausencia": ("people",),
+    "rutinas": ("people",),
     "sst": ("sst",),
     "ambiental": ("environment",),
 }
@@ -169,6 +173,31 @@ def print_help(root, console, products=None):
     console.print(Text("Escribe un comando con --help para ver sus opciones.", style="dim"))
 
 
+def where_it_failed(error):
+    """The CLI's own line closest to the error, as "commands/search.py:22".
+    run_command, which caught it, is always one of them."""
+    frames = traceback.extract_tb(error.__traceback__)
+    ours = [frame for frame in frames if os.path.normcase(frame.filename).startswith(PACKAGE_FOLDER)]
+    if not ours:
+        return "?"
+    relative = os.path.relpath(ours[-1].filename, PACKAGE_FOLDER)
+    return f"{relative.replace(os.sep, '/')}:{ours[-1].lineno}"
+
+
+def crash_message(error):
+    """What a crash says inside the session: a line a person can send to
+    support (the error, where in the CLI, and the version those line numbers
+    belong to), never what was typed, which can hold anything."""
+    what = type(error).__name__
+    lines = str(error).strip().splitlines()
+    if lines:
+        what = f"{what}: {lines[0]}"
+    return (
+        f"Error: algo falló ({what}; en {where_it_failed(error)}, operx {__version__}).\n"
+        "Para ver todo el detalle, corre el mismo comando con operx delante, fuera de esta sesión."
+    )
+
+
 def run_command(session, root, argv):
     """One command, as `operx` would run it, but an error never ends the session."""
     try:
@@ -181,7 +210,7 @@ def run_command(session, root, argv):
         if hasattr(error, "format_message"):
             typer.echo(f"Error: {spanish_usage_error(error)}", err=True)
         else:
-            typer.echo(f"Error: algo falló ({type(error).__name__}).", err=True)
+            typer.echo(crash_message(error), err=True)
 
 
 def handle_line(session, root, line, console, products=None):

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

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: operx-cli
-Version: 0.7.1
+Version: 0.7.3
 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
@@ -172,6 +172,12 @@ operx personas importar empleados.xlsx --confirmar   # guarda si ninguna fila ti
 operx personas ausencia "Ana Ruiz" --tipo vacaciones --desde 02/11/2026 --hasta 06/11/2026
 operx personas ausencia "Ana Ruiz" --tipo permiso --desde 09/11/2026 --horas 08:00-10:00
 
+operx rutinas listar
+operx rutinas crear "Contar la bodega" --rol Bodeguero      # sin --rol ni --persona, es para todos
+operx rutinas cambiar "Contar la bodega" --persona "Ana Ruiz"
+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 sst acciones --estado abiertas
 operx sst nueva-accion --titulo "Señalizar la salida" --para 15/11/2026 --responsable "Ana Ruiz"
 operx sst accion-hecha 7 --evidencia "Se instalaron dos señales"

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

@@ -40,6 +40,7 @@ operx_cli/commands/preventivos.py
 operx_cli/commands/raw.py
 operx_cli/commands/registros.py
 operx_cli/commands/reportes.py
+operx_cli/commands/rutinas.py
 operx_cli/commands/search.py
 operx_cli/commands/sst.py
 operx_cli/commands/update.py
@@ -63,6 +64,7 @@ tests/test_personas.py
 tests/test_preventivos.py
 tests/test_record_completion.py
 tests/test_registros.py
+tests/test_rutinas.py
 tests/test_self_update.py
 tests/test_settings.py
 tests/test_shell.py

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

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "operx-cli"
-version = "0.7.1"
+version = "0.7.3"
 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.1 → operx_cli-0.7.3}/tests/test_completar.py

@@ -3,6 +3,7 @@ writes a small script to operx's folder and, after asking, one line in the
 PowerShell profile that loads it; `--quitar` takes both away."""
 
 import os
+import sys
 from pathlib import Path
 
 import pytest
@@ -31,6 +32,7 @@ def test_the_script_completes_operx_and_leaves_tab_alone_elsewhere():
     script.encode("ascii")
 
 
+@pytest.mark.skipif(sys.platform != "win32", reason="PowerShell's profile is a Windows path")
 def test_the_profile_line_names_the_folder_by_appdata(tmp_path):
     """No user name in the line: a profile saved in another encoding can't
     garble it."""

operx_cli-0.7.3/tests/test_rutinas.py

@@ -0,0 +1,128 @@
+import json
+
+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}
+HERRAMIENTAS = {
+    "id": 4, "text": "Recoger herramientas", "roles": [{"id": 8, "name": "Bodeguero"}],
+    "employees": [{"id": 7, "name": "María Torres"}], "people_count": 2,
+}
+# people/routine_views.py RoutineRolesView
+ROLES = [{"id": 8, "name": "Bodeguero"}, {"id": 9, "name": "Técnico"}]
+# employees/serializers.py: the staff list
+MARIA = {"id": 7, "user_name": "María Torres", "employee_code": "E-007"}
+# people/routine_views.py RoutineComplianceView
+COMPLIANCE = {
+    "month": "2026-10", "previous_month": "2026-09", "next_month": None, "is_current_month": True,
+    "through": "2026-10-25", "days": ["2026-10-24", "2026-10-25"],
+    "people": [
+        {"employee_id": 7, "name": "María Torres", "position": "Técnico", "done": 3, "due": 4, "days_without_plan": 1,
+         "days": [{"date": "2026-10-24", "plan": True, "done": 3, "due": 4}]},
+    ],
+}
+
+
+@pytest.fixture(autouse=True)
+def signed_in(monkeypatch):
+    monkeypatch.setenv("OPERX_TOKEN", "opx_test")
+
+
+def test_listar_shows_who_gets_each_routine():
+    fake = FakeApi({("GET", "people/routines/"): [BITACORA, HERRAMIENTAS]})
+    result = run(fake, "rutinas", "listar")
+    assert result.exit_code == 0, result.output
+    assert "Llenar la bitácora" in result.output and "Recoger herramientas" in result.output
+    assert "Bodeguero" in result.output and "María Torres" in result.output
+
+
+def test_crear_names_the_roles_and_people_and_sends_their_ids():
+    created = {**HERRAMIENTAS, "id": 5, "text": "Contar la bodega"}
+    fake = FakeApi({
+        ("GET", "people/routines/roles/"): ROLES,
+        ("GET", "employees/"): page(MARIA),
+        ("POST", "people/routines/"): (201, created),
+    })
+    result = run(fake, "rutinas", "crear", "Contar la bodega", "--rol", "bodeguero", "--persona", "María Torres", "--si")
+    assert result.exit_code == 0, result.output
+    assert fake.sent("POST", "people/routines/") == {"text": "Contar la bodega", "role_ids": [8], "employee_ids": [7]}
+    assert "Rutina 5 creada" in result.output
+
+
+def test_crear_without_roles_or_people_is_for_everyone():
+    fake = FakeApi({("POST", "people/routines/"): (201, BITACORA)})
+    result = run(fake, "rutinas", "crear", "Llenar la bitácora", "--si")
+    assert result.exit_code == 0, result.output
+    assert fake.sent("POST", "people/routines/") == {"text": "Llenar la bitácora", "role_ids": [], "employee_ids": []}
+
+
+def test_crear_with_a_role_nobody_has_says_which_there_are():
+    fake = FakeApi({("GET", "people/routines/roles/"): ROLES})
+    result = run(fake, "rutinas", "crear", "Contar la bodega", "--rol", "Gerente", "--si")
+    assert result.exit_code == 1
+    assert "Gerente" in result.output and "Bodeguero, Técnico" in result.output
+    assert not fake.called("POST", "people/routines/")
+
+
+def test_crear_json_gives_the_record_with_its_id():
+    fake = FakeApi({("POST", "people/routines/"): (201, BITACORA)})
+    result = run(fake, "rutinas", "crear", "Llenar la bitácora", "--si", "--json")
+    assert json.loads(result.output)["id"] == 3
+
+
+def test_cambiar_finds_the_routine_by_its_text_and_replaces_the_roles():
+    changed = {**HERRAMIENTAS, "roles": [{"id": 9, "name": "Técnico"}]}
+    fake = FakeApi({
+        ("GET", "people/routines/"): [BITACORA, HERRAMIENTAS],
+        ("GET", "people/routines/roles/"): ROLES,
+        ("PATCH", "people/routines/4/"): changed,
+    })
+    result = run(fake, "rutinas", "cambiar", "recoger herramientas", "--rol", "Técnico", "--si")
+    assert result.exit_code == 0, result.output
+    assert fake.sent("PATCH", "people/routines/4/") == {"role_ids": [9]}
+
+
+def test_cambiar_para_todos_empties_both_lists():
+    fake = FakeApi({("GET", "people/routines/"): [HERRAMIENTAS], ("PATCH", "people/routines/4/"): HERRAMIENTAS})
+    result = run(fake, "rutinas", "cambiar", "4", "--para-todos", "--si")
+    assert result.exit_code == 0, result.output
+    assert fake.sent("PATCH", "people/routines/4/") == {"role_ids": [], "employee_ids": []}
+
+
+def test_cambiar_needs_something_to_change():
+    fake = FakeApi({})
+    result = run(fake, "rutinas", "cambiar", "4", "--si")
+    assert result.exit_code == 1
+    assert "Di qué cambiar" in result.output
+    assert fake.requests == []
+
+
+def test_quitar_takes_the_routine_off_the_plans():
+    fake = FakeApi({("GET", "people/routines/"): [BITACORA], ("DELETE", "people/routines/3/"): (204, None)})
+    result = run(fake, "rutinas", "quitar", "Llenar la bitácora", "--si")
+    assert result.exit_code == 0, result.output
+    assert fake.called("DELETE", "people/routines/3/")
+    assert "quitada" in result.output
+
+
+def test_quitar_with_an_unknown_routine_changes_nothing():
+    fake = FakeApi({("GET", "people/routines/"): [BITACORA]})
+    result = run(fake, "rutinas", "quitar", "Otra", "--si")
+    assert result.exit_code == 1
+    assert not fake.called("DELETE", "people/routines/3/")
+
+
+def test_cumplimiento_asks_for_the_month_and_shows_each_person():
+    fake = FakeApi({("GET", "people/routines/compliance/"): COMPLIANCE})
+    result = run(fake, "rutinas", "cumplimiento", "--mes", "10/2026")
+    assert result.exit_code == 0, result.output
+    assert fake.requests[0].url.params["month"] == "2026-10"
+    assert "María Torres" in result.output and "Técnico" in result.output
+
+
+def test_cumplimiento_json_is_the_whole_answer():
+    fake = FakeApi({("GET", "people/routines/compliance/"): COMPLIANCE})
+    result = run(fake, "rutinas", "cumplimiento", "--mes", "2026-10", "--json")
+    assert json.loads(result.output) == COMPLIANCE

{operx_cli-0.7.1 → operx_cli-0.7.3}/tests/test_shell.py

@@ -3,7 +3,7 @@ import pytest
 import typer.main
 from typer.testing import CliRunner
 
-from operx_cli import settings
+from operx_cli import __version__, settings
 from operx_cli.app import app
 from operx_cli.commands.common import Session
 from operx_cli.shell import STOP_WORDS, run_shell, split_line, suggestions
@@ -183,6 +183,23 @@ def test_a_refused_call_shows_the_apis_words_and_the_session_goes_on(capsys):
     assert "Ana Ruiz" in out.out
 
 
+def test_a_crash_says_what_failed_and_where_and_the_session_goes_on(capsys):
+    # A list where the CLI expects an object: an answer it doesn't know.
+    fake = FakeApi({("GET", "search/"): [{"kind": "employee", "id": 4, "title": "Ana Ríos"}], ("GET", "auth/me/"): ME})
+
+    shell(fake, "buscar prensa-zeta-77", "cuenta")
+
+    out = capsys.readouterr()
+    # The line a person sends to support: the error, the CLI's own line where
+    # it happened and the version, whose line numbers those are.
+    assert "AttributeError: 'list' object has no attribute 'get'" in out.err
+    assert "commands/search.py:" in out.err
+    assert f"operx {__version__}" in out.err
+    # Not what was typed: it can hold anything, a token included.
+    assert "prensa-zeta-77" not in out.err
+    assert "Ana Ruiz" in out.out
+
+
 def test_help_inside_the_shell_does_not_end_it(capsys):
     shell(FakeApi({("GET", "auth/me/"): ME}), "ordenes --help", "cuenta")
     assert "Ana Ruiz" in capsys.readouterr().out