Skip to content

Refactor methods to correct behavior and add constructor#94

Open
nathaliafab wants to merge 1 commit intospgroup:developfrom
nathaliafab:fix-motivating
Open

Refactor methods to correct behavior and add constructor#94
nathaliafab wants to merge 1 commit intospgroup:developfrom
nathaliafab:fix-motivating

Conversation

@nathaliafab
Copy link
Member

Este PR corrige os métodos normalizeWhiteSpace(), removeDuplicateWords() e cleanText(). Também adiciona um novo construtor à classe DFPBaseSample que aceita um parâmetro String text.

public class DFPBaseSample {
private String text;

public DFPBaseSample(String text){
Copy link
Collaborator

Choose a reason for hiding this comment

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

Tem que executar os testes que usam esse arquivo como entrada. Principalmente porque os números das linhas foram alterados. Verificar se o teste continua passando e alterar as linhas.

Copy link
Collaborator

Choose a reason for hiding this comment

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

Copy link
Member

Choose a reason for hiding this comment

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

@galilasmb podes fazer isso? Nathalia é da parte de testes, só está ajudando a gente de estática com esses ajustes.

Copy link
Member

Choose a reason for hiding this comment

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

@galilasmb fizesse isso?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants