Support get_mac_address_table() on L2NG switches

[mirceaulinic]

Hi @bkeifer - I had to close https://github.com/napalm-automation/napalm-junos/pull/199 (your branch was very old and there were many conflicts, so I gave up and picked your commits).

I have added four more commits to continue your work:

• Because you are looking under two different facts to determine what RPC to execute, see: https://github.com/napalm-automation/napalm-junos/blob/c710b0a701d8cb509a098ccb74b5a0ecc7b6199e/napalm_junos/junos.py#L1289-L1290 You need both facts specified in the static facts file under the test case: https://github.com/napalm-automation/napalm-junos/commit/2868662a70da2fc1db084599baf684fc5bab996f.
• The junos namespace should be removed from the input XML: https://github.com/napalm-automation/napalm-junos/commit/a52a25c5a51f9de6ef0387fc512928c0153e338c.
• I dropped the static field https://github.com/napalm-automation/napalm-junos/commit/fb4a2424cbea4da8eecd6c185a15a30bed4ba319 -- I'm not sure where you copied that from.
• The expected_result.json must be JSON & respect the correct output according to the mocked data you provided: https://github.com/napalm-automation/napalm-junos/commit/40da0b40812d3e62378e0d2844786c54a2810702

Let me know if you have any questions, I hope my information will help you in the future. Thanks for your PR, this enhancement will be included in the next release, 0.12.1.

[coveralls]

Coverage increased (+0.04%) to 78.804% when pulling 40da0b40812d3e62378e0d2844786c54a2810702 on mirceaulinic:mac-l2ng-switches into 3a73f1b8519a846c960361784ce0295b1c354b3d on napalm-automation:develop.

[mirceaulinic]

@ktbyers @dbarrosop @bkeifer can you review please?

[dbarrosop]

Yay! I didn't have to ask for tests ;)

+1