tests: Fix ACS freqlist testing
The freqlist parameter uses comma, not space, separated values. Add another test case for checking freqlist values that do not use the preferred channel numbers to make sure the frequency list is actually used as a constraint for selecting the channel. Signed-off-by: Jouni Malinen <j@w1.fi>
This commit is contained in:
parent
025ab330b4
commit
87c8e86da9
1 changed files with 10 additions and 3 deletions
|
@ -90,16 +90,23 @@ def test_ap_acs_chanlist(dev, apdev):
|
||||||
|
|
||||||
def test_ap_acs_freqlist(dev, apdev):
|
def test_ap_acs_freqlist(dev, apdev):
|
||||||
"""Automatic channel selection with freqlist set"""
|
"""Automatic channel selection with freqlist set"""
|
||||||
|
run_ap_acs_freqlist(dev, apdev, [2412, 2437, 2462])
|
||||||
|
|
||||||
|
def test_ap_acs_freqlist2(dev, apdev):
|
||||||
|
"""Automatic channel selection with freqlist set"""
|
||||||
|
run_ap_acs_freqlist(dev, apdev, [2417, 2432, 2457])
|
||||||
|
|
||||||
|
def run_ap_acs_freqlist(dev, apdev, freqlist):
|
||||||
force_prev_ap_on_24g(apdev[0])
|
force_prev_ap_on_24g(apdev[0])
|
||||||
params = hostapd.wpa2_params(ssid="test-acs", passphrase="12345678")
|
params = hostapd.wpa2_params(ssid="test-acs", passphrase="12345678")
|
||||||
params['channel'] = '0'
|
params['channel'] = '0'
|
||||||
params['freqlist'] = '2412 2437 2462'
|
params['freqlist'] = ','.join([str(x) for x in freqlist])
|
||||||
hapd = hostapd.add_ap(apdev[0], params, wait_enabled=False)
|
hapd = hostapd.add_ap(apdev[0], params, wait_enabled=False)
|
||||||
wait_acs(hapd)
|
wait_acs(hapd)
|
||||||
|
|
||||||
freq = int(hapd.get_status_field("freq"))
|
freq = int(hapd.get_status_field("freq"))
|
||||||
if freq not in [2412, 2437, 2462]:
|
if freq not in freqlist:
|
||||||
raise Exception("Unexpected frequency: " + freq)
|
raise Exception("Unexpected frequency: %d" % freq)
|
||||||
|
|
||||||
dev[0].connect("test-acs", psk="12345678", scan_freq=str(freq))
|
dev[0].connect("test-acs", psk="12345678", scan_freq=str(freq))
|
||||||
|
|
||||||
|
|
Loading…
Reference in a new issue