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

Netty 3: ThreadRenamingRunnableTest fails #951

Closed
CruzBishop opened this issue Jan 18, 2013 · 6 comments
Closed

Netty 3: ThreadRenamingRunnableTest fails #951

CruzBishop opened this issue Jan 18, 2013 · 6 comments

Comments

@CruzBishop
Copy link
Contributor

Jan 18, 2013 4:11:40 PM org.jboss.netty.util.ThreadRenamingRunnable

WARNING: Failed to determine the thread name

org.junit.ComparisonFailure: expected:<[proposed]> but was:<[foo]>

at org.junit.Assert.assertEquals(Assert.java:125)
at org.junit.Assert.assertEquals(Assert.java:147)
at org.jboss.netty.util.ThreadRenamingRunnableTest$5.determineThreadName(ThreadRenamingRunnableTest.java:105)
at org.jboss.netty.util.ThreadRenamingRunnableTest$ImmediateExecutor.execute(ThreadRenamingRunnableTest.java:146)
at org.jboss.netty.util.ThreadRenamingRunnableTest.testWithoutSecurityManager(ThreadRenamingRunnableTest.java:43)
@normanmaurer
Copy link
Member

@CruzBishop can you re-test with latest 3 code ? Everything works here

@CruzBishop
Copy link
Contributor Author

Will be able to re- test on Sunday. Thanks, Norman :-)
On 25 Jan 2013 20:32, "Norman Maurer" notifications@github.com wrote:

@CruzBishop https://github.com/CruzBishop can you re-test with latest 3
code ? Everything works here


Reply to this email directly or view it on GitHubhttps://github.com//issues/951#issuecomment-12695548.

@normanmaurer
Copy link
Member

@CruzBishop ping :)

@CruzBishop
Copy link
Contributor Author

Arghh, sorry! D:

Keep getting called in to work and to tired to do anything once I get home.
Will test tomorrow if everything goes well
On 30 Jan 2013 16:40, "Norman Maurer" notifications@github.com wrote:

@CruzBishop https://github.com/CruzBishop ping :)


Reply to this email directly or view it on GitHubhttps://github.com//issues/951#issuecomment-12876537.

@CruzBishop
Copy link
Contributor Author

Alright, managed to score Monday off. Will let you know then, sorry about
the delays
On 30 Jan 2013 16:40, "Norman Maurer" notifications@github.com wrote:

@CruzBishop https://github.com/CruzBishop ping :)


Reply to this email directly or view it on GitHubhttps://github.com//issues/951#issuecomment-12876537.

@normanmaurer
Copy link
Member

I will close this as I can't reproduce it at all with current 3 branch

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants