Econify/graphql-rest-router

Update Broken Tests To Passing

Mk-Etlinger opened this issue · 0 comments

There's currently 4 failing tests that need to be fixed.

Here's the output from running npm run test:

  1. Route
    #constructor
    with valid arguments
    when using minimal configuration
    should set path to the operation name:

    AssertionError: expected '/GetUserById' to equal 'GetUserById'

    • expected - actual

    -/GetUserById
    +GetUserById

    at Context. (test/Route.test.js:20:18)
    at processImmediate (internal/timers.js:456:21)

  2. Route
    private#setOperationName
    when given an operation name that exists in the schema
    should set operation variables:
    AssertionError: expected { Object (id) } to deeply equal [ Array(1) ]
    at Context. (test/Route.test.js:114:16)
    at processImmediate (internal/timers.js:456:21)

  3. Route
    #path
    should use the operation name as the default path:

    AssertionError: expected '/GetUserById' to equal 'GetUserById'

    • expected - actual

    -/GetUserById
    +GetUserById

    at Context. (test/Route.test.js:124:14)
    at processImmediate (internal/timers.js:456:21)

  4. Router
    #mount
    argument overloading
    should get operation name from configuration if only single argument provided:
    TypeError: operationNameOrMountableItem.withOptions is not a function
    at Router.mount (Router.js:80:58)
    at Context. (test/Router.test.js:67:16)
    at processImmediate (internal/timers.js:456:21)

I need to make sure that I understand the proper passing tests before tackling this one.