Definir aspectos de QA do ExpressoMail

[amuller]

• Cobertura de código.
• UnitTest.
• Testes automatizados.

[amuller]

Teste de cobertura, funções mais chamadas

[amuller]

Mais um teste

Element = 1088 exist_className = 481 get_lang = 370 lang_folder = 249 get_current_folder = 214 add_className = 68 select_msg = 50 make_tr_message = 22 alternate_border = 20 cExecute = 18 round = 16 findPosY = 13 url_decode = 10 draw_paging = 9 setBorderAttributes = 7 findPosX = 6 update_menu = 4 draw_tree_folders = 4 draw_header_box = 3 draw_footer_box = 3 draw_box = 3 kill_current_box = 2 cRichTextEditor = 1

[amuller]

Teste de cobertura, funções PHP mais chamadas pelo connector

[amuller]

Conta no numero de funcoes por arquivo e de linhas por funcao

[amuller]

Eu fiz um script que conta no numero de funçoes globais por arquivo e de linhas por funçao o resultado é

FILES
common_functions.js - 57
main.js - 50
draw_api.js - 29


FUNCTIONS
draw_paging - 107
show_msg - 123
search_contacts - 124
ConstructMenuTools - 135
move_msgs2 - 143
return_save - 144
draw_box - 229
make_tr_message - 233
draw_tree_folders - 289
new_message - 481
draw_new_message - 567
draw_message - 892

O que se tira de conclusão disso é que: os maiores arquivos e funções deveriam ser quebrados. Pois quando elas são grandes elas fazem muitas coisas ao mesmo tempo. E fica difícil reaproveitamento de código desse jeito. Além da compreenção.

[amuller]

O draw_api é um arquivo excessivamente grande, é o maior em linhas de código. Além de ter 3 funções com mais de 800 linhas. Depois de muitas tentativas de descobrir porque o draw_api não compacta no packer tive que dividir o problema a r2551 quebra o draw_api em 2 arquivos (draw_api_folders e draw_api_messages), além de quebrar o draw_new_message em outros 3 pedaços.

[amuller]

Este ticket já cumpriu parte do papel dele. Foi aberto #1059