search

kytos-ng / topology

Kytos Main Topology Network Application (NApp)
https://kytos-ng.github.io/api/topology.html
MIT License
0 stars 7 forks source link

Added endpoint to delete link #179

Closed Alopalao closed 11 months ago

Alopalao commented 11 months ago

Closes #154

Summary

Added an endpoint to delete a link

Local Tests

Added unit tests Deleted a link that was previously removed with link s1 s3 down on mininet console.

Alopalao commented 11 months ago

Other than that, please explore these cases too:

I explored those cases. In all of them the results were as described.

Alopalao commented 11 months ago

Also, let me know if you had positive outcomes with the local/exploratory tests suggested on https://github.com/kytos-ng/topology/pull/179#pullrequestreview-1765343791

Yes, I tested these tests. They all had positive outcomes.

Alopalao commented 11 months ago

Created PR #273 for End-To-End tests

Alopalao commented 11 months ago

I created the issue and I think I am going to be able to finish it for this version.

viniarck commented 11 months ago

I created the issue and I think I am going to be able to finish it for this version.

Cool. Appreciated.