[logback-dev] [qos-ch/logback] ef9f12: Make sure AsyncAppenderBaseTest tests have a timeo...

Ceki Gulcu ceki at qos.ch
Wed May 30 17:00:07 CEST 2012


  Branch: refs/heads/master
  Home:   https://github.com/qos-ch/logback
  Commit: ef9f12ba16e25c08a021850eb21032099e50372c
      https://github.com/qos-ch/logback/commit/ef9f12ba16e25c08a021850eb21032099e50372c
  Author: Dale Wijnand <dale.wijnand at gmail.com>
  Date:   2012-05-30 (Wed, 30 May 2012)

  Changed paths:
    M logback-core/src/test/java/ch/qos/logback/core/AsyncAppenderBaseTest.java

  Log Message:
  -----------
  Make sure AsyncAppenderBaseTest tests have a timeout (LBGENERAL-54).

I had an issue where I wasn't able to run the tests because the build
hung after ch.qos.logback.core.util.StatusPrinterTest. By git
bisecting I found that the issue was in the
f2d15babe3275e7b0d618a020f09a863a0fa6a54 commit.

Eventually after much trial and error & head-scratching I stumbed upon
a fix involving making sure that each AsyncAppenderBaseTest test
method had a timeout.


  Commit: 5ebb0e03a9eb819cf2b3e83af0b7c0e13b3ff1b3
      https://github.com/qos-ch/logback/commit/5ebb0e03a9eb819cf2b3e83af0b7c0e13b3ff1b3
  Author: Ceki Gulcu <ceki at qos.ch>
  Date:   2012-05-30 (Wed, 30 May 2012)

  Changed paths:
    M logback-core/src/test/java/ch/qos/logback/core/AsyncAppenderBaseTest.java

  Log Message:
  -----------
  Merge pull request #48 from dwijnand/LBGENERAL-54

Make sure AsyncAppenderBaseTest tests have a timeout (LBGENERAL-54).


Compare: https://github.com/qos-ch/logback/compare/685b0aa...5ebb0e0


More information about the logback-dev mailing list