atlanticwave-sdx / sdx-controller

Central Controller for AtlanticWave SDX.
https://www.atlanticwave-sdx.net
MIT License
1 stars 3 forks source link

Add a test for `GET /connection/:connection_id` #271

Closed sajith closed 2 months ago

sajith commented 2 months ago

Resolves #270. Changes:

Although #270 is assigned to @italovalcy, I made an attempt at it. ;-)

coveralls commented 2 months ago

Pull Request Test Coverage Report for Build 9112570287

Details


Totals Coverage Status
Change from base Build 9112522793: 0.1%
Covered Lines: 725
Relevant Lines: 1433

💛 - Coveralls
sajith commented 2 months ago

The interesting thing about this particular untidiest is why it didn't fail before? I was concerned if I broke sth I didn't notice.

I think GET /connection/:connection_id was a stub that returned success. And then we added a DB lookup.