[show] invalid acl table output with single port

[d-dashkov]

- What I did
Fixed the bug "Observing the binding field is not proper while executing "show acl table" command"
fixes sonic-net/SONiC#305

- How I did it
Added changes to the function show_table in acl_lader.
Added the checking of a value type for the ports. If the value is a string then we print it as is in other cases we split and sort the values.

- How to verify it
Follow the description in the bug.

1.Create an acl :

"ACL_TABLE": {
	"1": {
		"stage": "INGRESS",
		"type": "MIRROR",
		"policy_desc": "Mirror Session",
		"ports": "Ethernet16"
	}
}

2. $ config reload -y
3. $ show acl table

- Previous command output (if the output of a command-line utility has changed)

Name    Type    Binding    Description     Stage
------  ------  ---------  --------------  -------
1       MIRROR  1          Mirror Session  ingress
                6
                E
                e
                e
                h
                n
                r
                t
                t

- New command output (if the output of a command-line utility has changed)

  Name  Type    Binding     Description     Stage
------  ------  ----------  --------------  -------
     1  MIRROR  Ethernet16  Mirror Session  ingress

[daall]

See question. Also, please add a new unit test case(s) to tests/acl_loader_test.py.

[daall]

Have you tried the case where "ports" is a comma-separated string like "Ethernet0,Ethernet1,..."? Does this come back as a string or a list?

[d-dashkov]

@daall
I`ve tried cases with different acl command and they all work fine:

#sudo config acl add table test_1 L3 -p Ethernet0,Ethernet4 -s ingress
"test_1": {
	"policy_desc": "Mirror Session",
	"ports": [
		"Ethernet0",
		"Ethernet4"
	],
	"stage": "INGRESS",
	"type": "MIRROR"
}

#sudo config acl add table test_2 L3 -p "Ethernet0,Ethernet4" -s ingress
"test_2": {
	"policy_desc": "Mirror Session",
	"ports": [
		"Ethernet0",
		"Ethernet4"
	],
	"stage": "INGRESS",
	"type": "MIRROR"
}

#sudo config acl add table test_single L3 -p Ethernet0 -s ingress
"test_single": {
	"policy_desc": "Mirror Session",
	"ports": [
		"Ethernet0"
	],
	"stage": "INGRESS",
	"type": "MIRROR"
}

This bug happens only when I manually write the wrong value in config db.json, as this stated in issue(instead of list, write single str port)

"1": {
	"stage": "INGRESS",
	"type": "MIRROR",
	"policy_desc": "Mirror Session",
	"ports": "Ethernet16" <= incorrect value
}