Skip to content

Fix res-adaptr {id} integration tests

Eduardo Silva Araújo requested to merge fix-adaptor-integration-tests into master

We were having some errors here, as the test relied on some subscription data on db that wasn't created each test, and persisted between tests. After lots of runs, it started giving errors. This fixes it.

The error was

Failures:

  1) /adaptor /subscriptions /{id} PUT / is expected to respond with success
     Failure/Error: expect(@response.status).to eq(200)

       expected: 200
            got: 404

With the response body being {"error": "This resource does not have these capabilities: [\"temperature\"]"}.

Other source of errors were connections closed by timeout. Increasing the timeout time may give more stability to our tests. That is the kind of error we avoid by increasing timeout:

Faraday::TimeoutError:
            Net::ReadTimeout with #<TCPSocket:(closed)>

Merge request reports