uqbar-project / wollok

Wollok Programming Language
GNU General Public License v3.0
60 stars 16 forks source link

El diagrama dinámico no muestra instancias en los tests con describe #1954

Closed asanzo closed 3 years ago

asanzo commented 3 years ago

Este código :

class Swallow {
    var energy = 100
}
only test "testX" {
    var littleJosephine = new Swallow()
}

Escupe este diagrama: Selección_999(574)

Pero esté código:

class Swallow {
    var energy = 100
}

describe "f" {
    only test "testX" {
        var littleJosephine = new Swallow()
    }   
}

No escupe ningún diagrama: Selección_999(533)

Sólo pasa con las instancias de las clases, porque con los wko parece funcionar correctamente

fdodino commented 3 years ago

Ah, @asanzo, qué bien que descubriste el patrón. Ahora creo que encontré el origen del problema:

image

fijate que si inicializás a la golondrina en el test lo detecta bien. Fue un error que introduje al tratar de determinar si pusieron el flag only a los tests, y la primera expresión es la que trata de determinar si está dentro de un only test. Pero resulta que la primera expresión que se evalúa para Swallow es la inicialización de la variable energía. Entonces la expresión 100 no está dentro de un only test sino dentro de una clase. Por eso no está guardando la info de los objetos. Good catch!

fdodino commented 3 years ago

Mucho mejor así: :smile: @asanzo

image

asanzo commented 3 years ago

Qué buena onda!!!!