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

git: improvement on success commit author tests #671

Merged
merged 1 commit into from
Oct 11, 2021

Conversation

matheusalcantarazup
Copy link
Contributor

@matheusalcantarazup matheusalcantarazup commented Oct 11, 2021

Previously on tests of success getting commit author we was just
asserting that the result is not empty, but actually the tests was not
getting the correct commit author and was returning commit author not
found. This commit add asserts to make sure that we are successful
getting the commit author info from a file and line.

Signed-off-by: Matheus Alcantara matheus.alcantara@zup.com.br

- What I did

- How to verify it

- Description for the changelog

@matheusalcantarazup matheusalcantarazup force-pushed the improvement-git-tests branch 2 times, most recently from 7f86942 to 718bd4e Compare October 11, 2021 18:00
@matheusalcantarazup matheusalcantarazup changed the title improvement git tests git: improvement on success commit author tests Oct 11, 2021
Previously on tests of success getting commit author we was just
asserting that the result is not empty, but actually the tests was not
getting the correct commit author and was returning commit author not
found. This commit add asserts to make sure that we are successful
getting the commit author info from a file and line.

Signed-off-by: Matheus Alcantara <matheus.alcantara@zup.com.br>
@matheusalcantarazup matheusalcantarazup merged commit 26f6e1d into main Oct 11, 2021
@matheusalcantarazup matheusalcantarazup deleted the improvement-git-tests branch October 11, 2021 20:00
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.

3 participants