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

[S2JUnit4] JUnit4.4と4.11の両方で動作するようにしました。 #8

Merged
merged 3 commits into from Nov 29, 2014

Conversation

manhole
Copy link
Contributor

@manhole manhole commented Sep 25, 2014

S2JUnit4を、JUnit 4.4とJUnit 4.11の両方で動作するようにしました。

今まではS2JUnit4をJUnit 4.11で使用すると下記のようなNoSuchMethodErrorが発生が発生していましたが、回避するようにしました。

(eclipseで1メソッドを指定してテスト実行した際のエラーです)

java.lang.NoSuchMethodError: org.junit.runner.manipulation.Filter.apply(Lorg/junit/runner/Runner;)V
    at org.seasar.framework.unit.Seasar2.filter(Seasar2.java:183)
    at org.junit.runner.manipulation.Filter.apply(Filter.java:79)
    at org.junit.internal.requests.FilterRequest.getRunner(FilterRequest.java:34)
    at org.eclipse.jdt.internal.junit4.runner.JUnit4TestReference.<init>(JUnit4TestReference.java:33)
    at org.eclipse.jdt.internal.junit4.runner.JUnit4TestMethodReference.<init>(JUnit4TestMethodReference.java:25)
    at org.eclipse.jdt.internal.junit4.runner.JUnit4TestLoader.createTest(JUnit4TestLoader.java:54)
    at org.eclipse.jdt.internal.junit4.runner.JUnit4TestLoader.loadTests(JUnit4TestLoader.java:38)
    at org.eclipse.jdt.internal.junit.runner.RemoteTestRunner.runTests(RemoteTestRunner.java:444)
    at org.eclipse.jdt.internal.junit.runner.RemoteTestRunner.runTests(RemoteTestRunner.java:675)
    at org.eclipse.jdt.internal.junit.runner.RemoteTestRunner.run(RemoteTestRunner.java:382)
    at org.eclipse.jdt.internal.junit.runner.RemoteTestRunner.main(RemoteTestRunner.java:192)

JUnitのJUnit4ClassRunnerやErrorReportingRunnerと同じように修正しました。
@koichik
Copy link
Member

koichik commented Nov 22, 2014

いいと思います。追加のコミット二つは適切にsquashしてください。

P.S.
ブランチにpushしただけだと通知されないので気づきませんでした。
push 後はコメントお願いします。

@manhole
Copy link
Contributor Author

manhole commented Nov 24, 2014

squashしてpush -fしました。

koichik added a commit that referenced this pull request Nov 29, 2014
[S2JUnit4] JUnit4.4と4.11の両方で動作するようにしました。
@koichik koichik merged commit a03f934 into seasarorg:master Nov 29, 2014
@koichik
Copy link
Member

koichik commented Nov 29, 2014

ありがとうございました、マージしました。

@koichik koichik mentioned this pull request Nov 29, 2014
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.

None yet

2 participants