Refactor ChannelsArray lookup by channel id and uid (#177)

lib/openhab/core/things/proxy.rb

@@ -29,7 +29,7 @@ class Proxy < Delegator
         #
         # @return [Array] channels
         def channels
-          Thing::ChannelsArray.new(super.to_a)
+          Thing::ChannelsArray.new(self, super.to_a)
         end
 
         #

lib/openhab/core/things/thing.rb

@@ -64,13 +64,17 @@ module Thing
         # Array wrapper class to allow searching a list of channels
         # by channel id
         class ChannelsArray < Array
+          def initialize(thing, array)
+            super(array)
+            @thing = thing
+          end
+
           # Allows indexing by both integer as an array or channel id acting like a hash.
-          # @param [Integer, String] index Numeric index or string channel id to search for.
+          # @param [Integer, String, ChannelUID] index
+          #   Numeric index, string channel id, or a {ChannelUID} to search for.
           def [](index)
-            if index.respond_to?(:to_str)
-              key = index.to_str
-              return find { |channel| channel.uid.id == key }
-            end
+            return @thing.get_channel(index) if index.is_a?(ChannelUID)
+            return @thing.get_channel(index.to_str) if index.respond_to?(:to_str)
 
             super
           end

spec/openhab/core/things/thing_spec.rb

@@ -54,6 +54,13 @@
       it "returns its thing" do
         expect(thing.channels["phase#name"].thing).to be thing
       end
+
+      it "supports lookup by channel UID" do
+        channel_id = "phase#name"
+        channel_uid = org.openhab.core.thing.ChannelUID.new(thing.uid, channel_id)
+        expect(thing.channels[channel_uid]).not_to be_nil
+        expect(thing.channels[channel_uid]).to be thing.channels[channel_id]
+      end
     end
   end