birling 0.1.3 → 0.3.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: 8a220611de2cd83e6b1a287c224613765b8ee134
-  data.tar.gz: 276014a8f6f9ab3d286ec16184379c459fa42dc2
+SHA256:
+  metadata.gz: d21ba47bfcdcfd8d53f782d406a93b300955217f9cc657897a359d16ebf71ca9
+  data.tar.gz: c78c0d8a855f174ae1ae10603815a77ffe595383c6dba1057960e5183b3c15c3
 SHA512:
-  metadata.gz: 3be82ad9a9bbeacfbde7cd6ce127992cc2270894bd8e7567cf8ecc39c5f8a3d1d966b52c9b19b2d79d6f03022422faa0c54c6f0e091e8903805b1fed2e3cd93f
-  data.tar.gz: 90ad6b767bccc672dd58365ef6c3592f725e737c6c0cbc0445b9968093fa16fba7f1ef8e3a8d8a0e1145a4e95760691571c213cdaffcd7c8403f59d6fe800839
+  metadata.gz: 154f866b171df5fc48b40d901f8b21749a4f3c7f25a8560dc4e60c412059ab6be8e54d6d74d0804d38774cda6517f2d5ca570d32621cc1b82a0a1e4e5ad320f2
+  data.tar.gz: 517c2a23524f253867037122622fac1ce20545634c0f9c4f79854fafa325268c3f61869bcdf77fa7026c70084010e785d0f1c819b9ca33d639c65fd14566f9e7

data/.travis.yml

@@ -0,0 +1,9 @@
+language: ruby
+rvm:
+  - 2.4.10
+  - 2.5.8
+  - 2.6.6
+  - 2.7.2
+  - 3.0.0
+before_install:
+- export TZ=EST5EDT

data/Gemfile

@@ -1,6 +1,7 @@
-source "https://rubygems.org/"
+source 'https://rubygems.org/'
 
 group :development do
-  gem "bundler"
-  gem "jeweler"
+  gem 'bundler'
+  gem 'test-unit'
+  gem 'jeweler'
 end

data/LICENSE.txt

@@ -1,4 +1,4 @@
-Copyright (c) 2012 Scott Tadman, The Working Group.
+Copyright (c) 2012-2019 Scott Tadman, PostageApp Ltd.
 
 Permission is hereby granted, free of charge, to any person obtaining
 a copy of this software and associated documentation files (the

data/README.md

@@ -34,8 +34,8 @@ of ten hourly logs:
 ```ruby
 logger = Birling.open(
   'test.log',
-  :period => :hourly,
-  :retain_count => 10
+  period: :hourly,
+  retain_count: 10
 )
 ```
 
@@ -46,8 +46,8 @@ that period will be removed:
 ```ruby
 logger = Birling.open(
   'test.log',
-  :period => :hourly,
-  :retain_period => 10 * 3600
+  period: :hourly,
+  retain_period: 10 * 3600
 )
 ```
 
@@ -60,7 +60,7 @@ Example:
 ```ruby
 logger = Birling.open(
   'test.log',
-  :formatter => lambda { |severity, time, program, message| "#{time}> #{message}\n" }
+  formatter: -> (severity, time, program, message) { "#{time}> #{message}\n" }
 )
 ```
 
@@ -76,4 +76,4 @@ using `STDOUT` the logger will not rotate.
 
 ## Copyright
 
-Copyright (c) 2011-2012 Scott Tadman, The Working Group Inc.
+Copyright (c) 2011-2019 Scott Tadman, PostageApp Ltd.

data/Rakefile

@@ -14,7 +14,7 @@ require 'jeweler'
 
 Jeweler::Tasks.new do |gem|
   gem.name = "birling"
-  gem.homepage = "http://github.com/twg/birling"
+  gem.homepage = "http://github.com/postageapp/birling"
   gem.license = "MIT"
   gem.summary = %Q{Logger with simple log rotation system}
   gem.description = %Q{Mostly drop-in replacement for Logger with a more robust log rotation facility}

data/VERSION

@@ -1 +1 @@
-0.1.3
+0.3.1

data/birling.gemspec

@@ -2,24 +2,25 @@
 # DO NOT EDIT THIS FILE DIRECTLY
 # Instead, edit Jeweler::Tasks in Rakefile, and run 'rake gemspec'
 # -*- encoding: utf-8 -*-
-# stub: birling 0.1.3 ruby lib
+# stub: birling 0.3.1 ruby lib
 
 Gem::Specification.new do |s|
-  s.name = "birling"
-  s.version = "0.1.3"
+  s.name = "birling".freeze
+  s.version = "0.3.1"
 
-  s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
-  s.require_paths = ["lib"]
-  s.authors = ["Scott Tadman"]
-  s.date = "2015-01-31"
-  s.description = "Mostly drop-in replacement for Logger with a more robust log rotation facility"
-  s.email = "github@tadman.ca"
+  s.required_rubygems_version = Gem::Requirement.new(">= 0".freeze) if s.respond_to? :required_rubygems_version=
+  s.require_paths = ["lib".freeze]
+  s.authors = ["Scott Tadman".freeze]
+  s.date = "2021-03-30"
+  s.description = "Mostly drop-in replacement for Logger with a more robust log rotation facility".freeze
+  s.email = "github@tadman.ca".freeze
   s.extra_rdoc_files = [
     "LICENSE.txt",
     "README.md"
   ]
   s.files = [
     ".document",
+    ".travis.yml",
     "Gemfile",
     "LICENSE.txt",
     "README.md",
@@ -35,24 +36,23 @@ Gem::Specification.new do |s|
     "test/test_birling_logger.rb",
     "test/test_birling_support.rb"
   ]
-  s.homepage = "http://github.com/twg/birling"
-  s.licenses = ["MIT"]
-  s.rubygems_version = "2.2.2"
-  s.summary = "Logger with simple log rotation system"
+  s.homepage = "http://github.com/postageapp/birling".freeze
+  s.licenses = ["MIT".freeze]
+  s.rubygems_version = "3.2.3".freeze
+  s.summary = "Logger with simple log rotation system".freeze
 
   if s.respond_to? :specification_version then
     s.specification_version = 4
+  end
 
-    if Gem::Version.new(Gem::VERSION) >= Gem::Version.new('1.2.0') then
-      s.add_development_dependency(%q<bundler>, [">= 0"])
-      s.add_development_dependency(%q<jeweler>, [">= 0"])
-    else
-      s.add_dependency(%q<bundler>, [">= 0"])
-      s.add_dependency(%q<jeweler>, [">= 0"])
-    end
+  if s.respond_to? :add_runtime_dependency then
+    s.add_development_dependency(%q<bundler>.freeze, [">= 0"])
+    s.add_development_dependency(%q<test-unit>.freeze, [">= 0"])
+    s.add_development_dependency(%q<jeweler>.freeze, [">= 0"])
   else
-    s.add_dependency(%q<bundler>, [">= 0"])
-    s.add_dependency(%q<jeweler>, [">= 0"])
+    s.add_dependency(%q<bundler>.freeze, [">= 0"])
+    s.add_dependency(%q<test-unit>.freeze, [">= 0"])
+    s.add_dependency(%q<jeweler>.freeze, [">= 0"])
   end
 end
 

data/lib/birling.rb

@@ -7,6 +7,15 @@ class Birling
   
   # == Module Methods =======================================================
   
+  # Opens a new log file at the given path with options:
+  # * encoding: The encoding of the file (default: 'UTF8')
+  # * period: The rotation period to use (optional)
+  # * retain_count: How many log files to retain when rotating (optional)
+  # * retain_period: How long rotated log files are retained for (optional)
+  # * formatter: Custom log formatter (optional)
+  # * program: Name of program being logged (optional)
+  # * time_source: Source of time to use (optional)
+  # * path_format: The strftime-compatible format for the path (optional)
   def self.open(path, options = nil)
     Birling::Logger.new(path, options)
   end

data/lib/birling/formatter.rb

@@ -1,3 +1,5 @@
+require 'time'
+
 module Birling::Formatter
   # == Constants ============================================================
 
@@ -5,6 +7,12 @@ module Birling::Formatter
 
   # == Module Methods =======================================================
 
+  # Default time formatter method.
+  def self.time_format(time)
+    (time || Time.now).strftime(TIME_FORMAT_DEFAULT)
+  end
+
+  # Implementation for a plug-in Logger formatter.
   def self.call(severity, time, program, message)
     if (program)
       "[#{time.strftime(TIME_FORMAT_DEFAULT)}] <#{program}> #{message}\n"

data/lib/birling/logger.rb

@@ -1,29 +1,31 @@
+require 'fileutils'
+
 class Birling::Logger
   # == Constants ============================================================
-  
+
   # These level constants are the same as the syslog system utility
   SEVERITY = {
-    :emergency => EMERGENCY = 0,
-    :alert => ALERT = 1,
-    :critical => CRITICAL = 2,
-    :error => ERROR = 3,
-    :warning => WARNING = 4,
-    :notice => NOTICE = 5,
-    :info => INFO = 6,
-    :debug => DEBUG = 7,
-    :unknown => UNKNOWN = 999
+    emergency: EMERGENCY = 0,
+    alert: ALERT = 1,
+    critical: CRITICAL = 2,
+    error: ERROR = 3,
+    warning: WARNING = 4,
+    notice: NOTICE = 5,
+    info: INFO = 6,
+    debug: DEBUG = 7,
+    unknown: UNKNOWN = 999
   }.freeze
-  
+
   DEFAULT_SEVERITY = UNKNOWN
 
   SEVERITY_LABEL = SEVERITY.invert.freeze
-  
+
   PATH_TIME_DEFAULT = {
-    :hourly => '%Y%m%d%H'.freeze,
-    :daily => '%Y%m%d'.freeze,
-    :default => '%s'.freeze
+    hourly: '%Y%m%d%H'.freeze,
+    daily: '%Y%m%d'.freeze,
+    default: '%s'.freeze
   }.freeze
-  
+
   # == Properties ===========================================================
 
   attr_reader :severity
@@ -38,24 +40,25 @@ class Birling::Logger
   attr_reader :retain_period
   attr_reader :period
   attr_reader :rotation_time
-  
+
   # == Class Methods ========================================================
-  
+
   def self.severity(value)
     case (value)
     when Symbol
       SEVERITY[value] or DEFAULT_SEVERITY
     when String
       SEVERITY[value.to_sym] or DEFAULT_SEVERITY
-    when Fixnum
+    when Integer
       SEVERITY_LABEL[value] and value or DEFAULT_SEVERITY
     else
       DEFAULT_SEVERITY
     end
   end
-  
+
   # == Instance Methods =====================================================
 
+  # Use Birling.open(...) to create new instances.
   def initialize(log, options = nil)
     @encoding = (options and options[:encoding])
     @period = (options and options[:period])
@@ -69,6 +72,10 @@ class Birling::Logger
 
     @file_open_options = { }
 
+    @rotation_time = nil
+    @path = nil
+    @log = nil
+
     if (@encoding)
       @file_open_options[:encoding] = @encoding
     end
@@ -79,79 +86,112 @@ class Birling::Logger
     when String
       @path = log
     end
-      
+
     if (@path and @period)
       @rotation_time = self.next_rotation_time
-      
+
       @path_time_format = (PATH_TIME_DEFAULT[@period] or PATH_TIME_DEFAULT[:default])
-      
+
       @path_format ||=
         @path.sub(/\.(\w+)$/) do |s|
           '.' + @path_time_format + '.' + $1
         end
     end
-    
+
     if (@path and !@log)
       self.log_open!
     end
-  
+
     yield(self) if (block_given?)
   end
-  
+
+  # Sets the severity filter for logging. Any messages with a lower severity
+  # will be ignored. Any invalid severity options will reset the severity
+  # filter to defaults.
   def severity=(value)
     @severity = self.class.severity(value)
   end
-  
+
+  # Returns true if the log can be rotated, false otherwise.
   def can_rotate?
     !!@path
   end
-  
-  def size
-    @log and @log.respond_to
-  end
-  
+
+  # Sets the retention interval for log files. Value should respond to to_i
+  # and yield an integer value that's a positive number of seconds between
+  # rotation operations.
   def retain=(value)
     @retain = value ? value.to_i : nil
-    
+
     if (@retain_period and @retain_period <= 0)
       @retain_period = nil
     end
-    
+
     @retain_period
   end
-  
+
+  # An IO compatible method for writing a message to the file. Only non-empty
+  # messages are actually logged.
+  def write(message)
+    return unless (message.match(/\S/))
+
+    self.log(:debug, message.chomp)
+  end
+
+  def sync=(sync)
+    # Auto-sync is always turned on, so this operation is ignored.
+  end
+
+  def flush
+    # Auto-sync is always turned on, so this operation is ignored.
+  end
+
+  # Log the message for the (optional) program at the given log level. No
+  # data will be written if the current log level is not sufficiently high.
   def log(level, message = nil, program = nil)
     return unless (@log)
-    
+
     level = self.class.severity(level)
     program ||= @program
 
     self.check_log_rotation!
-    
+
     @log.write(@formatter.call(level, @time_source.now, program, message))
   end
   alias_method :add, :log
 
+  def puts(*args)
+    args.each do |arg|
+      self.log(:debug, arg)
+    end
+  end
+
+  # Writes to the log file regardless of log level.
   def <<(message)
     return unless (@log)
-    
+
     self.check_log_rotation!
-    
+
     @log.write(message)
   end
-  
+
+  # Each of the severity levels has an associated method name. For example:
+  # * debug? - Returns true if the logging level is at least debug, false
+  #            otherwise.
+  # * debug(message, program = nil) - Used to log a message with an optional
+  #                                   program name.
   SEVERITY.each do |name, level|
     define_method(:"#{name}?") do
       @severity >= level
     end
-    
-    define_method(name) do |message = nil, program = nil|
+
+    define_method(name) do |message = nil, program = nil, &block|
       return unless (@log and @severity >= level)
-      
+
       program ||= @program
-      
+
       if (!message and block_given?)
-        message = yield
+        message = block.call
       end
 
       self.check_log_rotation!
@@ -160,37 +200,43 @@ class Birling::Logger
     end
   end
 
+  # Closes the log.
   def close
     return unless (@log)
-    
+
     @log.close
     @log = nil
   end
-  
+
+  # Returns true if the log is opened, false otherwise.
   def opened?
     !!@log
   end
 
+  # Returns true if the log is closed, false otherwise.
   def closed?
     !@log
   end
 
+  # Returns the creation time of the log if opened, nil otherwise.
   def create_time
     @log and @log.ctime
   end
-  
+
+  # Returns size of the log if opened, nil otherwise.
   def size
     @log and @log.size
   end
-   
+
+  # Returns the age of the log file in seconds if opened, nil otherwise.
   def age(relative_to = nil)
-    (relative_to || @time_source.now) - @log.ctime
+    @log and (relative_to || @time_source.now) - @log.ctime
   end
-  
+
 protected
   def next_rotation_time
     case (@period)
-    when Fixnum, Float
+    when Integer, Float
       @time_source.now + @period
     when :daily
       Birling::Support.next_day(@time_source.now)
@@ -200,21 +246,21 @@ protected
       nil
     end
   end
-  
+
   def prune_logs!
     return unless (@path and (@retain_period or @retain_count))
-    
+
     log_spec = @path.sub(/\.(\w+)$/, '*')
-    
+
     logs = (Dir.glob(log_spec) - [ @path ]).collect do |p|
       stat = File.stat(p)
       create_time = (stat and stat.ctime or @time_source.now)
-      
+
       [ p, create_time ]
     end.sort_by do |r|
       r[1] || @time_source.now
     end
-    
+
     if (@retain_period)
       logs.reject! do |r|
         if (Time.now - r[1] > @retain_period)
@@ -222,23 +268,23 @@ protected
         end
       end
     end
-    
+
     if (@retain_count)
       # The logs array is sorted from oldest to newest, so retaining the N
       # newest entries entails stripping them off the end with pop.
 
       logs.pop(@retain_count)
-      
+
       FileUtils.rm_f(logs.collect { |r| r[0] })
     end
   end
-  
+
   def check_log_rotation!
     return unless (@rotation_time)
-    
+
     if (@time_source.now >= @rotation_time)
       self.log_open!
-      
+
       @rotation_time = self.next_rotation_time
     end
   end
@@ -246,23 +292,25 @@ protected
   def log_open!
     if (@path_format)
       @current_path = @time_source.now.strftime(@path_format)
-      
-      @log = File.open(@current_path, 'a', @file_open_options)
+
+      @log = File.open(@current_path, 'a', **@file_open_options)
       @log.sync = true
-      
-      if (File.exist?(@path) and File.symlink?(@path))
+
+      if (File.symlink?(@path))
         File.unlink(@path)
       end
 
       unless (File.exist?(@path))
         File.symlink(@current_path, @path)
       end
-      
+
       self.prune_logs!
     else
       @current_path = @path
-      
-      @log = File.open(@current_path, 'a', @file_open_options)
+
+      @log = File.open(@current_path, 'a', **@file_open_options)
+
+      @log.sync = true
     end
   end
 end

data/lib/birling/support.rb

@@ -1,4 +1,7 @@
 module Birling::Support
+  # == Module + Mixin Methods ===============================================
+
+  # Computes the beginning of the next day.
   def next_day(time, time_source = nil)
     (time_source || Time).local(
       time.year,
@@ -10,6 +13,7 @@ module Birling::Support
     ) + 1
   end
   
+  # Computes the beginning of the next hour.
   def next_hour(time, time_source = nil)
     seconds_left = time.to_i % 3600
     

data/test/helper.rb

@@ -54,9 +54,9 @@ class Test::Unit::TestCase
       name = "#{name}.log"
     end
     
-    @temp_path ||= File.expand_path('../tmp', File.dirname(__FILE__))
+    temp_path ||= File.expand_path('../tmp', File.dirname(__FILE__))
     
-    full_path = File.expand_path(name, @temp_path)
+    full_path = File.expand_path(name, temp_path)
     
     FileUtils::mkdir_p(File.dirname(full_path))
     
@@ -64,7 +64,7 @@ class Test::Unit::TestCase
       begin
         yield(full_path)
       ensure
-        remove_spec = File.expand_path('*', @temp_path)
+        remove_spec = File.expand_path('*', temp_path)
         
         FileUtils.rm_f(Dir.glob(remove_spec))
       end

data/test/test_birling.rb

@@ -35,17 +35,4 @@ class TestBirling < Test::Unit::TestCase
     
     assert !File.exist?(_path)
   end
-  
-  def test_time_warped
-    _now = Time.now
-    
-    Time::Warped.now = _now
-
-    assert_not_equal Time.now, Time.now
-    assert_equal _now, Time::Warped.now
-    
-    Time::Warped.now = nil
-    
-    assert_not_equal _now, Time::Warped.now
-  end
 end

data/test/test_birling_logger.rb

@@ -1,47 +1,77 @@
-require File.expand_path('helper', File.dirname(__FILE__))
+require File.expand_path('helper', __dir__)
+require 'stringio'
 
 class TestBirlingLogger < Test::Unit::TestCase
   def test_defaults
     temp_path do |path|
       log = Birling::Logger.new(path)
-    
+
       assert log
 
       assert log.opened?
       assert !log.closed?
       assert_equal 0, log.size
       assert Time.now >= log.create_time
-      
+
       assert_equal Birling::Formatter, log.formatter
       assert_equal Time, log.time_source
       assert_equal nil, log.period
     end
   end
-  
+
   def test_with_handle
     s = StringIO.new
-    
+
     log = Birling::Logger.new(s)
-    
+
     assert log.opened?
-    
+
     assert_equal 0, log.size
   end
-  
+
+  def test_io_compatible
+    stdout = $stdout
+
+    buffer = StringIO.new
+
+    log = Birling::Logger.new(buffer, time_source: Time::Warped)
+
+    assert log.opened?
+
+    assert_equal 0, log.size
+
+    $stdout = log
+
+    start = Time.parse('2017-10-10 12:00:00')
+
+    Time::Warped.now = start
+
+    puts "Test"
+
+    log.close
+
+    expected = "[2017-10-10 12:00:00] Test\n"
+
+    assert_equal expected, buffer.string.to_s
+
+  ensure
+    $stdout = stdout
+  end
+
   def test_formatter
     formatter_called = false
     formatter = lambda do |severity, time, program, message|
       formatter_called = true
       message
     end
-    
+
     output = StringIO.new
-    log = Birling::Logger.new(output, :formatter => formatter)
-    
+    log = Birling::Logger.new(output, formatter: formatter)
+
     log.debug("Test")
-    
+
     assert_equal true, formatter_called
-    
+
     output.rewind
     assert_equal "Test", output.read
   end
@@ -49,45 +79,44 @@ class TestBirlingLogger < Test::Unit::TestCase
   def test_default_level
     temp_path do |path|
       log = Birling::Logger.new(path)
-    
+
       assert log
 
       assert log.opened?
       assert !log.closed?
       assert_equal 0, log.size
-      
+
       assert log.debug?
 
       log.debug("Test")
-      
+
       current_size = log.size
       assert current_size > 0
     end
   end
-  
+
   def test_direct_write
     output = StringIO.new
-  
+
     log = Birling::Logger.new(output)
-    
+
     log << "TEST"
-    
-    output.rewind
-    assert_equal "TEST", output.read
+
+    assert_equal "TEST", output.string
   end
-  
+
   def test_level_filter
     output = StringIO.new
-  
+
     log = Birling::Logger.new(
       output,
-      :formatter => lambda { |s, t, p, m| "#{m}\n" },
-      :severity => :info
+      formatter: lambda { |s, t, p, m| "#{m}\n" },
+      severity: :info
     )
-    
+
     log.debug("DEBUG")
     log.info("INFO")
-    
+
     output.rewind
     assert_equal "INFO\n", output.read
   end
@@ -99,98 +128,100 @@ class TestBirlingLogger < Test::Unit::TestCase
       assert log.debug?
 
       log.debug("Test")
-      
+
       current_size = log.size
       assert current_size > 0
-      
+
       create_time = log.create_time
       assert create_time <= Time.now
-      
+
       log.close
-      
+
       log = Birling::Logger.new(path)
-      
+
       assert_equal current_size, log.size
       assert_equal create_time, log.create_time
     end
   end
-  
+
   def test_time_source
     temp_path do |path|
       frozen_time = Time.now
       Time::Warped.now = frozen_time
-      
-      logger = Birling::Logger.new(path, :time_source => Time::Warped)
-      
+
+      logger = Birling::Logger.new(path, time_source: Time::Warped)
+
       assert_equal frozen_time, logger.time_source.now
     end
   end
-  
+
   def test_cycling
     temp_path(:cycle) do |path|
       start = Time.now
       Time::Warped.now = start
-      logger = Birling::Logger.new(path, :period => 1, :time_source => Time::Warped)
-      
+      logger = Birling::Logger.new(path, period: 1, time_source: Time::Warped)
+
       assert_equal 1, logger.period
-      
+
       current_path = logger.current_path
       assert_equal '%s', logger.path_time_format
-    
+
       logger.debug("Test")
-      
+
       Time::Warped.now += 1
-      
+
       logger.debug("Test")
 
       assert_not_equal current_path, logger.current_path
-      
+
       current_path = logger.current_path
 
       Time::Warped.now += 1
-      
+
       logger.debug("Test")
-      
+
       assert_not_equal current_path, logger.current_path
     end
   end
-  
+
   def test_retain_count
     temp_path(:cycle) do |path|
       start = Time.now
       Time::Warped.now = start
-      
+
       retain_count = 10
 
       logger = Birling::Logger.new(
         path,
-        :period => 1,
-        :time_source => Time::Warped,
-        :retain_count => retain_count
+        period: 1,
+        time_source: Time::Warped,
+        retain_count: retain_count
       )
 
       (retain_count + 5).times do |n|
         logger.debug("Test")
+
         Time::Warped.now += 1
+
         logger.debug("Test")
       end
-      
+
       assert_equal retain_count, Dir.glob(logger.path_format % '*').length
     end
   end
-  
+
   def test_default_formatter
     temp_path(:cycle) do |path|
       logger = Birling::Logger.new(path)
-      
+
       lines = 100
-      
+
       lines.times do
         logger.debug("Test")
       end
-      
+
       logger.close
-      
+
       assert_equal lines, File.readlines(path).length
     end
   end
@@ -198,13 +229,13 @@ class TestBirlingLogger < Test::Unit::TestCase
   def test_retain_period
     temp_path(:cycle) do |path|
       retain_period = 3
-      
+
       logger = Birling::Logger.new(
         path,
-        :period => 1,
-        :retain_period => retain_period
+        period: 1,
+        retain_period: retain_period
       )
-      
+
       assert_equal true, File.exist?(path)
       assert_equal true, File.symlink?(path)
 
@@ -215,7 +246,7 @@ class TestBirlingLogger < Test::Unit::TestCase
         Time::Warped.now += 1
         logger.debug("Test")
       end
-      
+
       assert_equal retain_period + 1, Dir.glob(logger.path_format % '*').length
     end
   end
@@ -227,6 +258,6 @@ class TestBirlingLogger < Test::Unit::TestCase
       invalid = (0..255).to_a.pack('C*')
 
       logger.debug(invalid)
-    end    
+    end
   end
 end

data/test/test_birling_support.rb

@@ -4,6 +4,8 @@ class TestBirlingSupport < Test::Unit::TestCase
   def test_next_day_on_dst_flip
     in_time_zone('EST5EDT') do
       time = Time.new(2012, 11, 4)
+
+      assert_equal 'EDT', time.zone
       
       assert_equal time.day, (time + 86400).day
       
@@ -21,6 +23,8 @@ class TestBirlingSupport < Test::Unit::TestCase
   def test_hour_day_on_dst_flip
     in_time_zone('EST5EDT') do
       time = Time.new(2012, 11, 4, 0, 59, 59) + 1
+
+      assert_equal 'EDT', time.zone
       
       assert_equal time.hour, (time + 3600).hour
       
@@ -40,6 +44,8 @@ class TestBirlingSupport < Test::Unit::TestCase
   def test_next_day_at_year_end
     in_time_zone('EST5EDT') do
       time = Time.new(2012, 12, 31)
+
+      assert_equal 'EST', time.zone
       
       next_day = Birling::Support.next_day(time)
       

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: birling
 version: !ruby/object:Gem::Version
-  version: 0.1.3
+  version: 0.3.1
 platform: ruby
 authors:
 - Scott Tadman
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2015-01-31 00:00:00.000000000 Z
+date: 2021-03-30 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -24,6 +24,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: test-unit
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: jeweler
   requirement: !ruby/object:Gem::Requirement
@@ -48,6 +62,7 @@ extra_rdoc_files:
 - README.md
 files:
 - ".document"
+- ".travis.yml"
 - Gemfile
 - LICENSE.txt
 - README.md
@@ -62,11 +77,11 @@ files:
 - test/test_birling.rb
 - test/test_birling_logger.rb
 - test/test_birling_support.rb
-homepage: http://github.com/twg/birling
+homepage: http://github.com/postageapp/birling
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -81,9 +96,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.2.2
-signing_key: 
+rubygems_version: 3.2.3
+signing_key:
 specification_version: 4
 summary: Logger with simple log rotation system
 test_files: []