Skip to content

Commit

Permalink
Merge pull request #14642 from juanvallejo/jvallejo/fix-describer-tes…
Browse files Browse the repository at this point in the history
…t-flake

update describer test to create event directly
  • Loading branch information
smarterclayton authored Jun 15, 2017
2 parents fcccc62 + b08065f commit 811a267
Showing 1 changed file with 18 additions and 2 deletions.
20 changes: 18 additions & 2 deletions test/cmd/describer.sh
Original file line number Diff line number Diff line change
Expand Up @@ -12,8 +12,24 @@ trap os::test::junit::reconcile_output EXIT

os::test::junit::declare_suite_start "cmd/describe"
# This test validates non-duplicate errors when describing an existing resource without a defined describer
os::cmd::expect_success 'oc new-app node'
os::cmd::try_until_success 'eventnum=$(oc get events | wc -l) && [[ $eventnum -gt 1 ]]'
os::cmd::expect_success 'oc create -f - << __EOF__
{
"apiVersion": "v1",
"involvedObject": {
"apiVersion": "v1",
"kind": "Pod",
"name": "test-pod",
"namespace": "cmd-describer"
},
"kind": "Event",
"message": "test message",
"metadata": {
"name": "test-event"
}
}
__EOF__
'
os::cmd::try_until_success 'eventnum=$(oc get events | wc -l) && [[ $eventnum -gt 0 ]]'
# resources without describers get a default
os::cmd::expect_success_and_text 'oc describe events' 'Namespace: cmd-describer'

Expand Down

0 comments on commit 811a267

Please sign in to comment.