Sprint 3 - Issue 10 - Grupo 3#52
Open
MariaClaudia1328 wants to merge 153 commits intoEngSwCIC:mainfrom
Open
Conversation
arquivos VisualizarCards.feature e VisualizarGrafos.feature adicionados
O bdd ficou mais especifico.
arquivos .feature adicionados
Corrigindo a palavra "pressione" que foi comida durante a ediçao.
Adicionado classe de dados usado nos cards, não está completo a questão de relacionar a entrada e o experimento, e nem está certo sobre quais informações devem ser pegas. Existem possiveis bugs relacionados ao regex, pode ocorrer de string der match em algo inesperado (confirmar quais caracteres podem ser usados no nome do robô), ou que log tenha um formado customizado ou inesperado (confirmar se arquivo log no db não pode ser inserido de forma irregular sob risco de falha na visualização da informação ou injection de código malicioso)
…ntos Corrigido nomes de variáveis e métodos para estarem de acordo com a convenção de nomes do Ruby. Problemas no código se mantém: -É preciso saber quem é o responsável por definir a entrada do DB (Para o código foi assumido que é definido quando o teste é executado, e que qualquer atualização nos cards devem ser iniciados nesse momento) -Problemas com o uso de regex para extração de dados, se não garantir que os dados inseridos nos logs do DB são sanatizados, pode gerar problema de visualização nos cards, ou no pior caso um ataque de injeção (talvez os dados do campo logs devam ser sanatizados nessa etapa novamente?).
…se é responsável por tratar os resultados dos logs do experimento. Movido e modificado classe trial_result para trial_execution.rb, classe é responsável por tratar os resultados dos logs do experimento. Problemas da classe se mantém: -É preciso que seja confirmado se a entrada dos logs vão ser informados para o card durante a execução do experimento. -É preciso confirmar se dados do campo log do Banco de Dados estão sanitizados, e se é seguro usar com regex, sob risco de apresentar falha de visualização e possivel ataque de injeção.
Corrigido bug relacionado ao String.scan e regex falho, substituido por String.match e usado regex para considerar apenas a última ocorrencia. A string não é obrigada a terminar com "," mais, pode terminar com "}" também.
…imentos-para-react' into sprint-3
…-dados' into sprint-3
…-a-mensagem-não-há-testes' into sprint-3
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Sobre a entrega
Issue 10
Dado que sou um usuario quero poder visualizar os testes de um experimento de maneira dinamica (visão de cards e grafos) - #10
Equipe
Para executar
bundle installeyarn installrake db:migraterails db:seedbin/devObservações