Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(28)

Issue 6484016: Add test for SSID that is too long (Closed)

Created:
9 years, 10 months ago by Paul Stewart
Modified:
9 years, 7 months ago
Reviewers:
Sam Leffler
CC:
chromium-os-reviews_chromium.org, truty+cc_chromium.org, sosa+cc_chromium.org, seano+cc_chromium.org, ericli, petkov+cc_chromium.org
Visibility:
Public.

Description

Add test for SSID that is too long Also account for change to error message for invalid passphrase. Also abort immediately if Connect() call fails. This needs to be pushed at the same time as 6486022 TEST=SecMat,MatFunc BUG=chromium-os:11668 Change-Id: Ia2cac6b9db15a7cab5f11a705b93fb47812c0f35 Committed: http://chrome-svn/viewvc/chromeos?view=rev&revision=54c1371

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+13 lines, -6 lines) Patch
M server/site_tests/network_WiFiMatFunc/090_SSID_Length_Limit View 1 chunk +4 lines, -0 lines 0 comments Download
M server/site_tests/network_WiFiSecMat/021CheckWPA_BadPassphraseLength View 5 chunks +6 lines, -6 lines 0 comments Download
M server/site_wlan_connect.py View 1 chunk +3 lines, -0 lines 0 comments Download

Messages

Total messages: 2 (0 generated)
Paul Stewart
9 years, 10 months ago (2011-02-11 00:46:00 UTC) #1
Sam Leffler
9 years, 10 months ago (2011-02-12 00:06:26 UTC) #2
LGTM

Powered by Google App Engine
This is Rietveld 408576698