Fix AIX and QNX file support

lib/train/file.rb

@@ -4,13 +4,7 @@
 # author: Dominik Richter
 
 require 'train/file/local'
-require 'train/file/local/unix'
-require 'train/file/local/windows'
 require 'train/file/remote'
-require 'train/file/remote/unix'
-require 'train/file/remote/linux'
-require 'train/file/remote/windows'
-require 'train/file/remote/qnx'
 require 'digest/sha2'
 require 'digest/md5'
 require 'train/extras/stat'

lib/train/file/local.rb

@@ -68,3 +68,8 @@ def linked_to?(dst)
     end
   end
 end
+
+# subclass requires are loaded after Train::File::Local is defined
+# to avoid superclass mismatch errors
+require 'train/file/local/unix'
+require 'train/file/local/windows'

lib/train/file/remote.rb

@@ -26,3 +26,11 @@ def detect_filename(path, sep)
     end
   end
 end
+
+# subclass requires are loaded after Train::File::Remote is defined
+# to avoid superclass mismatch errors
+require 'train/file/remote/aix'
+require 'train/file/remote/linux'
+require 'train/file/remote/qnx'
+require 'train/file/remote/unix'
+require 'train/file/remote/windows'

lib/train/file/remote/qnx.rb

@@ -8,7 +8,7 @@
 module Train
   class File
     class Remote
-      class Aix < Train::File::Remote::Unix
+      class Qnx < Train::File::Remote::Unix
         def content
           cat = 'cat'
           cat = '/proc/boot/cat' if @backend.os[:release].to_i >= 7
@@ -32,7 +32,7 @@ def type
           mode owner group uid gid mtime size selinux_label link_path mounted stat
         }.each do |field|
           define_method field.to_sym do
-            fail NotImplementedError, "QNX does not implement the #{m}() method yet."
+            fail NotImplementedError, "QNX does not implement the #{field}() method yet."
           end
         end
       end

test/unit/file/remote/aix_test.rb

@@ -0,0 +1,26 @@
+require 'helper'
+require 'train/transports/local'
+require 'train/file/remote/aix'
+require 'train/transports/mock'
+
+describe Train::File::Remote::Aix do
+  let(:cls) { Train::File::Remote::Aix }
+  let(:backend) {
+    backend = Train::Transports::Mock.new.connection
+    backend.mock_os({ name: 'aix', family: 'unix' })
+    backend
+  }
+
+  it 'returns a nil link_path if the object is not a symlink' do
+    file = cls.new(backend, 'path')
+    file.stubs(:symlink?).returns(false)
+    file.link_path.must_be_nil
+  end
+
+  it 'returns a correct link_path' do
+    file = cls.new(backend, 'path')
+    file.stubs(:symlink?).returns(true)
+    backend.mock_command("perl -e 'print readlink shift' path", 'our_link_path')
+    file.link_path.must_equal 'our_link_path'
+  end
+end

test/unit/file/remote/qnx_test.rb

@@ -0,0 +1,44 @@
+require 'helper'
+require 'train/transports/local'
+require 'train/file/remote/qnx'
+require 'train/transports/mock'
+
+describe Train::File::Remote::Qnx do
+  let(:cls) { Train::File::Remote::Qnx }
+  let(:backend) {
+    backend = Train::Transports::Mock.new.connection
+    backend.mock_os({ name: 'qnx', family: 'unix' })
+    backend
+  }
+
+  it 'returns file contents when the file exists' do
+    out = rand.to_s
+    backend.mock_command('cat path', out)
+    file = cls.new(backend, 'path')
+    file.stubs(:exist?).returns(true)
+    file.content.must_equal out
+  end
+
+  it 'returns nil contents when the file does not exist' do
+    file = cls.new(backend, 'path')
+    file.stubs(:exist?).returns(false)
+    file.content.must_be_nil
+  end
+
+  it 'returns a file type' do
+    backend.mock_command('file path', 'blah directory blah')
+    cls.new(backend, 'path').type.must_equal :directory
+  end
+
+  it 'returns a directory type' do
+    backend.mock_command('file path', 'blah regular file blah')
+    cls.new(backend, 'path').type.must_equal :file
+  end
+
+  it 'raises exception for unimplemented methods' do
+    file = cls.new(backend, 'path')
+    %w{mode owner group uid gid mtime size selinux_label link_path mounted stat}.each do |m|
+      proc { file.send(m) }.must_raise NotImplementedError
+    end
+  end
+end