diff --git a/internal/manager/job_compilers/js_globals.go b/internal/manager/job_compilers/js_globals.go index 2d36dd77..de9dddf9 100644 --- a/internal/manager/job_compilers/js_globals.go +++ b/internal/manager/job_compilers/js_globals.go @@ -41,7 +41,7 @@ type ErrInvalidRange struct { func (e ErrInvalidRange) Error() string { if e.err != nil { - return fmt.Sprintf("invalid range \"%v\": %s (%s)", e.Range, e.Message, e.Error()) + return fmt.Sprintf("invalid range \"%v\": %s (%s)", e.Range, e.Message, e.err.Error()) } return fmt.Sprintf("invalid range \"%v\": %s", e.Range, e.Message) } diff --git a/internal/manager/job_compilers/js_globals_test.go b/internal/manager/job_compilers/js_globals_test.go index b0abb8c3..9ce7eb06 100644 --- a/internal/manager/job_compilers/js_globals_test.go +++ b/internal/manager/job_compilers/js_globals_test.go @@ -42,6 +42,12 @@ func TestFrameChunkerHappyExtraWhitespace(t *testing.T) { assert.Equal(t, []string{"1-4", "5-8", "9,10,20,21", "22-25", "40"}, chunks) } +func TestFrameChunkerUnhappy(t *testing.T) { + _, err := jsFrameChunker(" 1 10", 4) + assert.Error(t, err) + assert.Contains(t, err.Error(), "1 10") +} + func TestFrameRangeExplode(t *testing.T) { frames, err := frameRangeExplode("1..10,20..25,40") assert.NoError(t, err)