bug: test_parse_attachments_size_limit_accumulates fails with path validation error

Abierto Apto para principiantes
#593 0 comentarios 0 reacciones 0 asignados Ver en GitHub

Nadie ha tomado este issue todavía.

Evaluación

Dificultad
2/5
Tiempo estimado
1-3 horas
Aptitud para principiantes
64/100
Tipo de issue
Error
Claridad
Bastante claro
Estado de actividad
Estancado
Stack tecnológico
rust
Área
testing

Línea de trabajo

Comienza con src/helpers/gmail/mod.rs y ejecuta helpers::gmail::tests::test_parse_attachments_size_limit_accumulates para reproducir el error intermitente de validación de rutas. Lee validate_safe_file_path y la configuración del archivo temporal de la prueba, y luego haz que la prueba pase de forma fiable, conservando la aserción de que se supera el límite de tamaño de los adjuntos.

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

Descripción

area: core type: bug

Description

The test helpers::gmail::tests::test_parse_attachments_size_limit_accumulates intermittently fails with a path validation error:

thread 'helpers::gmail::tests::test_parse_attachments_size_limit_accumulates' panicked at src/helpers/gmail/mod.rs:2670:9:
error should mention exceeding limit: --attach '/path/to/.tmpmkVJJo/big1.bin' resolves to '/path/to/.tmpmkVJJo/big1.bin' which is outside the current directory

Root Cause

The test creates temporary files using a relative path with a hidden directory prefix (.tmp*), but the path validation logic (validate_safe_file_path) rejects the resolved canonical path as "outside the current directory" due to how symlink/canonical resolution interacts with temporary directories.

Impact

  • Blocks git push when the pre-push hook runs the full test suite
  • Not related to any specific code change — reproducible on main

Expected Behavior

The test should either:

  1. Create temp files in a way that passes path validation, or
  2. Use std::env::temp_dir() with an appropriate test-only bypass for path validation
Lenguaje dominante
Rust
Estrellas
31.1k
Forks
1.8k
Sin PR fusionados en 30 d

Guía de contribución

Abrir la guía de contribución

Primeros pasos

  1. Lee el issue completo y luego la guía de contribución del proyecto.
  2. Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
  3. Haz un fork del repositorio y trabaja en una rama.
  4. Abre un pull request que haga referencia al número del issue.

Más de googleworkspace/cli

Todos los issues de googleworkspace/cli

Issues similares

Más issues de Rust

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.