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

Teste Unitário resolveProfessor.spec.js #110

Open
wants to merge 23 commits into
base: master
Choose a base branch
from

Conversation

hadeoliveira
Copy link
Contributor

Change Request

Description

Implementação do teste unitário resolveProfessor.spec.js

How do I test this?

O teste pode ser feito executando o comando abaixo

yarn test

@hadeoliveira hadeoliveira changed the title Unit tests helpers Teste Unitário resolveProfessor.spec.js Dec 8, 2021
JhonataSantana and others added 6 commits December 7, 2021 23:12
Agora o comando "Coverage" está funcional e o report é registrado no arquivo "coverage.lcov", deixando a função de análise dos resultados, para a geração de badge ou outras finalidades, de forma independente por meio de outros comandos, como o utilizado no CI em conjunto com o Codecov.
name: 'Vanessa Kruth',
error: null,
},
pratica: {
Copy link
Member

Choose a reason for hiding this comment

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

Codacy has a fix for the issue: Expected indentation of 8 spaces but found 12.

Suggested change
pratica: {
pratica: {

},
{
teoria: {
name: 'Isidro',
Copy link
Member

Choose a reason for hiding this comment

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

Codacy has a fix for the issue: Expected indentation of 10 spaces but found 16.

Suggested change
name: 'Isidro',
name: 'Isidro',

teoria: {
name: 'Vanessa Kruth',
error: null,
},
Copy link
Member

Choose a reason for hiding this comment

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

Codacy has a fix for the issue: Expected indentation of 8 spaces but found 12.

Suggested change
},
},

{
teoria: {
name: 'Vanessa Kruth',
error: null,
Copy link
Member

Choose a reason for hiding this comment

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

Codacy has a fix for the issue: Expected indentation of 10 spaces but found 16.

Suggested change
error: null,
error: null,

error: null,
}
},
];
Copy link
Member

Choose a reason for hiding this comment

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

Codacy has a fix for the issue: Extra semicolon.

Suggested change
];
]

it('should return empty list when has not errors', function () {
const disciplinas = [
{
teoria: {
Copy link
Member

Choose a reason for hiding this comment

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

Codacy has a fix for the issue: Expected indentation of 8 spaces but found 12.

Suggested change
teoria: {
teoria: {

pratica: {
name: 'Vanessa Kruth',
error: null,
}
Copy link
Member

Choose a reason for hiding this comment

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

Codacy has a fix for the issue: Expected indentation of 8 spaces but found 12.

Suggested change
}
}

describe('teachers', function () {
it('should return empty list when has not errors', function () {
const disciplinas = [
{
Copy link
Member

Choose a reason for hiding this comment

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

Codacy has a fix for the issue: Expected indentation of 6 spaces but found 8.

Suggested change
{
{

const disciplinas = [
{
teoria: {
name: 'Vanessa Kruth',
Copy link
Member

Choose a reason for hiding this comment

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

Codacy has a fix for the issue: Expected indentation of 10 spaces but found 16.

Suggested change
name: 'Vanessa Kruth',
name: 'Vanessa Kruth',

{
teoria: {
name: 'Isidro',
error: 'Sala não encontrada',
Copy link
Member

Choose a reason for hiding this comment

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

Codacy has a fix for the issue: Expected indentation of 10 spaces but found 16.

Suggested change
error: 'Sala não encontrada',
error: 'Sala não encontrada',

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