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.

Sign up for GitHub

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

Jump to bottom

Use gtest_main #856

Merged
merged 1 commit into from
Mar 19, 2017
Merged

Use gtest_main #856

merged 1 commit into from
Mar 19, 2017

Conversation

jslee02
Copy link
Member

@jslee02 jslee02 commented Mar 18, 2017 •
edited
Loading

This PR links gtest_main to each test target so that we don't need to repeatedly write the same main() function.

Reference

This change is Reviewable

@jslee02 jslee02 added this to the DART 6.1.3 milestone Mar 18, 2017
@psigen
Copy link
Collaborator

psigen commented Mar 18, 2017

👍 This is exactly what gtest_main exists for. Good find.

@mkoval
Copy link
Collaborator

mkoval commented Mar 18, 2017

👍 LGTM. Great find! 🎉

@jslee02 jslee02 merged commit 2945b27 into release-6.1 Mar 19, 2017
@jslee02 jslee02 deleted the feature/gtest_main branch March 21, 2017 21:14
@jslee02 jslee02 modified the milestones: DART 6.2.1, DART 6.1.3 Aug 7, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
DART 6.2.1
Development

Successfully merging this pull request may close these issues.
3 participants
@jslee02 @psigen @mkoval