pi_driver 0.0.3 → 0.0.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 3a7911c66c972390a7e29125befb058062ca4843
-  data.tar.gz: 96117a7d39eb4ff1b0a391ee672a9c49b0460353
+  metadata.gz: ec7c160fe4b3aeb5e32d745b696ce308eb4a75db
+  data.tar.gz: 868d3c59043a67f94ea6a27cfc8d6d82236090de
 SHA512:
-  metadata.gz: 4b63ed0825623d485602850e0f70ae8ad2d5768a9a2145b11b1c97cd37698622e19ab5f14e2c61833e81fe3e7ca7fc6ddc5e65e9b4ca962b3c577d80828fa9db
-  data.tar.gz: 1d0249b62cf2f6e653178760dc17e3b8caf7d62a83bd22096b4b48ef38d211b1e0967bcea18f2cbfe5eccb3da0183418e23de228aea9481160ae8341b28d82b5
+  metadata.gz: 9836ae29da58513975b9b46aaa00df30bc388db8e02f21daa5aafd732371472ba7a1d411e7536d7080a89fd173b2e45321e162a9b4ecd122b7fabde95f748007
+  data.tar.gz: 9f9175009294e86637cee0a10350a66a955b6efee79c33f3a29843a64ecd670e4ad67a0d157b6488f793b793f24e93c9f408cce1c77126528fb2ca6516b8528c

data/lib/pi_driver.rb

@@ -4,4 +4,4 @@ require_relative 'pi_driver/pin'
 require_relative 'pi_driver/utils'
 
 module PiDriver
-end
+end

data/lib/pi_driver/device.rb

@@ -3,4 +3,4 @@ require_relative 'device/mcp23017'
 module PiDriver
   class Device
   end
-end
+end

data/lib/pi_driver/device/mcp23017.rb

@@ -19,7 +19,7 @@ module PiDriver
       attr_reader :hardware_address
 
       def initialize(options = {})
-        @argument_helper = Utils::ArgumentHelper.new prefix: "MCP23017"
+        @argument_helper = Utils::ArgumentHelper.new prefix: 'MCP23017'
         @i2c_master = options[:i2c_master]
         @hardware_address = HardwareAddress.new observer: self
         update_opcodes
@@ -27,7 +27,7 @@ module PiDriver
 
       def update_registers
         bank = registers[:iocon].bank
-        registers.each do |_, register|
+        registers.each_value do |register|
           register.update_address bank
         end
       end
@@ -39,50 +39,33 @@ module PiDriver
         @opcode_for_read = PiDriver::I2CMaster.prepare_address_for_read base
       end
 
-      def write(*register_array)
-        @i2c_master.start
-
-        register_array.each_with_index do |register, index|
-          @argument_helper.check(:register, register, registers.keys)
+      # TODO: read and write look similar. consider consolidation
+      def read(*register_array)
+        check_registers register_array
 
-          @i2c_master.write @opcode_for_write
-          @i2c_master.ack
-          @i2c_master.write registers[register].address
-          @i2c_master.ack
-          @i2c_master.write registers[register].byte
-          @i2c_master.ack
+        @i2c_master.start
 
-          @i2c_master.restart unless index == register_array.length - 1
+        register_array.each_with_index do |register, sequence_index|
+          @i2c_master.restart unless sequence_index.zero?
+          read_register register
         end
 
         @i2c_master.stop
       end
 
-      def read(*register_array)
-        @i2c_master.start
-
-        register_array.each_with_index do |register, index|
-          @argument_helper.check(:register, register, registers.keys)
-
-          @i2c_master.write @opcode_for_write
-          @i2c_master.ack
-          @i2c_master.write registers[register].address
-          @i2c_master.ack
-
-          @i2c_master.restart
+      def write(*register_array)
+        check_registers register_array
 
-          @i2c_master.write @opcode_for_read
-          @i2c_master.ack
-          registers[register].byte = @i2c_master.read
+        @i2c_master.start
 
-          @i2c_master.restart unless index == register_array.length - 1
+        register_array.each_with_index do |register, sequence_index|
+          @i2c_master.restart unless sequence_index.zero?
+          write_register register
         end
 
         @i2c_master.stop
       end
 
-      private
-
       def self.register_reader(*register_array)
         register_array.each do |register|
           define_method register do
@@ -91,6 +74,7 @@ module PiDriver
         end
       end
 
+      private_class_method :register_reader
       register_reader :iodira, :iodirb
       register_reader :ipola, :ipolb
       register_reader :gpintena, :gpintenb
@@ -103,10 +87,13 @@ module PiDriver
       register_reader :gpioa, :gpiob
       register_reader :olata, :olatb
 
+      private
+
       def registers
         @registers ||= initialize_registers
       end
 
+      # rubocop:disable Metrics/AbcSize
       def initialize_registers
         {
           iodira: Iodir.new(port: :a),      iodirb: Iodir.new(port: :b),
@@ -124,6 +111,35 @@ module PiDriver
           olata: Olat.new(port: :a),        olatb: Olat.new(port: :b)
         }
       end
+      # rubocop:enable Metrics/AbcSize
+
+      def check_registers(register_array)
+        register_array.each do |register|
+          @argument_helper.check(:register, register, registers.keys)
+        end
+      end
+
+      def read_register(register)
+        @i2c_master.write @opcode_for_write
+        @i2c_master.ack
+        @i2c_master.write registers[register].address
+        @i2c_master.ack
+
+        @i2c_master.restart
+
+        @i2c_master.write @opcode_for_read
+        @i2c_master.ack
+        registers[register].byte = @i2c_master.read
+      end
+
+      def write_register(register)
+        @i2c_master.write @opcode_for_write
+        @i2c_master.ack
+        @i2c_master.write registers[register].address
+        @i2c_master.ack
+        @i2c_master.write registers[register].byte
+        @i2c_master.ack
+      end
     end
   end
-end
+end

data/lib/pi_driver/device/mcp23017/hardware_address.rb

@@ -5,7 +5,7 @@ module PiDriver
         attr_reader :a0, :a1, :a2
 
         def initialize(options)
-          @argument_helper = Utils::ArgumentHelper.new prefix: "MCP23017::HardwareAddress"
+          @argument_helper = Utils::ArgumentHelper.new prefix: 'MCP23017::HardwareAddress'
 
           @observer = options.delete :observer
 
@@ -14,8 +14,6 @@ module PiDriver
           @a2 = Utils::State::LOW
         end
 
-        private
-
         def self.address_writer(*addresses)
           addresses.each do |address|
             setter_symbol = "#{address}=".to_sym
@@ -34,4 +32,4 @@ module PiDriver
       end
     end
   end
-end
+end

data/lib/pi_driver/device/mcp23017/port.rb

@@ -8,7 +8,7 @@ module PiDriver
         VALID_PORTS = [
           A,
           B
-        ]
+        ].freeze
 
         def self.default(options)
           if options[:register] == MCP23017::Register::RegisterHelper::IODIR
@@ -20,4 +20,4 @@ module PiDriver
       end
     end
   end
-end
+end

data/lib/pi_driver/device/mcp23017/register.rb

@@ -10,7 +10,7 @@ module PiDriver
         attr_reader :address, :byte
 
         def initialize(options)
-          @argument_helper = Utils::ArgumentHelper.new prefix: "MCP23017::Register"
+          @argument_helper = Utils::ArgumentHelper.new prefix: 'MCP23017::Register'
 
           @register = options[:register]
           @port = options[:port]
@@ -26,7 +26,6 @@ module PiDriver
           @argument_helper.check(:register_byte, value, Utils::Byte::VALID_BYTES)
           @byte = value
           mirror_bits_from_byte
-          @byte
         end
 
         def update_address(bank)
@@ -34,8 +33,6 @@ module PiDriver
           @address = RegisterHelper.address options
         end
 
-        private
-
         def self.bit_accessors
           Utils::Byte::NUM_BITS_PER_BYTE.times do |bit_number|
             string = "bit#{bit_number}"
@@ -58,6 +55,7 @@ module PiDriver
           end
         end
 
+        private_class_method :bit_accessors
         bit_accessors
 
         def self.mirror_bit_accessors(prefix_symbol)
@@ -76,8 +74,10 @@ module PiDriver
           end
         end
 
+        private_class_method :mirror_bit_accessors
+
         def mirror_bits_from_byte
-          byte_bits = Utils::Byte::byte_to_bits(@byte)
+          byte_bits = Utils::Byte.byte_to_bits(@byte)
 
           @bit0 = byte_bits.pop
           @bit1 = byte_bits.pop
@@ -90,9 +90,9 @@ module PiDriver
         end
 
         def mirror_byte_from_bits
-          @byte = Utils::Byte::bits_to_byte [@bit7, @bit6, @bit5, @bit4, @bit3, @bit2, @bit1, @bit0]
+          @byte = Utils::Byte.bits_to_byte [@bit7, @bit6, @bit5, @bit4, @bit3, @bit2, @bit1, @bit0]
         end
       end
     end
   end
-end
+end

data/lib/pi_driver/device/mcp23017/register/defval.rb

@@ -11,4 +11,4 @@ module PiDriver
       end
     end
   end
-end
+end

data/lib/pi_driver/device/mcp23017/register/gpinten.rb

@@ -11,4 +11,4 @@ module PiDriver
       end
     end
   end
-end
+end

data/lib/pi_driver/device/mcp23017/register/gpio.rb

@@ -11,4 +11,4 @@ module PiDriver
       end
     end
   end
-end
+end

data/lib/pi_driver/device/mcp23017/register/gppu.rb

@@ -11,4 +11,4 @@ module PiDriver
       end
     end
   end
-end
+end

data/lib/pi_driver/device/mcp23017/register/intcap.rb

@@ -11,4 +11,4 @@ module PiDriver
       end
     end
   end
-end
+end

data/lib/pi_driver/device/mcp23017/register/intcon.rb

@@ -11,4 +11,4 @@ module PiDriver
       end
     end
   end
-end
+end

data/lib/pi_driver/device/mcp23017/register/intf.rb

@@ -11,4 +11,4 @@ module PiDriver
       end
     end
   end
-end
+end

data/lib/pi_driver/device/mcp23017/register/iocon.rb

@@ -15,27 +15,27 @@ module PiDriver
           @observer.update_registers
         end
 
-        alias_method :bank, :bit7
-        alias_method :bank=, :bit7=
+        alias bank bit7
+        alias bank= bit7=
 
-        alias_method :mirror, :bit6
-        alias_method :mirror=, :bit6=
+        alias mirror bit6
+        alias mirror= bit6=
 
-        alias_method :seqop, :bit5
-        alias_method :seqop=, :bit5=
+        alias seqop bit5
+        alias seqop= bit5=
 
-        alias_method :disslw, :bit4
-        alias_method :disslw=, :bit4=
+        alias disslw bit4
+        alias disslw= bit4=
 
-        alias_method :haen, :bit3
-        alias_method :haen=, :bit3=
+        alias haen bit3
+        alias haen= bit3=
 
-        alias_method :odr, :bit2
-        alias_method :odr=, :bit2=
+        alias odr bit2
+        alias odr= bit2=
 
-        alias_method :intpol, :bit1
-        alias_method :intpol=, :bit1=
+        alias intpol bit1
+        alias intpol= bit1=
       end
     end
   end
-end
+end

data/lib/pi_driver/device/mcp23017/register/iodir.rb

@@ -11,4 +11,4 @@ module PiDriver
       end
     end
   end
-end
+end

data/lib/pi_driver/device/mcp23017/register/ipol.rb

@@ -11,4 +11,4 @@ module PiDriver
       end
     end
   end
-end
+end

data/lib/pi_driver/device/mcp23017/register/olat.rb

@@ -11,4 +11,4 @@ module PiDriver
       end
     end
   end
-end
+end

data/lib/pi_driver/device/mcp23017/register/register_helper.rb

@@ -27,7 +27,7 @@ module PiDriver
             INTCAP,
             GPIO,
             OLAT
-          ]
+          ].freeze
 
           def self.address(options)
             bank = options[:bank]
@@ -47,4 +47,4 @@ module PiDriver
       end
     end
   end
-end
+end

data/lib/pi_driver/i2c_master.rb

@@ -2,7 +2,7 @@ module PiDriver
   class I2CMaster
     def initialize(options = {})
       @frequency = 100_000
-      @delta_time = @frequency ** -1.0
+      @delta_time = @frequency**-1.0
       @clock_pin = options[:clock_pin]
       @data_pin = options[:data_pin]
       stop
@@ -18,7 +18,7 @@ module PiDriver
       drive_clock_pin
     end
 
-    alias_method :restart, :start
+    alias restart start
 
     def write(byte)
       send_data byte

data/lib/pi_driver/pin.rb

@@ -7,7 +7,7 @@ module PiDriver
     attr_reader :gpio_number
 
     def initialize(gpio_number, options = {})
-      @argument_helper = Utils::ArgumentHelper.new prefix: "PiDriver::Pin"
+      @argument_helper = Utils::ArgumentHelper.new prefix: 'PiDriver::Pin'
 
       @gpio_number = gpio_number
       @argument_helper.check(:gpio_number, @gpio_number, Board::VALID_NUMBERS)
@@ -54,13 +54,13 @@ module PiDriver
       @state
     end
 
-    alias_method :off, :clear
+    alias off clear
 
     def clear?
       state == Utils::State::LOW
     end
 
-    alias_method :off?, :clear?
+    alias off? clear?
 
     def set
       return unless output?
@@ -69,13 +69,13 @@ module PiDriver
       @state
     end
 
-    alias_method :on, :set
+    alias on set
 
     def set?
       state == Utils::State::HIGH
     end
 
-    alias_method :on?, :set?
+    alias on? set?
 
     def state
       input? ? @file_helper.read_value : @state
@@ -89,7 +89,7 @@ module PiDriver
     end
 
     def clear_interrupt
-      @interrupt.clear if @interrupt
+      @interrupt&.clear
     end
   end
 end

data/lib/pi_driver/pin/board.rb

@@ -59,7 +59,7 @@ module PiDriver
         PIN40,
         PIN27,
         PIN28
-      ]
+      ].freeze
     end
   end
 end

data/lib/pi_driver/pin/direction.rb

@@ -7,7 +7,7 @@ module PiDriver
       VALID_DIRECTIONS = [
         INPUT,
         OUTPUT
-      ]
+      ].freeze
     end
   end
 end

data/lib/pi_driver/pin/directory_helper.rb

@@ -1,10 +1,10 @@
 module PiDriver
   class Pin
     class DirectoryHelper
-      # TODO figure out environment variables for test
+      # TODO: figure out environment variables for test
       # DIR_BASE = '/sys/class'
       DIR_BASE = File.expand_path '~/pi/gpio/sys/class'
-      DIR_GPIO = "#{DIR_BASE}/gpio"
+      DIR_GPIO = "#{DIR_BASE}/gpio".freeze
 
       def initialize(gpio_number)
         @gpio_number = gpio_number

data/lib/pi_driver/pin/file_helper.rb

@@ -8,10 +8,6 @@ module PiDriver
         @directory_helper = DirectoryHelper.new @gpio_number
       end
 
-      def read_direction
-        File.read(@directory_helper.direction).to_sym
-      end
-
       def read_value
         File.read(@directory_helper.value).to_i
       end

data/lib/pi_driver/utils/argument_helper.rb

@@ -9,10 +9,12 @@ module PiDriver
       end
 
       def check(type, arg, valid_options)
-        valid_options_for_message = valid_options.map { |value| "#{value.inspect}"}.join(', ')
-        middle = "invalid argument #{arg.inspect} for #{type.inspect} expected to be one of #{valid_options_for_message}"
+        return if valid_options.include?(arg)
+        options = valid_options.map(&:to_s).join(', ')
+        middle = "invalid argument for #{type.inspect}, "
+        middle += "#{arg.inspect} was given but expected to be one of #{options}"
         message = "#{@prefix if @prefix} #{middle} #{@suffix if @suffix}"
-        raise ArgumentError, message unless valid_options.include?(arg)
+        raise ArgumentError, message
       end
     end
   end

data/lib/pi_driver/utils/byte.rb

@@ -7,8 +7,6 @@ module PiDriver
 
       VALID_BYTES = (ALL_BITS_LOW..ALL_BITS_HIGH).to_a
 
-      private
-
       def self.byte_to_bits(byte)
         byte.to_s(2).rjust(NUM_BITS_PER_BYTE, '0').chars.map(&:to_i)
       end

data/lib/pi_driver/utils/edge.rb

@@ -11,7 +11,7 @@ module PiDriver
         FALLING,
         BOTH,
         NONE
-      ]
+      ].freeze
     end
   end
 end

data/lib/pi_driver/utils/interrupt.rb

@@ -2,7 +2,7 @@ module PiDriver
   module Utils
     class Interrupt
       def initialize(edge, &block)
-        @argument_helper = Utils::ArgumentHelper.new prefix: "Utils::Interrupt"
+        @argument_helper = Utils::ArgumentHelper.new prefix: 'Utils::Interrupt'
         @edge = edge
 
         @argument_helper.check(:block, block_given?, [true])
@@ -16,8 +16,8 @@ module PiDriver
           last_state = check
           loop do
             new_state = check
-            edge = get_interrupt_edge(new_state, last_state)
-            yield(edge) if edge
+            edge = get_current_edge(new_state, last_state)
+            yield edge if valid_edge? edge
             last_state = new_state
           end
         end
@@ -36,25 +36,36 @@ module PiDriver
         state
       end
 
-      def get_interrupt_edge(new_state, last_state)
-        rising_trigger = new_state == Utils::State::HIGH && last_state == Utils::State::LOW
-        falling_trigger = new_state == Utils::State::LOW && last_state == Utils::State::HIGH
+      # TODO: I hate that I had to break this into separate functions. Revisit.
+      def get_current_edge(new_state, last_state)
+        is_rising = new_state == Utils::State::HIGH && last_state == Utils::State::LOW
+        is_falling = new_state == Utils::State::LOW && last_state == Utils::State::HIGH
 
-        return Utils::Edge::RISING if rising_trigger && (both? || rising?)
-        return Utils::Edge::FALLING if falling_trigger && (both? || falling?)
+        if is_rising
+          Utils::Edge::RISING
+        elsif is_falling
+          Utils::Edge::FALLING
+        end
       end
 
-      def both?
+      def both_edge?
         @edge == Utils::Edge::BOTH
       end
 
-      def rising?
+      def rising_edge?
         @edge == Utils::Edge::RISING
       end
 
-      def falling?
+      def falling_edge?
         @edge == Utils::Edge::FALLING
       end
+
+      def valid_edge?(edge)
+        valid_rising_edge = (Utils::Edge::RISING == edge) && (both_edge? || rising_edge?)
+        valid_falling_edge = (Utils::Edge::FALLING == edge) && (both_edge? || falling_edge?)
+
+        valid_rising_edge || valid_falling_edge
+      end
     end
   end
 end

data/lib/pi_driver/utils/state.rb

@@ -7,7 +7,7 @@ module PiDriver
       VALID_STATES = [
         LOW,
         HIGH
-      ]
+      ].freeze
     end
   end
 end

data/test/device/mcp23017/test_hardware_address.rb

@@ -26,4 +26,4 @@ class MCP23017HardwareAddressTest < MCP23017Test
 
     @mcp23017.hardware_address.a0 = 0
   end
-end
+end

data/test/device/mcp23017/test_i2c_master.rb

@@ -7,4 +7,4 @@ class MCP23017I2CMasterTest < MCP23017Test
 
     PiDriver::Device::MCP23017.new i2c_master: @i2c_master
   end
-end
+end

data/test/device/mcp23017/test_read.rb

@@ -35,7 +35,7 @@ class MCP23017ReadTest < MCP23017Test
     @i2c_master.expects(:ack).with(nil).in_sequence(seq)
 
     @i2c_master.expects(:restart).with(nil).in_sequence(seq)
-    
+
     @i2c_master.expects(:write).with(0b01001111).in_sequence(seq)
     @i2c_master.expects(:ack).with(nil).in_sequence(seq)
     @i2c_master.expects(:read).with(nil).returns(0b01010101).in_sequence(seq)
@@ -60,4 +60,4 @@ class MCP23017ReadTest < MCP23017Test
     assert_equal 0b01010101, @mcp23017.olata.byte
     assert_equal 0b10101010, @mcp23017.olatb.byte
   end
-end
+end

data/test/device/mcp23017/test_register.rb

@@ -36,7 +36,7 @@ class MCP23017RegisterTest < MCP23017Test
   end
 
   def test_addresses_bank_high
-  	@mcp23017.iocon.bank = 1
+    @mcp23017.iocon.bank = 1
     assert_equal 0x05, @mcp23017.iocon.address
 
     assert_equal 0x00, @mcp23017.iodira.address
@@ -115,4 +115,4 @@ class MCP23017RegisterTest < MCP23017Test
     @mcp23017.iodira.bit7 = 0
     assert_equal 0b01010101, @mcp23017.iodira.byte
   end
-end
+end

data/test/device/mcp23017/test_write.rb

@@ -23,7 +23,7 @@ class MCP23017WriteTest < MCP23017Test
     seq = sequence('multiple')
 
     @i2c_master.expects(:start).with(nil).in_sequence(seq)
-    
+
     @i2c_master.expects(:write).with(0b01001110).in_sequence(seq)
     @i2c_master.expects(:ack).with(nil).in_sequence(seq)
     @i2c_master.expects(:write).with(0x14).in_sequence(seq)
@@ -46,4 +46,4 @@ class MCP23017WriteTest < MCP23017Test
     @mcp23017.olatb.byte = 0b10101010
     @mcp23017.write :olata, :olatb
   end
-end
+end

data/test/i2c_master/test_stop.rb

@@ -4,7 +4,7 @@ class I2CStopTest < I2CMasterTest
   def test_new_i2c_stop
     @sequence = sequence('stop')
     expect_sequence_stop
-    
+
     PiDriver::I2CMaster.new clock_pin: @clock_pin, data_pin: @data_pin
   end
 

data/test/i2c_master/test_write.rb

@@ -18,7 +18,7 @@ class I2CWriteTest < I2CMasterTest
   private
 
   def expect_bit_write(value)
-    if value == 0
+    if value.zero?
       expect_data_pin_to_be_driven
     elsif value == 1
       expect_data_pin_to_be_released

data/test/pin/test_clear_interrupt.rb

@@ -3,7 +3,7 @@ require_relative '../pin_test_helper'
 class PinClearInterruptTest < PinTest
   def test_clear_interrupt
     pin = PiDriver::Pin.new @pin_number
-    interrupt_thread = pin.interrupt { }
+    interrupt_thread = pin.interrupt {}
     interrupt_thread.expects(:kill)
     pin.clear_interrupt
   end

data/test/pin/test_error.rb

@@ -29,7 +29,7 @@ class PinErrorTest < PinTest
   def test_error_edge
     pin = PiDriver::Pin.new @pin_number
     assert_raises ArgumentError do
-      pin.interrupt(:invalid_edge) { }
+      pin.interrupt(:invalid_edge) {}
     end
   end
 end

data/test/pin/test_interrupt.rb

@@ -5,7 +5,7 @@ class PinInterruptTest < PinTest
     pin = PiDriver::Pin.new @pin_number
     interrupt = mock
 
-    [:rising, :falling, :both, :none].each do |edge|
+    %i[rising falling both none].each do |edge|
       begin
         PiDriver::Utils::Interrupt.expects(:new).with(edge).returns(interrupt)
         interrupt.expects(:start)

data/test/test_helper.rb

@@ -1,14 +1,16 @@
+require 'simplecov'
+SimpleCov.start
+
 require 'minitest/autorun'
 require 'pi_driver'
 require 'mocha/minitest'
 
 class TestCase < Minitest::Test
   def timeout(seconds = 0.5)
-    success = false
     started_at = Time.now
-    while !(success = yield) do
+    until (success = yield)
       break if Time.now - started_at > seconds
     end
     success
   end
-end
+end

data/test/utils/test_error.rb

@@ -4,7 +4,6 @@ class UtilsErrorTest < UtilsTest
   def test_error_interrupt_new_block
     assert_raises ArgumentError do
       PiDriver::Utils::Interrupt.new(:rising)
-      timeout { false }
     end
   end
 
@@ -12,7 +11,6 @@ class UtilsErrorTest < UtilsTest
     assert_raises ArgumentError do
       interrupt = PiDriver::Utils::Interrupt.new(:rising) { [0, 1].sample }
       interrupt.start
-      timeout { false }
     end
   end
 

data/test/utils/test_interrupt.rb

@@ -10,7 +10,7 @@ class UtilsInterruptTest < UtilsTest
     thread = mock
     thread.stubs(:abort_on_exception=)
     Thread.expects(:new).returns(thread)
-    interrupt.start { }
+    interrupt.start {}
     timeout 0.1 { false }
   end
 
@@ -100,10 +100,10 @@ class UtilsInterruptTest < UtilsTest
       interrupted = false
       interrupt.start { interrupted = true }
 
-      timeout { }
+      timeout {}
       refute interrupted
     ensure
       interrupt.clear
     end
   end
-end
+end

data/test/utils_test_helper.rb

@@ -1,6 +1,5 @@
 require 'test_helper'
 
 class UtilsTest < TestCase
-  def setup
-  end
+  def setup; end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: pi_driver
 version: !ruby/object:Gem::Version
-  version: 0.0.3
+  version: 0.0.4
 platform: ruby
 authors:
 - Lucas Winningham
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-04-17 00:00:00.000000000 Z
+date: 2018-04-18 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: mocha
@@ -30,6 +30,46 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: 1.4.0
+- !ruby/object:Gem::Dependency
+  name: rubocop
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 0.51.0
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 0.51.0
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 0.51.0
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 0.51.0
+- !ruby/object:Gem::Dependency
+  name: simplecov
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 0.15.1
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 0.15.1
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: 0.15.1
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 0.15.1
 description: Ruby driver for Raspberry Pi with device drivers
 email: lucas.winningham@gmail.com
 executables: