Skip to content

Commit

Permalink
t1029: remove brackets from "grep" tests
Browse files Browse the repository at this point in the history
Problem: The priority jobtap plugin will soon support jobspec-updates of
both banks and projects, and the "jobspec-update" event in the eventlog
holds both of these updates on one line, which breaks the assumption in
t1029-mf-priority-default-bank.t that the jobspec-update of a user's
default bank will be surrounded by brackets {}.

Remove the outer brackets from the tests in
t1029-mf-priority-default-bank.t that grep for a certain bank.
  • Loading branch information
cmoussa1 committed Mar 20, 2024
1 parent b8d960c commit 2f157b8
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions t/t1029-mf-priority-default-bank.t
Original file line number Diff line number Diff line change
Expand Up @@ -55,7 +55,7 @@ test_expect_success 'add user to flux-accounting DB and update plugin' '
test_expect_success 'check that bank was added to jobspec' '
flux job wait-event -f json $jobid priority &&
flux job info $jobid eventlog > eventlog.out &&
grep "{\"attributes.system.bank\":\"account1\"}" eventlog.out &&
grep "\"attributes.system.bank\":\"account1\"" eventlog.out &&
flux job cancel $jobid
'

Expand All @@ -75,7 +75,7 @@ test_expect_success 'successfully submit a job under a default bank' '
jobid=$(flux python ${SUBMIT_AS} 1001 hostname) &&
flux job wait-event -f json $jobid priority &&
flux job info $jobid eventlog > eventlog.out &&
grep "{\"attributes.system.bank\":\"account1\"}" eventlog.out &&
grep "\"attributes.system.bank\":\"account1\"" eventlog.out &&
flux job cancel $jobid
'

Expand All @@ -88,7 +88,7 @@ test_expect_success 'successfully submit a job under the new default bank' '
jobid=$(flux python ${SUBMIT_AS} 1001 hostname) &&
flux job wait-event -f json $jobid priority &&
flux job info $jobid eventlog > eventlog.out &&
grep "{\"attributes.system.bank\":\"account2\"}" eventlog.out &&
grep "\"attributes.system.bank\":\"account2\"" eventlog.out &&
flux job cancel $jobid
'

Expand Down

0 comments on commit 2f157b8

Please sign in to comment.