test should use node.id instead of '123'

[Ruby Loo]

In ironic/tests/drivers/test_utils.py, test_get_node_mac_addresses() uses node_id='123'. This value is actually from self.node.id, and the code would be easier to understand and more maintainable if we used self.node.id. (Eg, what if that id value was changed in the future.)

Bonus points - see if this hardcoding is done in other tests and fix as well.

This is a follow-up, from https://review.openstack.org/#/c/78787/3/ironic/tests/drivers/test_utils.py.

[Openstack Gerrit (openstack-gerrit)]

Fix proposed to branch: master
Review: https://review.openstack.org/88433

[Openstack Gerrit (openstack-gerrit)]

Reviewed: https://review.openstack.org/88433
Committed: https://git.openstack.org/cgit/openstack/ironic/commit/?id=b79aeb58a9cef67b471863d4940f33bf4cd044d5
Submitter: Jenkins
Branch: master

commit b79aeb58a9cef67b471863d4940f33bf4cd044d5
Author: Tushar Kalra <email address hidden>
Date: Thu Apr 17 13:45:53 2014 -0700

    Remove hardcoded node id value

    Change-Id: I9e4e74ce3a41e1cd3dd4e0d9ff1d1bde0656527f
    Closes-Bug: 1306214