Addresses #2953 (#5174)

* Addresses #2953

* Added proper test for new error message

* Didn't realize it ran this test as well, whoops

* Implementing changes as suggested by @jennifer-shehane

* Fixing tests and error output. Moved the checks to the start of the get command to ensure we always catch improper options

* Removing issue test since the querying spec covers it

* Using coffescript isArray check

Author: ryan-snyder

packages/driver/src/cy/commands/querying.coffee

@@ -68,7 +68,10 @@ module.exports = (Commands, Cypress, cy, state, config) ->
 
     get: (selector, options = {}) ->
       ctx = @
-
+      
+      if options is null or Array.isArray(options) or typeof options isnt 'object' then return $utils.throwErrByPath "get.invalid_options", {
+          args: { options  }
+      }
       _.defaults(options, {
         retry: true
         withinSubject: cy.state("withinSubject")
@@ -78,7 +81,6 @@ module.exports = (Commands, Cypress, cy, state, config) ->
       })
 
       consoleProps = {}
-
       start = (aliasType) ->
         return if options.log is false
 
@@ -467,4 +469,4 @@ module.exports = (Commands, Cypress, cy, state, config) ->
           cy.state("withinSubject", null)
 
       return subject
-  })
+  })

packages/driver/src/cypress/error_messages.coffee

@@ -292,6 +292,7 @@ module.exports = {
   get:
     alias_invalid: "'{{prop}}' is not a valid alias property. Only 'numbers' or 'all' is permitted."
     alias_zero: "'0' is not a valid alias property. Are you trying to ask for the first response? If so write @{{alias}}.1"
+    invalid_options: "#{cmd('get')} only accepts an options object for its second argument. You passed {{options}}"
 
   getCookie:
     invalid_argument: "#{cmd('getCookie')} must be passed a string argument for name."

packages/driver/test/cypress/integration/commands/querying_spec.coffee

@@ -1173,7 +1173,14 @@ describe "src/cy/commands/querying", ->
           .server()
           .route(/users/, {}).as("getUsers")
           .get("@getUsers.all ")
+      _.each ["", "foo", [], 1, null ], (value) =>
+        it "throws when options property is not an object. Such as: #{value}", (done) ->
+          cy.on "fail", (err) ->
+            expect(err.message).to.include "only accepts an options object for its second argument. You passed #{value}"
+            done()
 
+          cy.get("foobar", value)
+          
       it "logs out $el when existing $el is found even on failure", (done) ->
         button = cy.$$("#button").hide()