cosmos 3.1.1 → 3.1.2

data/spec/tools/cmd_tlm_server/router_thread_spec.rb

@@ -22,16 +22,7 @@ module Cosmos
       @packet.buffer = "\x01\x02"
 
       @interface = Interface.new
-      @connected_count = 0
-      allow(@interface).to receive(:connected?) do
-        if @connected_count == 0
-          @connected_count += 1
-          false
-        else
-          @connected_count += 1
-          true
-        end
-      end
+      # Interface#connected? implemented in each test case
       allow(@interface).to receive(:connect)
       allow(@interface).to receive(:disconnect)
       allow(@interface).to receive(:read) do
@@ -40,24 +31,24 @@ module Cosmos
       end
     end
 
-    describe "start" do
-      it "should log the connection" do
-        commanding = double("commanding")
-        expect(commanding).to receive(:send_command_to_interface).at_least(1).times
-        allow(CmdTlmServer).to receive(:commanding).and_return(commanding)
+    describe "handle_packet" do
+      it "handles disconnectd interfaces" do
+        allow(@interface).to receive(:connected?).and_return(false)
         @interface.interfaces = [@interface]
         thread = RouterThread.new(@interface)
-        thread.start
-        sleep 1
-        Thread.list.length.should eql(2)
-        thread.stop
-        sleep 0.5
-        Thread.list.length.should eql(1)
+        capture_io do |stdout|
+          thread.start
+          sleep 0.2
+          Thread.list.length.should eql(2)
+          thread.stop
+          sleep 0.5
+          Thread.list.length.should eql(1)
+          stdout.string.should match "disconnected interface"
+        end
       end
-    end
 
-    describe "handle_packet" do
-      it "should handle errors when sending packets" do
+      it "handles errors when sending packets" do
+        allow(@interface).to receive(:connected?).and_return(true)
         commanding = double("commanding")
         expect(commanding).to receive(:send_command_to_interface).and_raise(RuntimeError.new("Death")).at_least(1).times
         allow(CmdTlmServer).to receive(:commanding).and_return(commanding)
@@ -74,10 +65,43 @@ module Cosmos
         end
       end
 
-      it "should handle identified yet unknown commands" do
+      it "handles identified yet unknown commands" do
+        allow(@interface).to receive(:connected?).and_return(true)
+        commanding = spy("commanding")
+        interface2 = double("interface")
+        allow(interface2).to receive(:connected?).and_return(true)
+        # Setup two interface
+        @interface.interfaces = [@interface, interface2]
+        # Verify that the command gets sent twice (one for each interface)
+        expect(commanding).to receive(:send_command_to_interface).at_least(2).times
+        allow(CmdTlmServer).to receive(:commanding).and_return(commanding)
+        @packet.target_name = 'BOB'
+        @packet.packet_name = 'SMITH'
+        thread = RouterThread.new(@interface)
+        capture_io do |stdout|
+          thread.start
+          sleep 0.2
+          Thread.list.length.should eql(2)
+          thread.stop
+          sleep 0.5
+          Thread.list.length.should eql(1)
+          stdout.string.should match "Received unknown identified command: BOB SMITH"
+        end
+      end
+
+      it "handles identified known commands" do
+        allow(@interface).to receive(:connected?).and_return(true)
         commanding = double("commanding")
+        # Setup two interface
+        interface2 = double("interface")
+        allow(interface2).to receive(:connected?).and_return(true)
+        @interface.interfaces = [@interface, interface2]
+        # Verify that the command gets sent once to the target interface
         expect(commanding).to receive(:send_command_to_interface).at_least(1).times
         allow(CmdTlmServer).to receive(:commanding).and_return(commanding)
+        target = spy("target")
+        allow(target).to receive(:interface).and_return(@interface)
+        allow(System).to receive(:targets).and_return({'BOB' => target})
         @interface.interfaces = [@interface]
         @packet.target_name = 'BOB'
         @packet.packet_name = 'SMITH'
@@ -91,6 +115,35 @@ module Cosmos
           Thread.list.length.should eql(1)
           stdout.string.should match "Received unknown identified command: BOB SMITH"
         end
+        expect(target).to have_received(:interface).twice #and_return(@interface)
+      end
+
+      it "does not send identified commands with a target and no interface" do
+        allow(@interface).to receive(:connected?).and_return(true)
+        commanding = double("commanding")
+        # Setup two interface
+        interface2 = double("interface")
+        allow(interface2).to receive(:connected?).and_return(true)
+        @interface.interfaces = [@interface, interface2]
+        # Verify that the command gets sent once to the target interface
+        expect(commanding).to_not receive(:send_command_to_interface)
+        allow(CmdTlmServer).to receive(:commanding).and_return(commanding)
+        target = spy("target")
+        allow(target).to receive(:interface).and_return(nil)
+        allow(System).to receive(:targets).and_return({'BOB' => target})
+        @interface.interfaces = [@interface]
+        @packet.target_name = 'BOB'
+        @packet.packet_name = 'SMITH'
+        thread = RouterThread.new(@interface)
+        capture_io do |stdout|
+          thread.start
+          sleep 0.2
+          Thread.list.length.should eql(2)
+          thread.stop
+          sleep 0.5
+          Thread.list.length.should eql(1)
+          stdout.string.should match "target with no interface"
+        end
       end
     end
   end

data/spec/tools/launcher/launcher_config_spec.rb

@@ -0,0 +1,460 @@
+# 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/tools/launcher/launcher_config'
+require 'tempfile'
+
+module Cosmos
+
+  describe LauncherConfig do
+    describe "initialize" do
+      it "checks for a given filename" do
+        expect { LauncherConfig.new('blah_file.txt') }.to raise_error
+      end
+
+      it "raises on unknown parameters" do
+        tf = Tempfile.new('mylauncher.txt')
+        tf.puts "UNKNOWN"
+        tf.close
+        expect { LauncherConfig.new(tf.path) }.to raise_error(Cosmos::ConfigParser::Error)
+        tf.unlink
+      end
+
+      describe 'TITLE' do
+        it "parses a single string" do
+          tf = Tempfile.new('mylauncher.txt')
+          tf.puts "TITLE \"MyLauncher\""
+          tf.close
+
+          lc = LauncherConfig.new(tf.path)
+          expect(lc.title).to eq "MyLauncher"
+          tf.unlink
+        end
+
+        it "raises with more than 1 parameter" do
+          tf = Tempfile.new('mylauncher.txt')
+          tf.puts "TITLE My Launcher"
+          tf.close
+
+          expect { LauncherConfig.new(tf.path) }.to raise_error(Cosmos::ConfigParser::Error)
+          tf.unlink
+        end
+      end
+
+      describe 'DIVIDER' do
+        it "parses without arguments" do
+          tf = Tempfile.new('mylauncher.txt')
+          tf.puts "DIVIDER"
+          tf.close
+
+          expect { LauncherConfig.new(tf.path) }.to_not raise_error
+          tf.unlink
+        end
+
+        it "raises with a parameter" do
+          tf = Tempfile.new('mylauncher.txt')
+          tf.puts "DIVIDER 1"
+          tf.close
+
+          expect { LauncherConfig.new(tf.path) }.to raise_error(Cosmos::ConfigParser::Error)
+          tf.unlink
+        end
+      end
+
+      describe 'NUM_COLUMNS' do
+        it "parses a single value" do
+          tf = Tempfile.new('mylauncher.txt')
+          tf.puts "NUM_COLUMNS 6"
+          tf.close
+
+          lc = LauncherConfig.new(tf.path)
+          expect(lc.num_columns).to eq 6
+          tf.unlink
+        end
+
+        it "raises with a string" do
+          tf = Tempfile.new('mylauncher.txt')
+          tf.puts "NUM_COLUMNS TWO"
+          tf.close
+
+          expect { LauncherConfig.new(tf.path) }.to raise_error(Cosmos::ConfigParser::Error)
+          tf.unlink
+        end
+
+        it "raises with more than 1 parameter" do
+          tf = Tempfile.new('mylauncher.txt')
+          tf.puts "NUM_COLUMNS 2 3"
+          tf.close
+
+          expect { LauncherConfig.new(tf.path) }.to raise_error(Cosmos::ConfigParser::Error)
+          tf.unlink
+        end
+      end
+
+      describe 'LABEL' do
+        it "parses a single value" do
+          tf = Tempfile.new('mylauncher.txt')
+          tf.puts "LABEL 'HI THERE'"
+          tf.close
+
+          lc = LauncherConfig.new(tf.path)
+          expect(lc.items[0][0]).to eq :LABEL
+          expect(lc.items[0][1]).to eq "HI THERE"
+          expect(lc.items[0][2]).to be_nil
+          expect(lc.items[0][3]).to be_nil
+          expect(lc.items[0][4]).to be_nil
+          tf.unlink
+        end
+
+        it "raises with more than 1 parameter" do
+          tf = Tempfile.new('mylauncher.txt')
+          tf.puts "LABEL HI THERE"
+          tf.close
+
+          expect { LauncherConfig.new(tf.path) }.to raise_error(Cosmos::ConfigParser::Error)
+          tf.unlink
+        end
+      end
+
+      describe 'TOOL_FONT' do
+        it "parses the font and size" do
+          tf = Tempfile.new('mylauncher.txt')
+          tf.puts "TOOL_FONT tahoma 12"
+          tf.close
+
+          lc = LauncherConfig.new(tf.path)
+          expect(lc.tool_font_settings[0]).to eq 'tahoma'
+          expect(lc.tool_font_settings[1]).to eq 12
+          tf.unlink
+        end
+
+        it "raises with only 1 parameter" do
+          tf = Tempfile.new('mylauncher.txt')
+          tf.puts "TOOL_FONT tahoma"
+          tf.close
+
+          expect { LauncherConfig.new(tf.path) }.to raise_error(Cosmos::ConfigParser::Error)
+          tf.unlink
+        end
+
+        it "raises with more than 2 parameters" do
+          tf = Tempfile.new('mylauncher.txt')
+          tf.puts "TOOL_FONT tahoma 12 16"
+          tf.close
+
+          expect { LauncherConfig.new(tf.path) }.to raise_error(Cosmos::ConfigParser::Error)
+          tf.unlink
+        end
+
+        # TODO: Is it possible to check for bad fonts?
+
+        it "raises with a bad size" do
+          tf = Tempfile.new('mylauncher.txt')
+          tf.puts "TOOL_FONT tahoma arial"
+          tf.close
+
+          expect { LauncherConfig.new(tf.path) }.to raise_error(Cosmos::ConfigParser::Error)
+          tf.unlink
+        end
+      end
+
+      describe 'LABEL_FONT' do
+        it "parses the font and size" do
+          tf = Tempfile.new('mylauncher.txt')
+          tf.puts "LABEL_FONT tahoma 12"
+          tf.close
+
+          lc = LauncherConfig.new(tf.path)
+          expect(lc.label_font_settings[0]).to eq 'tahoma'
+          expect(lc.label_font_settings[1]).to eq 12
+          tf.unlink
+        end
+
+        it "raises with only 1 parameter" do
+          tf = Tempfile.new('mylauncher.txt')
+          tf.puts "LABEL_FONT tahoma"
+          tf.close
+
+          expect { LauncherConfig.new(tf.path) }.to raise_error(Cosmos::ConfigParser::Error)
+          tf.unlink
+        end
+
+        it "raises with more than 2 parameters" do
+          tf = Tempfile.new('mylauncher.txt')
+          tf.puts "LABEL_FONT tahoma 12 16"
+          tf.close
+
+          expect { LauncherConfig.new(tf.path) }.to raise_error(Cosmos::ConfigParser::Error)
+          tf.unlink
+        end
+
+        # TODO: Is it possible to check for bad fonts?
+
+        it "raises with a bad size" do
+          tf = Tempfile.new('mylauncher.txt')
+          tf.puts "LABEL_FONT tahoma arial"
+          tf.close
+
+          expect { LauncherConfig.new(tf.path) }.to raise_error(Cosmos::ConfigParser::Error)
+          tf.unlink
+        end
+      end
+
+      describe "DONT_CAPTURE_IO" do
+        it "must follow a TOOL" do
+          tf = Tempfile.new('mylauncher.txt')
+          tf.puts '  DONT_CAPTURE_IO'
+          tf.close
+
+          expect { LauncherConfig.new(tf.path) }.to raise_error(Cosmos::ConfigParser::Error)
+          tf.unlink
+
+          tf = Tempfile.new('mylauncher.txt')
+          tf.puts 'MULTITOOL_START "MyTool"'
+          tf.puts '  DONT_CAPTURE_IO'
+          tf.puts '  TOOL "LAUNCH CmdTlmServer"'
+          tf.puts 'MULTITOOL_END'
+          tf.close
+
+          expect { LauncherConfig.new(tf.path) }.to raise_error(Cosmos::ConfigParser::Error)
+          tf.unlink
+
+          tf = Tempfile.new('mylauncher.txt')
+          tf.puts 'TOOL "Server" "LAUNCH CmdTlmServer"'
+          tf.puts '  DONT_CAPTURE_IO'
+          tf.close
+
+          lc = LauncherConfig.new(tf.path)
+          expect(lc.items[0][3]).to be false
+          tf.unlink
+        end
+
+        it "raises with a parameter" do
+          tf = Tempfile.new('mylauncher.txt')
+          tf.puts 'TOOL "Server" "LAUNCH CmdTlmServer"'
+          tf.puts '  DONT_CAPTURE_IO true'
+          tf.close
+
+          expect { LauncherConfig.new(tf.path) }.to raise_error(Cosmos::ConfigParser::Error)
+          tf.unlink
+        end
+
+        it "valid within MULTITOOL" do
+          tf = Tempfile.new('mylauncher.txt')
+          tf.puts 'MULTITOOL_START "MyTool"'
+          tf.puts '  TOOL "LAUNCH CmdTlmServer"'
+          tf.puts '    DONT_CAPTURE_IO'
+          tf.puts 'MULTITOOL_END'
+          tf.close
+
+          lc = LauncherConfig.new(tf.path)
+          expect(lc.items[0][2][0][2]).to be false
+          tf.unlink
+        end
+      end
+
+      describe "TOOL" do
+        it "parses name and LAUNCH shell command" do
+          tf = Tempfile.new('mylauncher.txt')
+          tf.puts 'TOOL "Server" "LAUNCH CmdTlmServer --system system.txt -x 0 -y 0"'
+          tf.close
+
+          lc = LauncherConfig.new(tf.path)
+          expect(lc.items[0][0]).to eq :TOOL
+          expect(lc.items[0][1]).to eq 'Server'
+          if Kernel.is_mac? and File.exist?(File.join(USERPATH, 'tools', 'mac'))
+            expect(lc.items[0][2]).to eq 'open tools/mac/CmdTlmServer.app --args --system system.txt -x 0 -y 0'
+          else
+            expect(lc.items[0][2]).to eq 'RUBYW tools/CmdTlmServer --system system.txt -x 0 -y 0'
+          end
+          expect(lc.items[0][3]).to be true
+          expect(lc.items[0][4]).to be_nil
+          expect(lc.items[0][5]).to be_nil
+          tf.unlink
+        end
+
+        it "parses name and LAUNCH_TERMINAL shell command" do
+          tf = Tempfile.new('mylauncher.txt')
+          tf.puts 'TOOL "Example" "LAUNCH_TERMINAL Example"'
+          tf.close
+
+          lc = LauncherConfig.new(tf.path)
+          expect(lc.items[0][0]).to eq :TOOL
+          expect(lc.items[0][1]).to eq 'Example'
+          if Kernel.is_mac?
+            expect(lc.items[0][2]).to eq "osascript -e 'tell application \"Terminal\" to do script \"cd #{File.expand_path(USERPATH)} && ruby tools/Example \"' -e 'return'"
+          elsif Kernel.is_windows?
+            expect(lc.items[0][2]).to eq "start ruby tools/Example"
+          else
+            expect(lc.items[0][2]).to eq "gnome-terminal -e \"ruby tools/Example \""
+          end
+          expect(lc.items[0][3]).to be true
+          expect(lc.items[0][4]).to be_nil
+          expect(lc.items[0][5]).to be_nil
+          tf.unlink
+        end
+
+        it "parses name and random shell command" do
+          tf = Tempfile.new('mylauncher.txt')
+          tf.puts 'TOOL "List" "ls -la"'
+          tf.close
+
+          lc = LauncherConfig.new(tf.path)
+          expect(lc.items[0][0]).to eq :TOOL
+          expect(lc.items[0][1]).to eq 'List'
+          expect(lc.items[0][2]).to eq 'ls -la'
+          tf.unlink
+        end
+
+        it "parses icon and parameters" do
+          tf = Tempfile.new('mylauncher.txt')
+          tf.puts 'TOOL "Server" "LAUNCH CmdTlmServer --system system.txt -x 0 -y 0" "cts.png" --config server.txt --width 500 --height 500'
+          tf.close
+
+          lc = LauncherConfig.new(tf.path)
+          expect(lc.items[0][4]).to eq 'cts.png'
+          expect(lc.items[0][5].join(' ')).to eq '--config server.txt --width 500 --height 500'
+          tf.unlink
+        end
+
+        it "raises with less than 2 parameters" do
+          tf = Tempfile.new('mylauncher.txt')
+          tf.puts 'TOOL "Server"'
+          tf.close
+
+          expect { LauncherConfig.new(tf.path) }.to raise_error(Cosmos::ConfigParser::Error)
+          tf.unlink
+        end
+      end
+
+      describe "DELAY" do
+        it "only valid within MULTITOOL" do
+          tf = Tempfile.new('mylauncher.txt')
+          tf.puts '  DELAY'
+          tf.close
+
+          expect { LauncherConfig.new(tf.path) }.to raise_error(Cosmos::ConfigParser::Error)
+          tf.unlink
+
+          tf = Tempfile.new('mylauncher.txt')
+          tf.puts 'MULTITOOL_START "MyTool" icon.png'
+          tf.puts '  DELAY 1'
+          tf.puts '  TOOL "LAUNCH CmdTlmServer"'
+          tf.puts 'MULTITOOL_END'
+          tf.close
+
+          lc = LauncherConfig.new(tf.path)
+          expect(lc.items[0][2][0][0]).to eq :DELAY
+          expect(lc.items[0][2][0][1]).to eq 1
+          tf.unlink
+        end
+
+        it "raises without a parameter" do
+          tf = Tempfile.new('mylauncher.txt')
+          tf.puts 'MULTITOOL_START "MyTool" icon.png'
+          tf.puts '  DELAY'
+          tf.puts '  TOOL "LAUNCH CmdTlmServer"'
+          tf.puts 'MULTITOOL_END'
+          tf.close
+
+          expect { LauncherConfig.new(tf.path) }.to raise_error(Cosmos::ConfigParser::Error)
+          tf.unlink
+        end
+
+        it "raises with more than 1 parameter" do
+          tf = Tempfile.new('mylauncher.txt')
+          tf.puts 'MULTITOOL_START "MyTool" icon.png'
+          tf.puts '  DELAY 1 2'
+          tf.puts '  TOOL "LAUNCH CmdTlmServer"'
+          tf.puts 'MULTITOOL_END'
+          tf.close
+
+          expect { LauncherConfig.new(tf.path) }.to raise_error(Cosmos::ConfigParser::Error)
+          tf.unlink
+        end
+      end
+
+      describe "MULTITOOL" do
+        it "parses name and icon" do
+          tf = Tempfile.new('mylauncher.txt')
+          tf.puts 'MULTITOOL_START "MyTool" icon.png'
+          tf.puts '  TOOL "LAUNCH CmdTlmServer"'
+          tf.puts 'MULTITOOL_END'
+          tf.close
+
+          lc = LauncherConfig.new(tf.path)
+          expect(lc.items[0][0]).to eq :MULTITOOL
+          expect(lc.items[0][1]).to eq 'MyTool'
+          expect(lc.items[0][2]).to_not be_empty
+          expect(lc.items[0][3]).to be true
+          expect(lc.items[0][4]).to eq 'icon.png'
+          expect(lc.items[0][5]).to be_nil
+          tf.unlink
+        end
+
+        it "raises with no parameters" do
+          tf = Tempfile.new('mylauncher.txt')
+          tf.puts 'MULTITOOL_START'
+          tf.puts '  TOOL "LAUNCH CmdTlmServer"'
+          tf.puts 'MULTITOOL_END'
+          tf.close
+
+          expect { LauncherConfig.new(tf.path) }.to raise_error(Cosmos::ConfigParser::Error)
+          tf.unlink
+        end
+
+        it "raises with more than two parameters" do
+          tf = Tempfile.new('mylauncher.txt')
+          tf.puts 'MULTITOOL_START Launch icon.png more'
+          tf.puts '  TOOL "LAUNCH CmdTlmServer"'
+          tf.puts 'MULTITOOL_END'
+          tf.close
+
+          expect { LauncherConfig.new(tf.path) }.to raise_error(Cosmos::ConfigParser::Error)
+          tf.unlink
+        end
+
+        it "raises if no tools are defined" do
+          tf = Tempfile.new('mylauncher.txt')
+          tf.puts 'MULTITOOL_START Launch'
+          tf.puts 'MULTITOOL_END'
+          tf.close
+
+          expect { LauncherConfig.new(tf.path) }.to raise_error(Cosmos::ConfigParser::Error)
+          tf.unlink
+
+          tf = Tempfile.new('mylauncher.txt')
+          tf.puts 'MULTITOOL_START Launch'
+          tf.puts '  DELAY 1'
+          tf.puts 'MULTITOOL_END'
+          tf.close
+
+          expect { LauncherConfig.new(tf.path) }.to raise_error(Cosmos::ConfigParser::Error)
+          tf.unlink
+        end
+      end
+    end
+
+#  DONT_CAPTURE_IO
+#MULTITOOL_START "COSMOS"
+#  TOOL "LAUNCH CmdTlmServer -x 827 -y 2 -w 756 -t 475 -c cmd_tlm_server.txt"
+#  DELAY 5
+#  TOOL "LAUNCH TlmViewer -x 827 -y 517 -w 424 -t 111"
+#    DONT_CAPTURE_IO
+#MULTITOOL_END
+#DIVIDER
+#LABEL "This is a test"
+#DOC
+
+  end
+end
+