setup JRuby::Rack.logger.class' level constants (for better ::Logger compatibility)

Author: kares

src/main/java/org/jruby/rack/ext/Logger.java

@@ -312,9 +312,7 @@ public IRubyObject fatal(final ThreadContext context,
     @JRubyMethod(name = "unknown")
     public IRubyObject unknown(final ThreadContext context,
         final IRubyObject msg, final Block block) {
-
-        // TODO support UNKNOWN in RackLogger ?!
-
+        // NOTE possible to "somehow" support UNKNOWN in RackLogger ?!
         return context.runtime.newBoolean( add(UNKNOWN, context, msg, block) );
     }
 
@@ -510,7 +508,7 @@ public void log(String level, String message, Throwable ex) {
 
 
     /**
-     * @deprecated mostly for 1.1 compatibility
+     * @deprecated Likely not used at all, mostly for 1.1 compatibility!
      */
     @JRubyClass(name="JRuby::Rack::ServletLog")
     public static class ServletLog extends RubyObject {

src/main/java/org/jruby/rack/ext/RackLibrary.java

@@ -61,6 +61,13 @@ public static void load(final Ruby runtime) {
               "Logger", _Object, Logger.ALLOCATOR
         );
         _Logger.defineAnnotatedMethods(Logger.class);
+        // Rails compatibility as it assumes logger.class::DEBUG to work :
+        _Logger.setConstant("DEBUG", runtime.newFixnum(Logger.DEBUG));
+        _Logger.setConstant("INFO",  runtime.newFixnum(Logger.INFO));
+        _Logger.setConstant("WARN",  runtime.newFixnum(Logger.WARN));
+        _Logger.setConstant("ERROR", runtime.newFixnum(Logger.ERROR));
+        _Logger.setConstant("FATAL", runtime.newFixnum(Logger.FATAL));
+        //_Logger.setConstant("UNKNOWN", runtime.newFixnum(Logger.UNKNOWN));
         // JRuby::Rack::ServletLog
         final RubyClass _ServletLog = _JRuby_Rack.defineClassUnder(
               "ServletLog", _Object, Logger.ServletLog.ALLOCATOR

src/spec/ruby/jruby/rack/logger_spec.rb

@@ -96,10 +96,10 @@
     expect( real_logger.formatting? ).to be false
   end
 
-#  it 'handles constant resolution (for Rails compatibility)' do
-#    expect( logger.class::DEBUG ).to eql 0
-#    expect( logger.class::FATAL ).to eql 4
-#  end
+  it 'handles constant resolution (for Rails compatibility)' do
+    expect( logger.class::DEBUG ).to eql 0
+    expect( logger.class::FATAL ).to eql 4
+  end
 
   describe JRuby::Rack::ServletLog do