cosmos 4.0.0-universal-java-1.8

data/spec/streams/serial_stream_spec.rb

@@ -0,0 +1,122 @@
+# encoding: ascii-8bit
+
+# Copyright 2014 Ball Aerospace & Technologies Corp.
+# All Rights Reserved.
+#
+# This program is free software; you can modify and/or redistribute it
+# under the terms of the GNU General Public License
+# as published by the Free Software Foundation; version 3 with
+# attribution addendums as found in the LICENSE.txt
+
+require 'spec_helper'
+require 'cosmos/streams/serial_stream'
+
+module Cosmos
+
+  if RUBY_ENGINE == 'ruby' or Gem.win_platform?
+
+    describe SerialStream do
+      describe "initialize" do
+        it "complains if neither a read or write port given" do
+          expect { SerialStream.new(nil,nil,9600,:EVEN,1,nil,nil) }.to raise_error("Either a write port or read port must be given")
+        end
+      end
+
+      describe "connected?" do
+        it "is connected when initialized" do
+          driver = double("driver")
+          expect(SerialDriver).to receive(:new).and_return(driver)
+          ss = SerialStream.new('COM1',nil,9600,:EVEN,1,nil,nil)
+          expect(ss.connected?).to be true
+        end
+      end
+
+      describe "read" do
+        it "raises an error if no read port given" do
+          driver = double("driver")
+          expect(SerialDriver).to receive(:new).and_return(driver)
+          ss = SerialStream.new('COM1',nil,9600,:EVEN,1,nil,nil)
+          expect { ss.read }.to raise_error("Attempt to read from write only stream")
+        end
+
+        it "calls read from the driver" do
+          driver = double("driver")
+          expect(driver).to receive(:read).and_return 'test'
+          expect(SerialDriver).to receive(:new).and_return(driver)
+          ss = SerialStream.new('COM1','COM1',9600,:EVEN,1,nil,nil)
+          expect(ss.read).to eql 'test'
+        end
+      end
+
+      describe "write" do
+        it "raises an error if no write port given" do
+          driver = double("driver")
+          expect(SerialDriver).to receive(:new).and_return(driver)
+          ss = SerialStream.new(nil,'COM1',9600,:EVEN,1,nil,nil)
+          expect { ss.write('') }.to raise_error("Attempt to write to read only stream")
+        end
+
+        it "calls write from the driver" do
+          driver = double("driver")
+          expect(driver).to receive(:write).with('test')
+          expect(SerialDriver).to receive(:new).and_return(driver)
+          ss = SerialStream.new('COM1','COM1',9600,:EVEN,1,nil,nil)
+          ss.write('test')
+        end
+      end
+
+      describe "disconnect" do
+        it "closes the write driver" do
+          driver = double("driver")
+          expect(driver).to receive(:closed?).and_return(false)
+          expect(driver).to receive(:close)
+          expect(SerialDriver).to receive(:new).and_return(driver)
+          ss = SerialStream.new('COM1',nil,9600,:EVEN,1,nil,nil)
+          expect(ss.connected?).to be true
+          ss.disconnect
+          expect(ss.connected?).to be false
+        end
+
+        it "closes the read driver" do
+          driver = double("driver")
+          expect(driver).to receive(:closed?).and_return(false)
+          expect(driver).to receive(:close)
+          expect(SerialDriver).to receive(:new).and_return(driver)
+          ss = SerialStream.new(nil,'COM1',9600,:EVEN,1,nil,nil)
+          expect(ss.connected?).to be true
+          ss.disconnect
+          expect(ss.connected?).to be false
+        end
+
+        it "does not close the driver twice" do
+          driver = double("driver")
+          expect(driver).to receive(:closed?).and_return(false, true)
+          expect(driver).to receive(:close).once
+          expect(SerialDriver).to receive(:new).and_return(driver)
+          ss = SerialStream.new('COM1','COM1',9600,:EVEN,1,nil,nil)
+          expect(ss.connected?).to be true
+          ss.disconnect
+          expect(ss.connected?).to be false
+          ss.disconnect
+          expect(ss.connected?).to be false
+        end
+      end
+
+      describe "connect" do
+        it "supports a connect method that does nothing" do
+          driver = double("driver")
+          expect(driver).to receive(:closed?).and_return(false)
+          expect(driver).to receive(:close).once
+          expect(SerialDriver).to receive(:new).and_return(driver)
+          ss = SerialStream.new(nil,'COM1',9600,:EVEN,1,nil,nil)
+          expect{ss.connect}.to_not raise_error
+          ss.disconnect
+        end
+      end
+
+    end
+
+  end
+
+end
+

data/spec/streams/stream_spec.rb

@@ -0,0 +1,30 @@
+# encoding: ascii-8bit
+
+# Copyright 2014 Ball Aerospace & Technologies Corp.
+# All Rights Reserved.
+#
+# This program is free software; you can modify and/or redistribute it
+# under the terms of the GNU General Public License
+# as published by the Free Software Foundation; version 3 with
+# attribution addendums as found in the LICENSE.txt
+
+require 'spec_helper'
+require 'cosmos/streams/stream'
+
+module Cosmos
+
+  describe Stream do
+
+    describe "read, write, connected?, disconnect" do
+      it "raises an error" do
+        expect { Stream.new.read       }.to raise_error(/not defined/)
+        expect { Stream.new.write(nil) }.to raise_error(/not defined/)
+        expect { Stream.new.connect }.to raise_error(/not defined/)
+        expect { Stream.new.connected? }.to raise_error(/not defined/)
+        expect { Stream.new.disconnect }.to raise_error(/not defined/)
+      end
+    end
+
+  end
+end
+

data/spec/streams/tcpip_client_stream_spec.rb

@@ -0,0 +1,65 @@
+# encoding: ascii-8bit
+
+# Copyright 2014 Ball Aerospace & Technologies Corp.
+# All Rights Reserved.
+#
+# This program is free software; you can modify and/or redistribute it
+# under the terms of the GNU General Public License
+# as published by the Free Software Foundation; version 3 with
+# attribution addendums as found in the LICENSE.txt
+
+require 'spec_helper'
+require 'cosmos/streams/tcpip_client_stream'
+
+module Cosmos
+
+  describe TcpipClientStream do
+    before(:all) do
+      addr = Socket.pack_sockaddr_in(8888, "0.0.0.0")
+      if RUBY_ENGINE == 'ruby'
+        @listen_socket = Socket.new(Socket::AF_INET, Socket::SOCK_STREAM, 0)
+      else
+        @listen_socket = ServerSocket.new(Socket::AF_INET, Socket::SOCK_STREAM, 0)
+      end
+      @listen_socket.setsockopt(Socket::SOL_SOCKET, Socket::SO_REUSEADDR, 1) unless Kernel.is_windows?
+      if RUBY_ENGINE == 'ruby'
+        @listen_socket.bind(addr)
+        @listen_socket.listen(5)
+      else
+        @listen_socket.bind(addr, 5)
+      end
+    end
+
+    after(:all) do
+      @listen_socket.close
+    end
+
+    describe "initialize" do
+      it "complains if the host is bad" do
+        expect { TcpipClientStream.new('asdf',8888,8888,nil,nil) }.to raise_error(/Invalid hostname/)
+      end
+
+      it "uses the same socket if read_port == write_port" do
+        ss = TcpipClientStream.new('localhost',8888,8888,nil,nil)
+        ss.connect
+        expect(ss.connected?).to be true
+        ss.disconnect
+      end
+
+      it "creates the write socket" do
+        ss = TcpipClientStream.new('localhost',8888,nil,nil,nil)
+        ss.connect
+        expect(ss.connected?).to be true
+        ss.disconnect
+      end
+
+      it "creates the read socket" do
+        ss = TcpipClientStream.new('localhost',nil,8888,nil,nil)
+        ss.connect
+        expect(ss.connected?).to be true
+        ss.disconnect
+      end
+    end
+  end
+end
+

data/spec/streams/tcpip_socket_stream_spec.rb

@@ -0,0 +1,226 @@
+# encoding: ascii-8bit
+
+# Copyright 2014 Ball Aerospace & Technologies Corp.
+# All Rights Reserved.
+#
+# This program is free software; you can modify and/or redistribute it
+# under the terms of the GNU General Public License
+# as published by the Free Software Foundation; version 3 with
+# attribution addendums as found in the LICENSE.txt
+
+require 'spec_helper'
+require 'cosmos/streams/tcpip_socket_stream'
+require 'socket'
+
+module Cosmos
+
+  if RUBY_ENGINE == 'ruby' # For some reason this test causes uncatchable exceptions in Jruby when run with other tests - Skip for now
+
+  describe TcpipSocketStream do
+    describe "initialize, connected?" do
+      it "is not be connected when initialized" do
+        ss = TcpipSocketStream.new(nil,nil,nil,nil)
+        expect(ss.connected?).to be false
+      end
+    end
+
+    # This test takes time and doesn't actually assert any functionality
+    # It was created to verify the changes to socket reads in PR#181.
+    #describe "read benchmark" do
+    #  it "determines how fast the read is" do
+    #    server = TCPServer.new(2000) # Server bound to port 2000
+    #    thread = Thread.new do
+    #      client = server.accept    # Wait for a client to connect
+    #      i = 0
+    #      100000.times do
+    #        client.write "test"
+    #        i += 1
+    #        sleep 0.001 if i % 100 == 1
+    #      end
+    #      client.close
+    #    end
+    #    socket = TCPSocket.new('localhost', 2000)
+    #    ss = TcpipSocketStream.new(nil,socket,nil,nil)
+    #    bytes = 0
+    #    while bytes < 400000
+    #      bytes += ss.read.length
+    #    end
+    #    Cosmos.close_socket(socket)
+    #    Cosmos.close_socket(server)
+    #    thread.kill
+    #    sleep 0.1
+    #  end
+    #end
+
+    describe "read" do
+      it "raises an error if no read socket given" do
+        ss = TcpipSocketStream.new('write',nil,nil,nil)
+        ss.connect
+        expect { ss.read }.to raise_error("Attempt to read from write only stream")
+      end
+
+      it "calls read_nonblock from the socket" do
+        server = TCPServer.new(2000) # Server bound to port 2000
+        thread = Thread.new do
+          loop do
+            client = server.accept    # Wait for a client to connect
+            sleep 0.1
+            client.write "test"
+            client.close
+          end
+        end
+        socket = TCPSocket.new('localhost', 2000)
+        ss = TcpipSocketStream.new(nil,socket,nil,nil)
+        expect(ss.read_nonblock).to eql ''
+        sleep 0.2
+        expect(ss.read_nonblock).to eql 'test'
+        Cosmos.close_socket(socket)
+        Cosmos.close_socket(server)
+        thread.kill
+        sleep 0.1
+      end
+
+      it "handles socket blocking exceptions" do
+        server = TCPServer.new(2000) # Server bound to port 2000
+        thread = Thread.new do
+          loop do
+            client = server.accept    # Wait for a client to connect
+            sleep 0.2
+            client.write "test"
+            client.close
+          end
+        end
+        socket = TCPSocket.new('localhost', 2000)
+        ss = TcpipSocketStream.new(nil,socket,nil,nil)
+        expect(ss.read).to eql 'test'
+        Cosmos.close_socket(socket)
+        Cosmos.close_socket(server)
+        thread.kill
+        sleep 0.1
+      end
+
+      it "handles socket timeouts" do
+        server = TCPServer.new(2000) # Server bound to port 2000
+        thread = Thread.new do
+          loop do
+            client = server.accept    # Wait for a client to connect
+            sleep 0.2
+            client.close
+          end
+        end
+        socket = TCPSocket.new('localhost', 2000)
+        ss = TcpipSocketStream.new(nil,socket,nil,0.1)
+        expect { ss.read }.to raise_error(Timeout::Error)
+        sleep 0.2
+        Cosmos.close_socket(socket)
+        Cosmos.close_socket(server)
+        thread.kill
+        sleep 0.1
+      end
+
+      it "handles socket connection reset exceptions" do
+        server = TCPServer.new(2000) # Server bound to port 2000
+        thread = Thread.new do
+          loop do
+            client = server.accept    # Wait for a client to connect
+            client.close
+          end
+        end
+        socket = TCPSocket.new('localhost', 2000)
+        ss = TcpipSocketStream.new(nil,socket,nil,5)
+        # Close the socket before trying to read from it
+        Cosmos.close_socket(socket)
+        expect(ss.read).to eql ''
+        Cosmos.close_socket(server)
+        thread.kill
+        sleep 0.1
+      end
+    end
+
+    describe "write" do
+      it "raises an error if no write port given" do
+        ss = TcpipSocketStream.new(nil,'read',nil,nil)
+        ss.connect
+        expect { ss.write('test') }.to raise_error("Attempt to write to read only stream")
+      end
+
+      it "calls write from the driver" do
+        write = double("write_socket")
+        # Simulate only writing two bytes at a time
+        expect(write).to receive(:write_nonblock).twice.and_return(2)
+        ss = TcpipSocketStream.new(write,nil,nil,nil)
+        ss.connect
+        ss.write('test')
+      end
+
+      it "handles socket blocking exceptions" do
+        write = double("write_socket")
+        allow(write).to receive(:write_nonblock) do
+          case $index
+          when 1
+            $index += 1
+            raise Errno::EWOULDBLOCK
+          when 2
+            4
+          end
+        end
+        expect(IO).to receive(:select).at_least(:once).and_return([])
+        $index = 1
+        ss = TcpipSocketStream.new(write,nil,nil,nil)
+        ss.connect
+        ss.write('test')
+      end
+
+      it "handles socket timeouts" do
+        write = double("write_socket")
+        allow(write).to receive(:write_nonblock).and_raise(Errno::EWOULDBLOCK)
+        expect(IO).to receive(:select).at_least(:once).and_return(nil)
+        ss = TcpipSocketStream.new(write,nil,nil,nil)
+        ss.connect
+        expect { ss.write('test') }.to raise_error(Timeout::Error)
+      end
+    end
+
+    describe "disconnect" do
+      it "closes the write socket" do
+        write = double("write_socket")
+        expect(write).to receive(:closed?).and_return(false)
+        expect(write).to receive(:close)
+        ss = TcpipSocketStream.new(write,nil,nil,nil)
+        ss.connect
+        expect(ss.connected?).to be true
+        ss.disconnect
+        expect(ss.connected?).to be false
+      end
+
+      it "closes the read socket" do
+        read = double("read_socket")
+        expect(read).to receive(:closed?).and_return(false)
+        expect(read).to receive(:close)
+        ss = TcpipSocketStream.new(nil,read,nil,nil)
+        ss.connect
+        expect(ss.connected?).to be true
+        ss.disconnect
+        expect(ss.connected?).to be false
+      end
+
+      it "does not close the socket twice" do
+        socket = double("socket")
+        expect(socket).to receive(:closed?).and_return(false, true, true, true)
+        expect(socket).to receive(:close).once
+        ss = TcpipSocketStream.new(socket,socket,nil,nil)
+        ss.connect
+        expect(ss.connected?).to be true
+        ss.disconnect
+        expect(ss.connected?).to be false
+        ss.disconnect
+        expect(ss.connected?).to be false
+      end
+    end
+
+  end
+
+  end # TEMP for Jruby
+
+end
+

data/spec/system/system_spec.rb

@@ -0,0 +1,699 @@
+# encoding: ascii-8bit
+
+# Copyright 2014 Ball Aerospace & Technologies Corp.
+# All Rights Reserved.
+#
+# This program is free software; you can modify and/or redistribute it
+# under the terms of the GNU General Public License
+# as published by the Free Software Foundation; version 3 with
+# attribution addendums as found in the LICENSE.txt
+
+require 'spec_helper'
+require 'cosmos'
+require 'cosmos/system/system'
+require 'tempfile'
+
+module Cosmos
+
+  describe System do
+    before(:all) do
+      clean_config()
+      System.class_eval('@@instance = nil')
+
+      # Save system.txt
+      @config_file = File.join(Cosmos::USERPATH,'config','system','system.txt')
+      FileUtils.mv @config_file, Cosmos::USERPATH
+
+      # Create a dummy system.txt
+      File.open(@config_file,'w') {|file| file.puts "# This is a comment" }
+      @config_targets = File.join(Cosmos::USERPATH,'config','targets')
+    end
+
+    after(:all) do
+      # Restore system.txt
+      FileUtils.mv File.join(Cosmos::USERPATH, 'system.txt'),
+        File.join(Cosmos::USERPATH,'config','system')
+    end
+
+    after(:each) do
+      clean_config()
+      System.class_eval('@@instance = nil')
+    end
+
+    describe "instance" do
+      it "creates default ports" do
+        # Don't check the actual port numbers but just that they exist
+        expect(System.ports.keys).to eql %w(CTS_API TLMVIEWER_API CTS_PREIDENTIFIED CTS_CMD_ROUTER)
+      end
+
+      it "creates default paths" do
+        # Don't check the actual paths but just that they exist
+        expect(System.paths.keys).to eql %w(LOGS TMP SAVED_CONFIG TABLES HANDBOOKS PROCEDURES SEQUENCES)
+      end
+    end
+
+    describe "System.commands" do
+      it "is just SYSTEM" do
+        expect(System.commands.target_names).to eql ['SYSTEM']
+      end
+
+      it "logs errors saving the configuration" do
+       # Write the system.txt file to auto declare targets
+       File.open(@config_file, 'w') { |file| file.puts "AUTO_DECLARE_TARGETS" }
+       # Force a reload of the configuration
+       System.class_eval('@@instance = nil')
+       capture_io do |stdout|
+         allow(FileUtils).to receive(:cp_r) { raise "Error" }
+         System.commands
+         expect(stdout.string).to match "Problem saving configuration"
+       end
+       File.open(@config_file, 'w') { |file| file.puts "# Comment" }
+      end
+    end
+
+    describe "System.telemetry" do
+      it "is just SYSTEM" do
+        expect(System.telemetry.target_names).to eql ['SYSTEM']
+      end
+    end
+
+    describe "System.limits" do
+      it "includes the DEFAULT limit set" do
+        expect(System.limits.sets).to include :DEFAULT
+      end
+    end
+
+    context "with valid targets" do
+      before(:all) do
+        # Write the system.txt file to auto declare targets
+        File.open(@config_file,'w') { |file| file.puts "AUTO_DECLARE_TARGETS" }
+      end
+
+      describe "System.clear_counters" do
+        it "clears the target, command and telemetry counters" do
+          expect(System.targets.length).to be > 0
+          System.targets.each do |name, tgt|
+            tgt.cmd_cnt = 100
+            tgt.tlm_cnt = 100
+          end
+          System.commands.all do |tgt, pkt|
+            pkt.received_count = 100
+          end
+          System.telemetry.all do |tgt, pkt|
+            pkt.received_count = 100
+          end
+
+          System.clear_counters
+
+          System.targets.each do |name, tgt|
+            expect(tgt.cmd_cnt).to eql 0
+            expect(tgt.tlm_cnt).to eql 0
+          end
+          System.commands.all do |tgt, pkt|
+            expect(pkt.received_count).to eql 0
+          end
+          System.telemetry.all do |tgt, pkt|
+            expect(pkt.received_count).to eql 0
+          end
+        end
+      end
+
+      describe "packets and System.packets" do
+        it "calculates MD5s across all the target files" do
+          capture_io do |stdout|
+            # This line actually does the work of reading the configuration
+            expect(System.telemetry.target_names).to eql ['INST', 'SYSTEM']
+            expect(System.commands.target_names).to eql ['INST', 'SYSTEM']
+
+            expect(stdout.string).to match "Marshal file does not exist"
+
+            # Reset stdout for another go at the processing
+            stdout.rewind
+            # Reset the instance variable so it will read the new configuration
+            System.class_eval('@@instance = nil')
+            # This line actually does the work of reading the configuration
+            expect(System.telemetry.target_names).to eql ['INST', 'SYSTEM']
+            expect(System.commands.target_names).to eql ['INST', 'SYSTEM']
+
+            expect(stdout.string).to match "Marshal load success"
+          end
+        end
+
+        it "handles target parsing errors" do
+          capture_io do |stdout|
+            allow_any_instance_of(PacketConfig).to receive(:process_file) { raise "ProcessError" }
+            # This line actually does the work of reading the configuration
+            expect { System.telemetry.target_names }.to raise_error("ProcessError")
+
+            expect(stdout.string).to match "Problem processing"
+          end
+        end
+      end
+
+      describe "load_configuration" do
+        after(:all) do
+          File.delete(File.join(@config_targets,'SYSTEM','cmd_tlm','test1_tlm.txt'))
+          File.delete(File.join(@config_targets,'SYSTEM','cmd_tlm','test2_tlm.txt'))
+        end
+
+        it "loads the initial configuration" do
+          System.load_configuration
+          expect(System.commands.target_names).to eql ['INST', 'SYSTEM']
+          expect(System.telemetry.target_names).to eql ['INST', 'SYSTEM']
+        end
+
+        it "loads a named configuration" do
+          File.open(@config_file,'w') do |file|
+            file.puts "DECLARE_TARGET INST OVERRIDE"
+            file.puts "DECLARE_TARGET SYSTEM"
+          end
+
+          # Load the original configuration
+          original_config_name, err = System.load_configuration
+          expect(err).to eql nil
+          expect(System.telemetry.target_names).to eql %w(OVERRIDE SYSTEM)
+          original_pkts = System.telemetry.packets('SYSTEM').keys
+
+          # Create a new configuration by writing another telemetry file
+          File.open(File.join(@config_targets,'SYSTEM','cmd_tlm','test1_tlm.txt'),'w') do |file|
+            file.puts "TELEMETRY SYSTEM TEST1 BIG_ENDIAN"
+            file.puts "  APPEND_ITEM DATA 240 STRING"
+          end
+          System.instance.process_file(@config_file)
+          # Verify the new telemetry packet is there
+          expect(System.telemetry.packets('SYSTEM').keys).to include "TEST1"
+          second_config_name = System.configuration_name
+
+          # Now load the original configuration
+          name, err = System.load_configuration(original_config_name)
+          expect(err).to eql nil
+          expect(original_config_name).to eql name
+          expect(System.telemetry.packets('SYSTEM').keys).not_to include "TEST1"
+
+          # Create yet another configuration by writing another telemetry file
+          File.open(File.join(@config_targets,'SYSTEM','cmd_tlm','test2_tlm.txt'),'w') do |file|
+            file.puts "TELEMETRY SYSTEM TEST2 BIG_ENDIAN"
+            file.puts "  APPEND_ITEM DATA 240 STRING"
+          end
+          System.instance.process_file(@config_file)
+          names = []
+          # Verify the new telemetry packet is there as well as the second one
+          expect(System.telemetry.packets('SYSTEM').keys).to include("TEST1", "TEST2")
+          third_config_name = System.configuration_name
+
+          # Try loading something that doesn't exist
+          # It should fail and reload the original configuration
+          name, err = System.load_configuration("BLAH")
+          expect(err).to eql nil
+          expect(name).to eql original_config_name
+
+          # Now load the second configuration. It shouldn't have the most
+          # recently defined telemetry packet.
+          System.load_configuration(second_config_name)
+          expect(System.telemetry.packets('SYSTEM').keys).to include "TEST1"
+          expect(System.telemetry.packets('SYSTEM').keys).not_to include "TEST2"
+
+          # Now remove system.txt from the third configuration and try to load it again to cause an error
+          third_config_path = System.instance.send(:find_configuration, third_config_name)
+          FileUtils.mv File.join(third_config_path, 'system.txt'), File.join(third_config_path, 'system2.txt')
+          result, err = System.load_configuration(third_config_name)
+          expect(result).to eql original_config_name
+          expect(err).to_not be_nil
+        end
+      end
+    end
+
+    describe "process_file" do
+      before(:all) do
+        begin
+          # Move the targets directory out of the way so we can make our own
+          FileUtils.mv @config_targets, Cosmos::USERPATH
+          FileUtils.mkdir_p(@config_targets)
+          File.open(@config_file,'w') { |file| file.puts "AUTO_DECLARE_TARGETS" }
+        rescue
+          puts "Cannot move targets folder... probably due to open editor or explorer"
+          exit 1
+        end
+      end
+      after(:all) do
+        # Restore the targets directory
+        FileUtils.rm_rf @config_targets
+        FileUtils.mv File.join(Cosmos::USERPATH, 'targets'), File.join(Cosmos::USERPATH, 'config')
+      end
+
+      it "complains about unknown keywords" do
+        tf = Tempfile.new('unittest')
+        tf.puts("BLAH")
+        tf.close
+        expect { System.instance.process_file(tf.path) }.to raise_error(ConfigParser::Error, /Unknown keyword 'BLAH'/)
+        tf.unlink
+      end
+
+      context "with AUTO_DECLARE_TARGETS" do
+        it "takes 0 parameters" do
+          tf = Tempfile.new('unittest')
+          tf.puts("AUTO_DECLARE_TARGETS TRUE")
+          tf.close
+          expect { System.instance.process_file(tf.path) }.to raise_error(ConfigParser::Error, /Too many parameters for AUTO_DECLARE_TARGETS./)
+          tf.unlink
+        end
+
+        it "complains if config/targets doesn't exist" do
+          tf = Tempfile.new('unittest')
+          tf.puts("AUTO_DECLARE_TARGETS")
+          tf.close
+          FileUtils.rm_rf(@config_targets)
+          expect { System.instance.process_file(tf.path) }.to raise_error(ConfigParser::Error, /\/config\/targets must exist/)
+          FileUtils.mkdir_p(@config_targets)
+          tf.unlink
+        end
+
+        it "complains if target directories aren't uppercase" do
+          tf = Tempfile.new('unittest')
+          tf.puts("AUTO_DECLARE_TARGETS")
+          tf.close
+          FileUtils.mkdir_p(File.join(@config_targets, 'tgt'))
+          expect { System.instance.process_file(tf.path) }.to raise_error(ConfigParser::Error, /Target folder must be uppercase: 'tgt'/)
+          Dir.rmdir(File.join(@config_targets, 'tgt'))
+          tf.unlink
+        end
+
+        it "processes target directories with the SYSTEM directory last" do
+          tf = Tempfile.new('unittest')
+          tf.puts("AUTO_DECLARE_TARGETS")
+          tf.close
+          FileUtils.mkdir_p(File.join(@config_targets, 'ABC'))
+          FileUtils.mkdir_p(File.join(@config_targets, 'SYSTEM'))
+          FileUtils.mkdir_p(File.join(@config_targets, 'XYZ'))
+          System.instance.process_file(tf.path)
+          # Since Ruby 1.9+ uses ordered Hashes we can ask for the keys and
+          # SYSTEM should be last
+          expect(System.instance.targets.keys).to eql %w(ABC XYZ SYSTEM)
+          Dir.rmdir(File.join(@config_targets, 'ABC'))
+          Dir.rmdir(File.join(@config_targets, 'SYSTEM'))
+          Dir.rmdir(File.join(@config_targets, 'XYZ'))
+          tf.unlink
+        end
+
+        it "ignores previous DECLARE_TARGET directories" do
+          tf = Tempfile.new('unittest')
+          tf.puts("DECLARE_TARGET ABC CBA")
+          tf.puts("AUTO_DECLARE_TARGETS")
+          tf.close
+          FileUtils.mkdir_p(File.join(@config_targets, 'ABC'))
+          FileUtils.mkdir_p(File.join(@config_targets, 'SYSTEM'))
+          FileUtils.mkdir_p(File.join(@config_targets, 'XYZ'))
+          System.instance.process_file(tf.path)
+          # Since Ruby 1.9+ uses ordered Hashes we can ask for the keys and
+          # SYSTEM should be last
+          expect(System.instance.targets.keys).to eql %w(CBA XYZ SYSTEM)
+          Dir.rmdir(File.join(@config_targets, 'ABC'))
+          Dir.rmdir(File.join(@config_targets, 'SYSTEM'))
+          Dir.rmdir(File.join(@config_targets, 'XYZ'))
+          tf.unlink
+        end
+      end
+
+      context "with DECLARE_TARGET" do
+        it "takes 1 or 2 parameters" do
+          tf = Tempfile.new('unittest')
+          tf.puts("DECLARE_TARGET")
+          tf.close
+          expect { System.instance.process_file(tf.path) }.to raise_error(ConfigParser::Error, /Not enough parameters for DECLARE_TARGET./)
+          tf.unlink
+
+          tf = Tempfile.new('unittest')
+          tf.puts("DECLARE_TARGET TGT TGT TGT TGT")
+          tf.close
+          expect { System.instance.process_file(tf.path) }.to raise_error(ConfigParser::Error, /Too many parameters for DECLARE_TARGET./)
+          tf.unlink
+        end
+
+        it "complains if the target directory doesn't exist" do
+          tf = Tempfile.new('unittest')
+          tf.puts("DECLARE_TARGET TGT")
+          tf.close
+          expect { System.instance.process_file(tf.path) }.to raise_error(ConfigParser::Error, /Target folder must exist/)
+          tf.unlink
+        end
+
+        it "processes the target directory" do
+          tf = Tempfile.new('unittest')
+          tf.puts("DECLARE_TARGET TGT")
+          tf.close
+          FileUtils.mkdir_p(File.join(@config_targets, 'TGT'))
+          System.instance.process_file(tf.path)
+          tf.unlink
+        end
+
+        it "processes the target directory with substitute name" do
+          tf = Tempfile.new('unittest')
+          tf.puts("DECLARE_TARGET TGT NEW")
+          tf.close
+          FileUtils.mkdir_p(File.join(@config_targets, 'TGT'))
+          System.instance.process_file(tf.path)
+          tf.unlink
+        end
+
+        it "processes the target directory with specified target.txt" do
+          tf = Tempfile.new('unittest')
+          tf.puts("DECLARE_TARGET TGT nil target.txt")
+          tf.close
+          FileUtils.mkdir_p(File.join(@config_targets, 'TGT'))
+          File.open(File.join(@config_targets, 'TGT', 'target.txt'), 'w') do |file|
+            file.puts "# target.txt"
+          end
+          System.instance.process_file(tf.path)
+          tf.unlink
+        end
+      end
+
+      context "with PORT" do
+        it "takes 2 parameters" do
+          tf = Tempfile.new('unittest')
+          tf.puts("PORT CTS_API")
+          tf.close
+          expect { System.instance.process_file(tf.path) }.to raise_error(ConfigParser::Error, /Not enough parameters for PORT./)
+          tf.unlink
+
+          tf = Tempfile.new('unittest')
+          tf.puts("PORT CTS_API 8888 TRUE")
+          tf.close
+          expect { System.instance.process_file(tf.path) }.to raise_error(ConfigParser::Error, /Too many parameters for PORT./)
+          tf.unlink
+        end
+
+        it "complains about unknown ports" do
+          tf = Tempfile.new('unittest')
+          tf.puts("PORT MYPORT 10")
+          tf.close
+          capture_io do |stdout|
+            System.instance.process_file(tf.path)
+            expect(stdout.string).to match /WARN: Unknown port name given: MYPORT/
+          end
+          tf.unlink
+        end
+
+        it "changes known ports" do
+          tf = Tempfile.new('unittest')
+          tf.puts("PORT CTS_API 8888")
+          tf.close
+          expect(System.ports['CTS_API']).to eql 7777
+          System.instance.process_file(tf.path)
+          expect(System.ports['CTS_API']).to eql 8888
+          tf.unlink
+        end
+      end
+
+      context "with PATH" do
+        it "takes 2 parameters" do
+          tf = Tempfile.new('unittest')
+          tf.puts("PATH C:/")
+          tf.close
+          expect { System.instance.process_file(tf.path) }.to raise_error(ConfigParser::Error, /Not enough parameters for PATH./)
+          tf.unlink
+
+          tf = Tempfile.new('unittest')
+          tf.puts("PATH MYPATH C:/ TRUE")
+          tf.close
+          expect { System.instance.process_file(tf.path) }.to raise_error(ConfigParser::Error, /Too many parameters for PATH./)
+          tf.unlink
+        end
+
+        it "complains about unknown paths" do
+          tf = Tempfile.new('unittest')
+          tf.puts("PATH MYPATH C:/")
+          tf.close
+          capture_io do |stdout|
+            System.instance.process_file(tf.path)
+            expect(stdout.string).to match /WARN: Unknown path name given: MYPATH/
+          end
+          tf.unlink
+        end
+
+        it "changes known paths" do
+          if Kernel.is_windows?
+            tf = Tempfile.new('unittest')
+            tf.puts("PATH LOGS C:/mylogs")
+            tf.close
+            expect(System.paths['LOGS']).to match 'outputs/logs'
+            System.instance.process_file(tf.path)
+            expect(System.paths['LOGS']).to eql 'C:/mylogs'
+            tf.unlink
+          end
+        end
+      end
+
+      context "with DEFAULT_PACKET_LOG_WRITER" do
+        it "takes 1 or more parameters" do
+          tf = Tempfile.new('unittest')
+          tf.puts("DEFAULT_PACKET_LOG_WRITER")
+          tf.close
+          expect { System.instance.process_file(tf.path) }.to raise_error(ConfigParser::Error, /Not enough parameters for DEFAULT_PACKET_LOG_WRITER./)
+          tf.unlink
+
+          tf = Tempfile.new('unittest')
+          tf.puts("DEFAULT_PACKET_LOG_WRITER packet_log_writer.rb nil false")
+          tf.close
+          System.instance.process_file(tf.path)
+          expect(System.default_packet_log_writer).to eql PacketLogWriter
+          expect(System.default_packet_log_writer_params).to eql ["nil", "false"]
+          tf.unlink
+        end
+
+        it "complains if the class doesn't exist" do
+          tf = Tempfile.new('unittest')
+          tf.puts("DEFAULT_PACKET_LOG_WRITER my_nonexistent_class")
+          tf.close
+          expect { System.instance.process_file(tf.path) }.to raise_error(/Unable to require my_nonexistent_class/)
+          tf.unlink
+        end
+
+        it "sets the packet writer" do
+          filename = File.join(File.dirname(__FILE__),'..','..','lib','my_writer.rb')
+          File.open(filename, 'w') do |file|
+            file.puts "class MyWriter"
+            file.puts "end"
+          end
+          tf = Tempfile.new('unittest')
+          tf.puts("DEFAULT_PACKET_LOG_WRITER my_writer")
+          tf.close
+          System.instance.process_file(tf.path)
+          expect(System.default_packet_log_writer).to eql MyWriter
+          File.delete filename
+          tf.unlink
+        end
+      end
+
+      context "with DEFAULT_PACKET_LOG_READER" do
+        it "takes 1 or more parameters" do
+          tf = Tempfile.new('unittest')
+          tf.puts("DEFAULT_PACKET_LOG_READER")
+          tf.close
+          expect { System.instance.process_file(tf.path) }.to raise_error(ConfigParser::Error, /Not enough parameters for DEFAULT_PACKET_LOG_READER./)
+          tf.unlink
+
+          tf = Tempfile.new('unittest')
+          tf.puts("DEFAULT_PACKET_LOG_READER packet_log_reader.rb nil false")
+          tf.close
+          System.instance.process_file(tf.path)
+          expect(System.default_packet_log_reader).to eql PacketLogReader
+          expect(System.default_packet_log_reader_params).to eql ["nil", "false"]
+          tf.unlink
+        end
+
+        it "complains if the class doesn't exist" do
+          tf = Tempfile.new('unittest')
+          tf.puts("DEFAULT_PACKET_LOG_READER my_nonexistent_class")
+          tf.close
+          expect { System.instance.process_file(tf.path) }.to raise_error(/Unable to require my_nonexistent_class/)
+          tf.unlink
+        end
+
+        it "sets the packet reader" do
+          filename = File.join(File.dirname(__FILE__),'..','..','lib','my_reader.rb')
+          File.open(filename, 'w') do |file|
+            file.puts "class MyReader"
+            file.puts "end"
+          end
+          tf = Tempfile.new('unittest')
+          tf.puts("DEFAULT_PACKET_LOG_READER my_reader")
+          tf.close
+          System.instance.process_file(tf.path)
+          expect(System.default_packet_log_reader).to eql MyReader
+          File.delete filename
+          tf.unlink
+        end
+      end
+
+      context "with DISABLE_DNS" do
+        it "takes 0 parameters" do
+          tf = Tempfile.new('unittest')
+          tf.puts("DISABLE_DNS BLAH TRUE")
+          tf.close
+          expect { System.instance.process_file(tf.path) }.to raise_error(ConfigParser::Error, /Too many parameters for DISABLE_DNS./)
+          tf.unlink
+        end
+
+        it "disables dns lookups" do
+          tf = Tempfile.new('unittest')
+          tf.puts("DISABLE_DNS")
+          tf.close
+          expect(System.use_dns).to be false
+          System.instance.process_file(tf.path)
+          expect(System.use_dns).to be false
+          tf.unlink
+        end
+
+        it "enables dns lookups" do
+          tf = Tempfile.new('unittest')
+          tf.puts("ENABLE_DNS")
+          tf.close
+          expect(System.use_dns).to be false
+          System.instance.process_file(tf.path)
+          expect(System.use_dns).to be true
+          tf.unlink
+        end
+      end
+
+      context "with ALLOW_ACCESS" do
+        it "takes 1 parameters" do
+          tf = Tempfile.new('unittest')
+          tf.puts("ALLOW_ACCESS")
+          tf.close
+          expect { System.instance.process_file(tf.path) }.to raise_error(ConfigParser::Error, /Not enough parameters for ALLOW_ACCESS./)
+          tf.unlink
+
+          tf = Tempfile.new('unittest')
+          tf.puts("ALLOW_ACCESS localhost true")
+          tf.close
+          expect { System.instance.process_file(tf.path) }.to raise_error(ConfigParser::Error, /Too many parameters for ALLOW_ACCESS./)
+          tf.unlink
+        end
+
+        it "complains about bad addresses" do
+          tf = Tempfile.new('unittest')
+          tf.puts("ALLOW_ACCESS blah")
+          tf.close
+          expect { System.instance.process_file(tf.path) }.to raise_error(ConfigParser::Error)
+          tf.unlink
+
+          tf = Tempfile.new('unittest')
+          tf.puts("ALLOW_ACCESS hopefully_this_is_not_a_valid_machine_name_XYZ")
+          tf.close
+          expect { System.instance.process_file(tf.path) }.to raise_error(ConfigParser::Error)
+          tf.unlink
+        end
+
+        it "allows ALL" do
+          tf = Tempfile.new('unittest')
+          tf.puts("ALLOW_ACCESS ALL")
+          tf.close
+          System.instance.process_file(tf.path)
+          expect(System.acl).to be_nil
+          tf.unlink
+        end
+
+        it "stores host by name" do
+          tf = Tempfile.new('unittest')
+          tf.puts("ALLOW_ACCESS localhost")
+          tf.close
+          System.instance.process_file(tf.path)
+          expect(System.acl.allow_addr?(["AF_INET",0,"localhost","127.0.0.1"])).to be true
+          tf.unlink
+        end
+
+        it "stores host by IP address" do
+          addr = IPSocket.getaddress("www.google.com")
+          if addr and !addr.index(':')
+            tf = Tempfile.new('unittest')
+            tf.puts("ALLOW_ACCESS #{addr}")
+            tf.close
+            System.instance.process_file(tf.path)
+            expect(System.acl.allow_addr?(["AF_INET",0,"www.google.com",addr])).to be true
+            tf.unlink
+          end
+        end
+      end
+
+      context "with STALENESS_SECONDS" do
+        it "takes 1 parameters" do
+          tf = Tempfile.new('unittest')
+          tf.puts("STALENESS_SECONDS")
+          tf.close
+          expect { System.instance.process_file(tf.path) }.to raise_error(ConfigParser::Error, /Not enough parameters for STALENESS_SECONDS./)
+          tf.unlink
+
+          tf = Tempfile.new('unittest')
+          tf.puts("STALENESS_SECONDS 1 2")
+          tf.close
+          expect { System.instance.process_file(tf.path) }.to raise_error(ConfigParser::Error, /Too many parameters for STALENESS_SECONDS./)
+          tf.unlink
+        end
+
+        it "sets the number of seconds required to be stale" do
+          tf = Tempfile.new('unittest')
+          tf.puts("STALENESS_SECONDS 3")
+          tf.close
+          expect(System.staleness_seconds).to eql 30
+          System.instance.process_file(tf.path)
+          expect(System.staleness_seconds).to eql 3
+          tf.unlink
+        end
+      end
+
+      context "with ADD_MD5_FILE" do
+        it "takes 1 parameters" do
+          tf = Tempfile.new('unittest')
+          tf.puts("ADD_MD5_FILE")
+          tf.close
+          expect { System.instance.process_file(tf.path) }.to raise_error(ConfigParser::Error, /Not enough parameters for ADD_MD5_FILE./)
+          tf.unlink
+
+          tf = Tempfile.new('unittest')
+          tf.puts("ADD_MD5_FILE 1 2")
+          tf.close
+          expect { System.instance.process_file(tf.path) }.to raise_error(ConfigParser::Error, /Too many parameters for ADD_MD5_FILE./)
+          tf.unlink
+        end
+
+        it "complains about missing files" do
+          tf = Tempfile.new('unittest')
+          tf.puts("ADD_MD5_FILE missing_file")
+          tf.close
+          expect { System.instance.process_file(tf.path) }.to raise_error(/Missing expected file: missing_file/)
+          tf.unlink
+        end
+
+        it "adds a file to the MD5 calculation" do
+          md5f = Tempfile.new('md5_file')
+          tf = Tempfile.new('unittest')
+          tf.puts("ADD_MD5_FILE #{File.expand_path(md5f.path)}")
+          tf.close
+          System.instance.process_file(tf.path)
+          expect(System.additional_md5_files.include?(File.expand_path(md5f.path))).to be true
+          md5f.close
+          md5f.unlink
+          tf.unlink
+        end
+      end
+    end
+
+    describe "Cosmos.write_exception_file" do
+      it "writes a file with the exception" do
+        filename = Cosmos.write_exception_file(RuntimeError.new("HELP!"))
+        expect(File.exist?(filename)).to be true
+        File.delete(filename)
+      end
+
+      it "writes a file without a defined LOGS directory" do
+        File.open(@config_file,'w') {|file| file.puts "PATH LOGS C:/this/is/not/a/real/path" }
+        # Reset the instance variable so it will read the new config file
+        System.instance_eval('@instance = nil')
+        filename = Cosmos.write_exception_file(RuntimeError.new("HELP!"))
+        expect(File.exist?(filename)).to be true
+        File.delete(filename)
+      end
+    end
+  end
+end