IslasGECI / wrangler

GNU Affero General Public License v3.0
0 stars 0 forks source link

Inspecci贸n NR 馃 #9

Closed nepito closed 3 years ago

nepito commented 3 years ago

Es posible utilizar decoradores para refactorizar las pruebas test_Field_add_property y test_Field_modify_property que tienen c贸digo repetido: image

Tal vez un cambio podr铆a ser:

def Field_methods_property(specific_method):
    def wrapper():
        name_property = "new_property"
        expected_value = specific_method(name_property)
        dict_field = Campo.get_dict()
        obtained_value = dict_field[name_property]
        assert expected_value == obtained_value
    return wrapper

@Field_methods_property
def test_Field_add_property(name_property):
    expected_value = 0
    Campo.add_property(name_property, expected_value)
    return expected_value

Aqu铆 hay c贸digo repetido, tal vez podr铆as aplicar un decorador que revise que la variable de entrada sea de la clase esperda: image


Estudio: 40 minutos Preparaci贸n: 60 minutos

nepito commented 3 years ago

Es posible utilizar decoradores para refactorizar las pruebas test_Field_add_property y test_Field_modify_property que tienen c贸digo repetido: image

Tal vez un cambio podr铆a ser:

def Field_methods_property(specific_method):
    def wrapper():
        name_property = "new_property"
        expected_value = specific_method(name_property)
        dict_field = Campo.get_dict()
        obtained_value = dict_field[name_property]
        assert expected_value == obtained_value
    return wrapper

@Field_methods_property
def test_Field_add_property(name_property):
    expected_value = 0
    Campo.add_property(name_property, expected_value)
    return expected_value

Aqu铆 hay c贸digo repetido, tal vez podr铆as aplicar un decorador que revise que la variable de entrada sea de la clase esperda: image

Estudio: 40 minutos Preparaci贸n: 60 minutos

Ya borramos estas pruebas, pues borramos la clase Field.

devarops commented 3 years ago

@nepito 驴Qu茅 le falta a este issue para poder cerrarlo?

nepito commented 3 years ago

Voy a cerrar el issue y abrir uno nuevo cuando hagamos la siguiente inspecci贸n.

devarops commented 3 years ago

Celebrate