-
Notifications
You must be signed in to change notification settings - Fork 10
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
Add unit test to function validate teachers #112
base: master
Are you sure you want to change the base?
Conversation
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.
describe('teachers', function () { | ||
it('should return empty list when has not errors', function () { | ||
const disciplinas = [ | ||
{ |
There was a problem hiding this comment.
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.
{ | |
{ |
{ | ||
teoria: { | ||
name: 'Vanessa Kruth', | ||
error: null, |
There was a problem hiding this comment.
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.
error: null, | |
error: null, |
name: 'Vanessa Kruth', | ||
error: null, | ||
}, | ||
pratica: { |
There was a problem hiding this comment.
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.
pratica: { | |
pratica: { |
const disciplinas = [ | ||
{ | ||
teoria: { | ||
name: 'Vanessa Kruth', |
There was a problem hiding this comment.
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.
name: 'Vanessa Kruth', | |
name: 'Vanessa Kruth', |
it('should return empty list when has not errors', function () { | ||
const disciplinas = [ | ||
{ | ||
teoria: { |
There was a problem hiding this comment.
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.
teoria: { | |
teoria: { |
}, | ||
{ | ||
teoria: { | ||
name: 'Isidro', |
There was a problem hiding this comment.
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.
name: 'Isidro', | |
name: 'Isidro', |
{ | ||
teoria: { | ||
name: 'Isidro', | ||
error: 'Sala não encontrada', |
There was a problem hiding this comment.
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.
error: 'Sala não encontrada', | |
error: 'Sala não encontrada', |
teoria: { | ||
name: 'Vanessa Kruth', | ||
error: null, | ||
}, |
There was a problem hiding this comment.
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.
}, | |
}, |
pratica: { | ||
name: 'Vanessa Kruth', | ||
error: null, | ||
} |
There was a problem hiding this comment.
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.
} | |
} |
error: null, | ||
} | ||
}, | ||
]; |
There was a problem hiding this comment.
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.
]; | |
] |
Change Request
Description
Adds new tests to ensure it works
How do I test this?
run
yarn test
Checklist