test: h>1 with OrthogonalCollocation

Author: franckgaga

src/controller/transcription.jl

@@ -129,9 +129,9 @@ end
 Construct an orthogonal collocation on finite elements [`TranscriptionMethod`](@ref).
 
 Also known as pseudo-spectral method. It supports continuous-time [`NonLinModel`](@ref)s
-only. The `h` argument is the hold order for ``\mathbf{u}``, and the `no` argument, the
-number of collocation points ``n_o``. The decision variable is similar to [`MultipleShooting`](@ref),
-but it also includes the collocation points:
+only. The `h` argument is the hold order for ``\mathbf{u}`` (`0` or `1`), and the `no`
+argument, the number of collocation points ``n_o``. The decision variable is similar to
+[`MultipleShooting`](@ref), but it also includes the collocation points:
 ```math
 \mathbf{Z} = \begin{bmatrix} \mathbf{ΔU} \\ \mathbf{X̂_0} \\ \mathbf{K} \end{bmatrix}
 ```

test/3_test_predictive_control.jl

@@ -877,10 +877,11 @@ end
     @test isa(nmpc15.optim, JuMP.GenericModel{Float64}) # Ipopt does not support Float32
 
     @test_throws ArgumentError NonLinMPC(nonlinmodel)
-    @test_throws ArgumentError NonLinMPC(nonlinmodel, Hp=2, transcription=TrapezoidalCollocation())
-    @test_throws ArgumentError NonLinMPC(nonlinmodel, Hp=2, transcription=TrapezoidalCollocation(2))
     @test_throws ArgumentError NonLinMPC(nonlinmodel, Hp=2, Wy=[1 0;0 1])
+    @test_throws ArgumentError NonLinMPC(nonlinmodel, Hp=2, transcription=TrapezoidalCollocation())
+    @test_throws ArgumentError TrapezoidalCollocation(2)
     @test_throws ArgumentError OrthogonalCollocation(roots=:gausslobatto)
+    @test_throws ArgumentError OrthogonalCollocation(2)
 end
 
 @testitem "NonLinMPC moves and getinfo (LinModel)" setup=[SetupMPCtests] begin