cosmos 3.3.3 → 3.4.0

data/spec/script/tools_spec.rb

@@ -0,0 +1,117 @@
+# 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/script/script'
+require 'tempfile'
+
+module Cosmos
+
+  describe Script do
+
+    before(:all) do
+      cts = File.join(Cosmos::USERPATH,'config','tools','cmd_tlm_server','cmd_tlm_server.txt')
+      FileUtils.mkdir_p(File.dirname(cts))
+      File.open(cts,'w') do |file|
+        file.puts 'INTERFACE INST_INT interface.rb'
+        file.puts 'TARGET INST'
+      end
+      System.class_eval('@@instance = nil')
+
+      require 'cosmos/script'
+    end
+
+    after(:all) do
+      clean_config()
+      FileUtils.rm_rf File.join(Cosmos::USERPATH,'config','tools')
+    end
+
+    before(:each) do
+      allow_any_instance_of(Interface).to receive(:connected?).and_return(true)
+      allow_any_instance_of(Interface).to receive(:disconnect)
+      allow_any_instance_of(Interface).to receive(:write)
+      allow_any_instance_of(Interface).to receive(:read)
+
+      @server = CmdTlmServer.new
+      shutdown_cmd_tlm()
+      initialize_script_module()
+      sleep 0.1
+    end
+
+    after(:each) do
+      @server.stop
+      shutdown_cmd_tlm()
+      sleep(0.1)
+    end
+
+    describe "display" do
+      it "displays a telemetry viewer screen" do
+        allow_any_instance_of(JsonDRbObject).to receive(:display)
+        display("HI")
+      end
+
+      it "complains if unable to start telemetry viewer" do
+        # Avoid the needless delay by stubbing sleep
+        allow_any_instance_of(Object).to receive(:sleep)
+        allow(Cosmos).to receive(:run_process)
+        expect { display("HI") }.to raise_error(RuntimeError, /HI could not be displayed/)
+      end
+
+      it "complains if the screen doesn't exist" do
+        allow_any_instance_of(JsonDRbObject).to receive(:display).and_raise(Errno::ENOENT)
+        expect { display("HI") }.to raise_error(RuntimeError, /HI.txt does not exist/)
+      end
+    end
+
+    describe "clear" do
+      it "closes a telemetry viewer screen" do
+        allow_any_instance_of(JsonDRbObject).to receive(:clear)
+        clear("HI")
+      end
+
+      it "complains if unable to start telemetry viewer" do
+        # Avoid the needless delay by stubbing sleep
+        allow_any_instance_of(Object).to receive(:sleep)
+        allow(Cosmos).to receive(:run_process)
+        expect { clear("HI") }.to raise_error(RuntimeError, /HI could not be cleared/)
+      end
+
+      it "complains if the screen doesn't exist" do
+        allow_any_instance_of(JsonDRbObject).to receive(:clear).and_raise(Errno::ENOENT)
+        expect { clear("HI") }.to raise_error(RuntimeError, /HI.txt does not exist/)
+      end
+    end
+
+    describe "ScriptRunnerFrame methods" do
+      it "calls various ScriptRunnerFrame methods" do
+        class Dummy; def method_missing(meth, *args, &block); end; end
+        class ScriptRunnerFrame
+          def self.method_missing(meth, *args, &block); end
+          def self.instance; Dummy.new; end
+        end
+        set_line_delay(1.0)
+        get_line_delay
+        get_scriptrunner_message_log_filename
+        start_new_scriptrunner_message_log
+        disable_instrumentation do
+          value = 1
+        end
+        set_stdout_max_lines(1000)
+        insert_return
+        step_mode
+        run_mode
+        show_backtrace
+      end
+    end
+
+  end
+end
+

data/spec/spec_helper.rb

@@ -36,6 +36,7 @@ unless ENV['COSMOS_NO_SIMPLECOV']
 end
 require 'rspec'
 require 'ruby-prof'
+require 'benchmark/ips'
 require 'cosmos'
 require 'cosmos/utilities/logger'
 
@@ -55,11 +56,6 @@ def exit(*args)
 end
 
 RSpec.configure do |config|
-  config.expect_with :rspec do |c|
-    # Explicitly enable the should and expect syntax
-    c.syntax = [:should, :expect]
-  end
-
   # Store standard output global and CONSTANT since we will mess with them
   config.before(:all) do
     $saved_stdout_global = $stdout
@@ -162,6 +158,15 @@ RSpec.configure do |c|
       end
     end
   end
+  if ENV.key?("BENCHMARK")
+    c.around(:each) do |example|
+      Benchmark.ips do |x|
+        x.report(example.metadata[:full_description]) do
+          example.run
+        end
+      end
+    end
+  end
 # This code causes a new profile file to be created for each test case which is excessive and hard to read
 #  c.around(:each) do |example|
 #    if ENV.key?("PROFILE")

data/spec/streams/preidentified_stream_protocol_spec.rb

@@ -105,14 +105,14 @@ module Cosmos
         packet = @psp.read
         expect(packet.target_name).to eql 'COSMOS'
         expect(packet.packet_name).to eql 'VERSION'
-        expect(packet.identified?).to be_truthy
-        expect(packet.defined?).to be_falsey
+        expect(packet.identified?).to be true
+        expect(packet.defined?).to be false
 
         pkt2 = System.telemetry.update!("COSMOS","VERSION",packet.buffer)
         expect(pkt2.read('PKT_ID')).to eql 1
         expect(pkt2.read('COSMOS')).to eql 'TEST'
-        expect(pkt2.identified?).to be_truthy
-        expect(pkt2.defined?).to be_truthy
+        expect(pkt2.identified?).to be true
+        expect(pkt2.defined?).to be true
       end
     end
 

data/spec/streams/serial_stream_spec.rb

@@ -25,7 +25,7 @@ module Cosmos
         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_truthy
+        expect(ss.connected?).to be true
       end
     end
 
@@ -70,9 +70,9 @@ module Cosmos
         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_truthy
+        expect(ss.connected?).to be true
         ss.disconnect
-        expect(ss.connected?).to be_falsey
+        expect(ss.connected?).to be false
       end
 
       it "closes the read driver" do
@@ -81,9 +81,9 @@ module Cosmos
         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_truthy
+        expect(ss.connected?).to be true
         ss.disconnect
-        expect(ss.connected?).to be_falsey
+        expect(ss.connected?).to be false
       end
 
       it "does not close the driver twice" do
@@ -92,11 +92,11 @@ module Cosmos
         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_truthy
+        expect(ss.connected?).to be true
         ss.disconnect
-        expect(ss.connected?).to be_falsey
+        expect(ss.connected?).to be false
         ss.disconnect
-        expect(ss.connected?).to be_falsey
+        expect(ss.connected?).to be false
       end
     end
 

data/spec/streams/stream_protocol_spec.rb

@@ -80,7 +80,7 @@ module Cosmos
 
     describe "connected" do
       it "returns false if the stream hasn't been set" do
-        expect(@sp.connected?).to be_falsey
+        expect(@sp.connected?).to be false
       end
 
       it "returns the status of the stream connection" do
@@ -90,7 +90,7 @@ module Cosmos
         end
         stream = MyStream2.new
         @sp.connect(stream)
-        expect(@sp.connected?).to be_truthy
+        expect(@sp.connected?).to be true
       end
     end
 
@@ -103,9 +103,9 @@ module Cosmos
         end
         stream = MyStream3.new
         @sp.connect(stream)
-        expect($test).to be_falsey
+        expect($test).to be false
         @sp.disconnect
-        expect($test).to be_truthy
+        expect($test).to be true
       end
     end
 

data/spec/streams/tcpip_client_stream_spec.rb

@@ -34,21 +34,21 @@ module Cosmos
       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_truthy
+        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_truthy
+        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_truthy
+        expect(ss.connected?).to be true
         ss.disconnect
       end
     end

data/spec/streams/tcpip_socket_stream_spec.rb

@@ -158,9 +158,9 @@ module Cosmos
         expect(write).to receive(:close)
         ss = TcpipSocketStream.new(write,nil,nil,nil)
         ss.connect
-        expect(ss.connected?).to be_truthy
+        expect(ss.connected?).to be true
         ss.disconnect
-        expect(ss.connected?).to be_falsey
+        expect(ss.connected?).to be false
       end
 
       it "closes the read socket" do
@@ -169,9 +169,9 @@ module Cosmos
         expect(read).to receive(:close)
         ss = TcpipSocketStream.new(nil,read,nil,nil)
         ss.connect
-        expect(ss.connected?).to be_truthy
+        expect(ss.connected?).to be true
         ss.disconnect
-        expect(ss.connected?).to be_falsey
+        expect(ss.connected?).to be false
       end
 
       it "does not close the socket twice" do
@@ -180,11 +180,11 @@ module Cosmos
         expect(socket).to receive(:close).once
         ss = TcpipSocketStream.new(socket,socket,nil,nil)
         ss.connect
-        expect(ss.connected?).to be_truthy
+        expect(ss.connected?).to be true
         ss.disconnect
-        expect(ss.connected?).to be_falsey
+        expect(ss.connected?).to be false
         ss.disconnect
-        expect(ss.connected?).to be_falsey
+        expect(ss.connected?).to be false
       end
     end
 

data/spec/streams/template_stream_protocol_spec.rb

@@ -58,7 +58,7 @@ module Cosmos
         expect(t.status).to eq("sleep")
         tsp.disconnect
         sleep 0.1
-        expect(t.status).to be_falsey
+        expect(t.status).to be false
       end
     end
 

data/spec/system/system_spec.rb

@@ -504,9 +504,9 @@ module Cosmos
           tf = Tempfile.new('unittest')
           tf.puts("DISABLE_DNS")
           tf.close
-          expect(System.use_dns).to be_truthy
+          expect(System.use_dns).to be true
           System.instance.process_file(tf.path)
-          expect(System.use_dns).to be_falsey
+          expect(System.use_dns).to be false
           tf.unlink
         end
       end
@@ -554,7 +554,7 @@ module Cosmos
           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_truthy
+          expect(System.acl.allow_addr?(["AF_INET",0,"localhost","127.0.0.1"])).to be true
           tf.unlink
         end
 
@@ -565,7 +565,7 @@ module Cosmos
             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_truthy
+            expect(System.acl.allow_addr?(["AF_INET",0,"www.google.com",addr])).to be true
             tf.unlink
           end
         end
@@ -627,7 +627,7 @@ module Cosmos
     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_truthy
+        expect(File.exist?(filename)).to be true
         File.delete(filename)
       end
 
@@ -636,7 +636,7 @@ module Cosmos
         # 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_truthy
+        expect(File.exist?(filename)).to be true
         File.delete(filename)
       end
     end

data/spec/system/target_spec.rb

@@ -49,6 +49,8 @@ module Cosmos
         File.open(File.join(cmd_tlm,'cmd2.txt'),'w') {}
         File.open(File.join(cmd_tlm,'tlm1.txt'),'w') {}
         File.open(File.join(cmd_tlm,'tlm2.txt'),'w') {}
+        File.open(File.join(cmd_tlm,'tlm2.txt~'),'w') {}
+        File.open(File.join(cmd_tlm,'tlm2.txt.mine'),'w') {}
 
         tgt = Target.new(tgt_name,nil,tgt_path)
         expect(tgt.dir).to eql File.join(tgt_path,tgt_name)

data/spec/tools/cmd_tlm_server/api_spec.rb

@@ -445,12 +445,12 @@ module Cosmos
 
     describe "get_cmd_hazardous" do
       it "returns whether the command with parameters is hazardous" do
-        expect(@api.get_cmd_hazardous("INST","COLLECT",{"TYPE"=>"NORMAL"})).to be_falsey
-        expect(@api.get_cmd_hazardous("INST","COLLECT",{"TYPE"=>"SPECIAL"})).to be_truthy
+        expect(@api.get_cmd_hazardous("INST","COLLECT",{"TYPE"=>"NORMAL"})).to be false
+        expect(@api.get_cmd_hazardous("INST","COLLECT",{"TYPE"=>"SPECIAL"})).to be true
       end
 
       it "returns whether the command is hazardous" do
-        expect(@api.get_cmd_hazardous("INST","CLEAR")).to be_truthy
+        expect(@api.get_cmd_hazardous("INST","CLEAR")).to be true
       end
     end
 
@@ -866,7 +866,7 @@ module Cosmos
       end
 
       it "returns whether limits are enable for an item" do
-        expect(@api.limits_enabled?("INST","HEALTH_STATUS","TEMP1")).to be_truthy
+        expect(@api.limits_enabled?("INST","HEALTH_STATUS","TEMP1")).to be true
       end
     end
 
@@ -884,11 +884,11 @@ module Cosmos
       end
 
       it "enables limits for an item" do
-        expect(@api.limits_enabled?("INST","HEALTH_STATUS","TEMP1")).to be_truthy
+        expect(@api.limits_enabled?("INST","HEALTH_STATUS","TEMP1")).to be true
         @api.disable_limits("INST","HEALTH_STATUS","TEMP1")
-        expect(@api.limits_enabled?("INST","HEALTH_STATUS","TEMP1")).to be_falsey
+        expect(@api.limits_enabled?("INST","HEALTH_STATUS","TEMP1")).to be false
         @api.enable_limits("INST","HEALTH_STATUS","TEMP1")
-        expect(@api.limits_enabled?("INST","HEALTH_STATUS","TEMP1")).to be_truthy
+        expect(@api.limits_enabled?("INST","HEALTH_STATUS","TEMP1")).to be true
       end
     end
 
@@ -906,9 +906,9 @@ module Cosmos
       end
 
       it "disables limits for an item" do
-        expect(@api.limits_enabled?("INST","HEALTH_STATUS","TEMP1")).to be_truthy
+        expect(@api.limits_enabled?("INST","HEALTH_STATUS","TEMP1")).to be true
         @api.disable_limits("INST","HEALTH_STATUS","TEMP1")
-        expect(@api.limits_enabled?("INST","HEALTH_STATUS","TEMP1")).to be_falsey
+        expect(@api.limits_enabled?("INST","HEALTH_STATUS","TEMP1")).to be false
         @api.enable_limits("INST","HEALTH_STATUS","TEMP1")
       end
     end
@@ -966,11 +966,11 @@ module Cosmos
       it "enables limits for all items in the group" do
         @api.disable_limits("INST","HEALTH_STATUS","TEMP1")
         @api.disable_limits("INST","HEALTH_STATUS","TEMP3")
-        expect(@api.limits_enabled?("INST","HEALTH_STATUS","TEMP1")).to be_falsey
-        expect(@api.limits_enabled?("INST","HEALTH_STATUS","TEMP3")).to be_falsey
+        expect(@api.limits_enabled?("INST","HEALTH_STATUS","TEMP1")).to be false
+        expect(@api.limits_enabled?("INST","HEALTH_STATUS","TEMP3")).to be false
         @api.enable_limits_group("FIRST")
-        expect(@api.limits_enabled?("INST","HEALTH_STATUS","TEMP1")).to be_truthy
-        expect(@api.limits_enabled?("INST","HEALTH_STATUS","TEMP3")).to be_truthy
+        expect(@api.limits_enabled?("INST","HEALTH_STATUS","TEMP1")).to be true
+        expect(@api.limits_enabled?("INST","HEALTH_STATUS","TEMP3")).to be true
       end
     end
 
@@ -982,11 +982,11 @@ module Cosmos
       it "disables limits for all items in the group" do
         @api.enable_limits("INST","HEALTH_STATUS","TEMP1")
         @api.enable_limits("INST","HEALTH_STATUS","TEMP3")
-        expect(@api.limits_enabled?("INST","HEALTH_STATUS","TEMP1")).to be_truthy
-        expect(@api.limits_enabled?("INST","HEALTH_STATUS","TEMP3")).to be_truthy
+        expect(@api.limits_enabled?("INST","HEALTH_STATUS","TEMP1")).to be true
+        expect(@api.limits_enabled?("INST","HEALTH_STATUS","TEMP3")).to be true
         @api.disable_limits_group("FIRST")
-        expect(@api.limits_enabled?("INST","HEALTH_STATUS","TEMP1")).to be_falsey
-        expect(@api.limits_enabled?("INST","HEALTH_STATUS","TEMP3")).to be_falsey
+        expect(@api.limits_enabled?("INST","HEALTH_STATUS","TEMP1")).to be false
+        expect(@api.limits_enabled?("INST","HEALTH_STATUS","TEMP3")).to be false
       end
     end