atlanticwave-sdx / pce

Path Computation Element for AtlanticWave SDX.
https://www.atlanticwave-sdx.net
0 stars 3 forks source link

expand allowed vlan_range format #190

Closed YufengXin closed 6 months ago

YufengXin commented 6 months ago
  1. "min-max"
  2. min
  3. list [min, max]
  4. tuple (min, max)

@sajith, I did a quick fix to allow all above format for vlan_range, esp, the tuple (), along with an unittest.

Well, port_handler._validate_vlan_range() would need to add the tuple too, and the unit tests cases.....

See if we all agree on this.

coveralls commented 6 months ago

Pull Request Test Coverage Report for Build 9295641916

Details


Changes Missing Coverage Covered Lines Changed/Added Lines %
src/sdx_pce/topology/temanager.py 19 20 95.0%
<!-- Total: 19 20 95.0% -->
Totals Coverage Status
Change from base Build 9289019017: 0.2%
Covered Lines: 1055
Relevant Lines: 1226

💛 - Coveralls