add test and fix to calc_theta_delta_bounds for multiconductor case

CHANGELOG.md

@@ -2,7 +2,7 @@ PowerModels.jl Change Log
 =========================
 
 ### Staged
-- nothing
+- Fix implementation of `calc_theta_delta_bounds` when conductor parameter is used (#870)
 
 ### v0.19.9
 - Fix implementation of polynomial costs above quadratic

src/core/data.jl

@@ -73,7 +73,7 @@ function calc_theta_delta_bounds(data::Dict{String,<:Any})
     end
 
     if haskey(data, "conductors")
-        return amin, amax
+        return angle_min, angle_max
     else
         return angle_min[1], angle_max[1]
     end

test/data-mc.jl

@@ -0,0 +1,13 @@
+@testset "test multiconductor data" begin
+
+    @testset "multiconductor calc_theta_delta_bounds" begin
+        data = PowerModels.parse_file("../test/data/matpower/case5.m")
+        data["conductors"] = 1
+
+        theta_lb, theta_ub = calc_theta_delta_bounds(data)
+
+        @test theta_lb[1] <= -2.0
+        @test theta_ub[1] >=  2.0
+    end
+
+end

test/runtests.jl

@@ -54,6 +54,8 @@ include("common.jl")
 
     include("data.jl")
 
+    include("data-mc.jl")
+
     include("data-basic.jl")
 
     include("model.jl")