Four TestConstraintBasedAlgorithm tests pass when nothing is thrown

Abierto Apto para principiantes
#3,210 0 comentarios 0 reacciones 0 asignados Ver en GitHub

@sharanggupta ya está trabajando en esto.

Desde el 12/11/2026.

  • #3211 de @sharanggupta — abierto

Evaluación

Dificultad
1/5
Tiempo estimado
1-3 horas
Aptitud para principiantes
85/100
Tipo de issue
Error
Claridad
Bien especificado
Estado de actividad
Activo
Stack tecnológico
java
Área
testing

Línea de trabajo

En helix-core, inspecciona los cuatro tests TestConstraintBasedAlgorithm mencionados y sus aserciones de excepciones. Añade una aserción de fallo después de cada llamada a calculate y luego ejecuta esos tests para confirmar que siguen pasando porque se lanza la excepción esperada.

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

Descripción

testCalculateNoValidAssignment, testCalculateNoValidAssignmentFirstAndThenRecovery, testSortingEarlyQuitLackCapacity and testCalculateWithInvalidAssignmentForNodeCapacity in helix-core call algorithm.calculate(clusterModel) in a try and assert only inside catch (HelixRebalanceException ex). There is no Assert.fail() after the call, so they pass when calculate returns normally.

I have a branch that adds the Assert.fail(). All four still pass, so the exception is thrown today.

Lenguaje dominante
Java
Estrellas
505
Forks
258
Merge medio
4 d 20 h
PR fusionados (30 d)
3

Preparar el entorno

  • Sin Dockerfile ni archivo de Docker Compose
  • Tiene una plantilla de pull request
  • Sin 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 apache/helix

Todos los issues de apache/helix

Issues similares

Más issues de Java

Recibe los nuevos issues en tu correo

Un resumen breve de issues de GitHub para principiantes.