Ticket #188 (closed tarefa: fixed)

Opened 9 years ago

Last modified 7 years ago

Definir aspectos de QA do ExpressoMail

Reported by: amuller Owned by: amuller
Priority: média Milestone:
Component: ExpressoMail Version: 1.0
Severity: Keywords:
Cc: WorkGroup:

Description (last modified by amuller) (diff)

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

Attachments

teste_de_cobertura_ExMailJS Download (1.1 KB) - added by amuller 9 years ago.
Teste de cobertura, funções mais chamadas
cover_connector Download (436 bytes) - added by amuller 9 years ago.
Teste de cobertura, funções PHP mais chamadas pelo connector
metrica.sh Download (497 bytes) - added by amuller 7 years ago.
Conta no numero de funcoes por arquivo e de linhas por funcao

Change History

Changed 9 years ago by amuller

Teste de cobertura, funções mais chamadas

comment:1 Changed 9 years ago by amuller

  • Description modified (diff)

comment:2 Changed 9 years ago by amuller

  • Status changed from new to assigned

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

Changed 9 years ago by amuller

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

Changed 7 years ago by amuller

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

comment:3 Changed 7 years ago by 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.

comment:4 Changed 7 years ago by 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.

comment:5 Changed 7 years ago by amuller

  • Status changed from assigned to closed
  • Resolution set to fixed

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

Note: See TracTickets for help on using tickets.