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

retorno de itens removido #651

Conversation

Alassana-Djigo
Copy link
Contributor

Saudações esta PR é devido ao problema #645 que se eu passar apenas um parâmetro no método encaixar() nenhum elemento é removido. mas não é que nenhum elemento é removido é todos elementos são removidos e se remover o escreva(elementos) ele só mostra ] que também é um erro no minimo deveria mostrar um vector vazio

var elementos = ['Oscilador', 'Circuito', 'Modulador', 'Refrigerador']

var elementosRemovidos = elementos.encaixar(1)

escreva(elementosRemovidos)
escreva(elementos)

Oque eu não percebi foi que expect(vetor).toStrictEqual([1, 2, 10, 3]); no codigo a baixo porque que ele esperava [1, 2, 10, 3] acho que o que deveria ser esperado é um vector vazio porque os parametros de inicio e fim no encaixar mostra o valor de inicio maior que o fim então oque deve acontecer nestes casos é apresnetar um vector vazio (o mesmo acontece no Js com metodo slice) por isso que o var elementosRemovidos = elementos.encaixar(1) Só esta apresentando ]

it('Inserindo novo elemento', async () => {
            let vetor = [1, 2, 3]
            await primitivasVetor.encaixar(interpretador, vetor, 2, 0, 10);
            expect(vetor).toStrictEqual([1, 2, 10, 3]);
        });

@leonelsanchesdasilva
Copy link
Contributor

Saudações @Alassana-Djigo. Poderia por favor escrever um teste unitário pelo menos para o caso do else da sua implementação? Obrigado pela colaboração.

@Alassana-Djigo
Copy link
Contributor Author

Saudações @Alassana-Djigo. Poderia por favor escrever um teste unitário pelo menos para o caso do else da sua implementação? Obrigado pela colaboração.

Claro que sim retornarei o mais breve possivel...

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.

Obrigado!

@leonelsanchesdasilva leonelsanchesdasilva merged commit abf77e7 into DesignLiquido:principal Feb 18, 2024
1 check passed
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.

2 participants