Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Issue #59] Atribuição Abreviada / Múltiplos Valores #124

Merged
merged 25 commits into from
Oct 10, 2024

Conversation

VitBrandao
Copy link
Contributor

No description provided.

@VitBrandao VitBrandao added the em andamento Trabalho em progresso label Sep 23, 2024
@VitBrandao VitBrandao self-assigned this Sep 23, 2024
@VitBrandao
Copy link
Contributor Author

@leonelsanchesdasilva trabalho em andamento, ainda não atingi o resultado esperado, mas estou no caminho, sigo testando e testando. Mudarei a label para concluído assim que finalizar.

Copy link

github-actions bot commented Sep 24, 2024

Coverage report

St.
Category Percentage Covered / Total
🟢 Statements
92.19% (+0.46% 🔼)
6259/6789
🟡 Branches
76.93% (+2% 🔼)
847/1101
🟡 Functions
78.87% (+0.27% 🔼)
642/814
🟢 Lines
92.32% (+0.46% 🔼)
6249/6769
Show new covered files 🐣
St.
File Statements Branches Functions Lines
🟢
... / atribuicao-abreviada.ts
100% 100% 100% 100%
Show files with reduced coverage 🔻
St.
File Statements Branches Functions Lines
🟢
... / avaliador-sintatico.ts
85.08% (+0.06% 🔼)
82.59% (-0.75% 🔻)
83.33% (+0.72% 🔼)
85.29% (+0.02% 🔼)

Test suite run success

102 tests passing in 18 suites.

Report generated by 🧪jest coverage report action from 9762106

@VitBrandao
Copy link
Contributor Author

VitBrandao commented Sep 24, 2024

@leonelsanchesdasilva Fiz alterações para que a nova estrutura do Avaliador Sintático siga funcionando para todos os casos de valor único - número, número-quantificador, método, qualitativo.

Ou seja, nesse momento, o Avaliador está funcionando da mesma forma que estava antes de abrir esse PR, mas com a diferença de que agora ele está estruturado para comportar múltiplos valores também, uma vez que o método valoresModificador() agora retorna um array.

O que está faltando ainda é implementar uma lógica que permita passar os devidos parâmetros para a classe SeletorModificador quando tivermos casos de múltiplos valores/ atribuição abreviada.

No geral, a classe SeletorModificador recebe no construtor os seguintes parâmetros:

  • nomeFolEs
  • valor
  • quantificador?
  • pragmas?

Mas agora, com o recebimento de múltiplos valores, o parâmetro valor está sendo diferente do que era antes.
E também temos casos em que pode haver mais de um quantificador, como por exemplo:

recuo: 10px 15px 10px 15px;

Enfim, apenas te atualizando para saber em que pé está o PR. O arquivo do Avaliador está cheio de trechos comentados, mas quando eu finalizar, limparei todos. Sigo trabalhando. Após solucionar essas questões, acredito que a Issue esteja solucionada. No momento, o PR segue em-andamento.

Copy link
Contributor

@leonelsanchesdasilva leonelsanchesdasilva left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Estamos no caminho certo. Além dos testes unitários, pedi algumas modificações.

fontes/avaliador-sintatico/avaliador-sintatico.ts Outdated Show resolved Hide resolved
fontes/avaliador-sintatico/avaliador-sintatico.ts Outdated Show resolved Hide resolved
@VitBrandao VitBrandao added concluído Trabalho finalizado e pronto para revisão and removed em andamento Trabalho em progresso labels Oct 8, 2024
@VitBrandao
Copy link
Contributor Author

VitBrandao commented Oct 8, 2024

@leonelsanchesdasilva Testes finalizados, PR pronto para review e merge. Issue #59 solucionada sem comprometer o comportamento geral de FolEs

Copy link
Contributor

@leonelsanchesdasilva leonelsanchesdasilva left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Excelente trabalho!

@leonelsanchesdasilva leonelsanchesdasilva merged commit afef974 into principal Oct 10, 2024
3 checks passed
@leonelsanchesdasilva leonelsanchesdasilva deleted the soluciona-issue-59 branch October 10, 2024 16:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
concluído Trabalho finalizado e pronto para revisão
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants