test: fix pummel test failures (!6012) · Merge requests · Rodrigo Test / Test Group-nodejs / node · GitLab

Do not update/delete: Banner broadcast message test data

Do not update/delete: Notification broadcast message test data

Rodrigo Muino Tomonari requested to merge github/fork/Trott/noodle into master Apr 02, 2016

Pull Request check-list

Please make sure to review and check all of these items:

Does make -j8 test (UNIX) or vcbuild test nosign (Windows) pass with this change (including linting)?
Is the commit message formatted according to [CONTRIBUTING.md][0]?

Affected core subsystem(s)

test, net, crypto

Description of change

A handful of tests in test/pummel were failing due to undefined variables.

The tests in pummel are not run in CI or otherwise exercised regularly so these failures can go unnoticed for a long time.