kemko/nomad
1
0
Fork 0
mirror of https://github.com/kemko/nomad.git synced 2026-01-02 00:15:43 +03:00
Code Issues Packages Projects Releases Wiki Activity

test: use Len instead of Equal on system and sysbatch node constraint tests

Browse Source
This commit is contained in:
Luiz Aoqui
2021-09-02 11:36:02 -04:00
parent 044ce3c994
commit 092abb8cd4
2 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
scheduler/scheduler_sysbatch_test.go
View File

@@ -934,7 +934,7 @@ func TestSysBatch_JobConstraint_AddNode(t *testing.T) {
require.Equal(t, "complete", h.Evals[1].Status)
// Ensure no new plans
require.Equal(t, 1, len(h.Plans))
require.Len(t, h.Plans, 1)
// Ensure all NodeAllocations are from first Eval
for _, allocs := range h.Plans[0].NodeAllocation {
@@ -966,7 +966,7 @@ func TestSysBatch_JobConstraint_AddNode(t *testing.T) {
require.Equal(t, "complete", h.Evals[2].Status)
// Ensure `groupA` fails to be placed due to its constraint, but `groupB` doesn't
require.Equal(t, 1, len(h.Evals[2].FailedTGAllocs))
require.Len(t, h.Evals[2].FailedTGAllocs, 1)
require.Contains(t, h.Evals[2].FailedTGAllocs, "groupA")
require.NotContains(t, h.Evals[2].FailedTGAllocs, "groupB")

4
scheduler/scheduler_system_test.go
View File

@@ -1264,7 +1264,7 @@ func TestSystemSched_JobConstraint_AddNode(t *testing.T) {
require.Equal(t, "complete", h.Evals[1].Status)
// Ensure no new plans
require.Equal(t, 1, len(h.Plans))
require.Len(t, h.Plans, 1)
// Ensure all NodeAllocations are from first Eval
for _, allocs := range h.Plans[0].NodeAllocation {
@@ -1296,7 +1296,7 @@ func TestSystemSched_JobConstraint_AddNode(t *testing.T) {
require.Equal(t, "complete", h.Evals[2].Status)
// Ensure `groupA` fails to be placed due to its constraint, but `groupB` doesn't
require.Equal(t, 1, len(h.Evals[2].FailedTGAllocs))
require.Len(t, h.Evals[2].FailedTGAllocs, 1)
require.Contains(t, h.Evals[2].FailedTGAllocs, "groupA")
require.NotContains(t, h.Evals[2].FailedTGAllocs, "groupB")