spec: model/datafixup linter fix, fixes SD-2020

correctly whitelist things that chain off of whitelisted methods, e.g.
connection

Change-Id: I38ec4699b324d0e5d26de6bde36d841fa696d7e5
Reviewed-on: https://gerrit.instructure.com/100186
Tested-by: Jenkins
Reviewed-by: Cody Cutrer <cody@instructure.com>
Product-Review: Jon Jensen <jon@instructure.com>
QA-Review: Jon Jensen <jon@instructure.com>

Author: jenseng

gems/rubocop-canvas/lib/rubocop_canvas/cops/migration/model_behavior.rb

@@ -32,7 +32,7 @@ def on_send(node)
           parts = receiver + methods
           return true if whitelist.any? do |whitelist_parts|
             if whitelist_parts[0] == :*
-              methods.last == whitelist_parts.last
+              methods.any? { |m| m == whitelist_parts.last }
             else
               parts[0, whitelist_parts.length] == whitelist_parts
             end

gems/rubocop-canvas/spec/rubocop/cop/migration/model_behavior_spec.rb

@@ -7,7 +7,8 @@
         "Whitelist" => [
           "Migrations::FooFix",
           "*.update_all",
-          "*.delete_all"
+          "*.delete_all",
+          "*.connection"
         ]
       }
     )
@@ -26,6 +27,17 @@ def up
     expect(subject.offenses.size).to eq(0)
   end
 
+  it "should find no offenses when calling methods on whitelisted classes/methods" do
+    inspect_source(subject, %{
+      class Foo < ActiveRecord::Migration
+        def up
+          User.connection.execute "YOLO"
+        end
+      end
+    })
+    expect(subject.offenses.size).to eq(0)
+  end
+
   it "should find no offenses when calling whitelisted methods" do
     inspect_source(subject, %{
       class Foo < ActiveRecord::Migration