Issue 52 in vogar: Bring back JUnit 4 support

[vo...@googlecode.com]

Status: Accepted
Owner: jessewil...@google.com
Labels: Type-Defect Priority-Low

New issue 52 by jessewil...@google.com: Bring back JUnit 4 support
http://code.google.com/p/vogar/issues/detail?id=52

I've attached an ugly patch that worked for running EasyMock's JUnit 4
tests. If this comes up again we may want to support JUnit 4 properly.

Attachments:
junit4.patch 5.7 KB

[vo...@googlecode.com]

Comment #1 on issue 52 by jessewil...@google.com: Bring back JUnit 4 support
http://code.google.com/p/vogar/issues/detail?id=52

Attached is a better JUnit 4 support patch that also supports Mockito's
tests.

Attachments:
junit4-v2.patch 7.0 KB

[vo...@googlecode.com]

Comment #2 on issue 52 by kr...@google.com: Bring back JUnit 4 support
http://code.google.com/p/vogar/issues/detail?id=52

Android is able to use JUnit4 style test cases with "adb am instrument"
now, so it would be nice to be able to run the same tests with vogar when
possible.

[vo...@googlecode.com]

Comment #3 on issue 52 by ke...@the-b.org: Bring back JUnit 4 support
http://code.google.com/p/vogar/issues/detail?id=52

Patch based on previous comment. Only implements Suite and Test support.
Nothing too fancy.

Attachments:
junit4.patch 45.4 KB

--
You received this message because this project is configured to send all
issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings

[vo...@googlecode.com]

Comment #4 on issue 52 by ke...@the-b.org: Bring back JUnit 4 support
http://code.google.com/p/vogar/issues/detail?id=52

Added Parameterized tests

Attachments:
junit4-with-param.patch 53.7 KB

[vo...@googlecode.com]

Updates:
Owner: ---
Cc: e...@google.com jwil...@squareup.com

Comment #5 on issue 52 by e...@google.com: Bring back JUnit 4 support
http://code.google.com/p/vogar/issues/detail?id=52

(we shouldn't add annotations we don't actually implement. i'd rather fail
to compile than compile but not do the right thing.)

[vo...@googlecode.com]

Comment #6 on issue 52 by e...@google.com: Bring back JUnit 4 support
http://code.google.com/p/vogar/issues/detail?id=52

kroot says the patch implements all the new annotations it defines.

[vo...@googlecode.com]

Updates:
Status: Fixed

Comment #7 on issue 52 by kr...@google.com: Bring back JUnit 4 support
http://code.google.com/p/vogar/issues/detail?id=52

Committed as r290