New issue
Advanced search Search tips
Note: Color blocks (like or ) mean that a user may not be available. Tooltip shows the reason.

Issue 748702 link

Starred by 1 user

Issue metadata

Status: Archived
Owner:
Closed: Jul 2017
Cc:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 1
Type: Feature



Sign in to add a comment

Allow users to configure what wifi access point tests should use

Project Member Reported by haddowk@chromium.org, Jul 25 2017

Issue description


Certain CTS tests ( and possibly others ) need to connect to a wifi access point to run correctly.

Allow the user to configure an AP name and password that will get passed to the test so it can (optionally) use it to connect to wifi.
 
Project Member

Comment 1 by bugdroid1@chromium.org, Jul 27 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/bbc45d971e3442c58d230829fdb6734b4a619324

commit bbc45d971e3442c58d230829fdb6734b4a619324
Author: Keith Haddow <haddowk@chromium.org>
Date: Thu Jul 27 03:03:58 2017

[moblab] Add UI option to specify AP name/password DUTs shouls use.

Certain CTS tests require the DUT to connect to a wifi AP.

During the setup of the moblab the user can specify the AP name
and password and that will be passed to all tests for it to
(optionally) connect to the wifi AP.

Example UI changes at http://100.96.59.141/moblab_setup

BUG= chromium:748702 
TEST=local testing on moblab

Change-Id: I48d2b3e6790995e54adaad98ab6c3b1f64c06afd
Reviewed-on: https://chromium-review.googlesource.com/584861
Commit-Ready: Keith Haddow <haddowk@chromium.org>
Tested-by: Keith Haddow <haddowk@chromium.org>
Reviewed-by: Keith Haddow <haddowk@chromium.org>
Reviewed-by: Michael Tang <ntang@chromium.org>

[modify] https://crrev.com/bbc45d971e3442c58d230829fdb6734b4a619324/frontend/client/src/autotest/moblab/rpc/MoblabRpcCallbacks.java
[modify] https://crrev.com/bbc45d971e3442c58d230829fdb6734b4a619324/frontend/afe/moblab_rpc_interface.py
[add] https://crrev.com/bbc45d971e3442c58d230829fdb6734b4a619324/frontend/client/src/autotest/moblab/wizard/WifiCard.java
[modify] https://crrev.com/bbc45d971e3442c58d230829fdb6734b4a619324/frontend/client/src/autotest/moblab/rpc/MoblabRpcHelper.java
[add] https://crrev.com/bbc45d971e3442c58d230829fdb6734b4a619324/frontend/client/src/autotest/moblab/rpc/WifiInfo.java
[modify] https://crrev.com/bbc45d971e3442c58d230829fdb6734b4a619324/frontend/client/src/autotest/moblab/ConfigWizardView.java

Project Member

Comment 2 by bugdroid1@chromium.org, Jul 27 2017

Labels: merge-merged-release-R61-9765.B
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/8ded543ee8ea212ab5476776f15da7bb7045d8b2

commit 8ded543ee8ea212ab5476776f15da7bb7045d8b2
Author: Keith Haddow <haddowk@chromium.org>
Date: Thu Jul 27 17:12:08 2017

[moblab] Add UI option to specify AP name/password DUTs shouls use.

Certain CTS tests require the DUT to connect to a wifi AP.

During the setup of the moblab the user can specify the AP name
and password and that will be passed to all tests for it to
(optionally) connect to the wifi AP.

Example UI changes at http://100.96.59.141/moblab_setup

BUG= chromium:748702 
TEST=local testing on moblab

Change-Id: I48d2b3e6790995e54adaad98ab6c3b1f64c06afd
Reviewed-on: https://chromium-review.googlesource.com/584861
Commit-Ready: Keith Haddow <haddowk@chromium.org>
Tested-by: Keith Haddow <haddowk@chromium.org>
Reviewed-by: Keith Haddow <haddowk@chromium.org>
Reviewed-by: Michael Tang <ntang@chromium.org>
(cherry picked from commit bbc45d971e3442c58d230829fdb6734b4a619324)
Reviewed-on: https://chromium-review.googlesource.com/588487
Commit-Queue: Keith Haddow <haddowk@chromium.org>

[modify] https://crrev.com/8ded543ee8ea212ab5476776f15da7bb7045d8b2/frontend/client/src/autotest/moblab/rpc/MoblabRpcCallbacks.java
[modify] https://crrev.com/8ded543ee8ea212ab5476776f15da7bb7045d8b2/frontend/afe/moblab_rpc_interface.py
[add] https://crrev.com/8ded543ee8ea212ab5476776f15da7bb7045d8b2/frontend/client/src/autotest/moblab/wizard/WifiCard.java
[modify] https://crrev.com/8ded543ee8ea212ab5476776f15da7bb7045d8b2/frontend/client/src/autotest/moblab/rpc/MoblabRpcHelper.java
[add] https://crrev.com/8ded543ee8ea212ab5476776f15da7bb7045d8b2/frontend/client/src/autotest/moblab/rpc/WifiInfo.java
[modify] https://crrev.com/8ded543ee8ea212ab5476776f15da7bb7045d8b2/frontend/client/src/autotest/moblab/ConfigWizardView.java

Project Member

Comment 3 by bugdroid1@chromium.org, Jul 27 2017

Labels: merge-merged-release-R60-9592.B
The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/e8af1693c589edf5601ae9ccc27c0d59949226c3

commit e8af1693c589edf5601ae9ccc27c0d59949226c3
Author: Keith Haddow <haddowk@chromium.org>
Date: Thu Jul 27 17:12:18 2017

[moblab] Add UI option to specify AP name/password DUTs shouls use.

Certain CTS tests require the DUT to connect to a wifi AP.

During the setup of the moblab the user can specify the AP name
and password and that will be passed to all tests for it to
(optionally) connect to the wifi AP.

Example UI changes at http://100.96.59.141/moblab_setup

BUG= chromium:748702 
TEST=local testing on moblab

Change-Id: I48d2b3e6790995e54adaad98ab6c3b1f64c06afd
Reviewed-on: https://chromium-review.googlesource.com/584861
Commit-Ready: Keith Haddow <haddowk@chromium.org>
Tested-by: Keith Haddow <haddowk@chromium.org>
Reviewed-by: Keith Haddow <haddowk@chromium.org>
Reviewed-by: Michael Tang <ntang@chromium.org>
(cherry picked from commit bbc45d971e3442c58d230829fdb6734b4a619324)
Reviewed-on: https://chromium-review.googlesource.com/588488
Commit-Queue: Keith Haddow <haddowk@chromium.org>

[modify] https://crrev.com/e8af1693c589edf5601ae9ccc27c0d59949226c3/frontend/client/src/autotest/moblab/rpc/MoblabRpcCallbacks.java
[modify] https://crrev.com/e8af1693c589edf5601ae9ccc27c0d59949226c3/frontend/afe/moblab_rpc_interface.py
[add] https://crrev.com/e8af1693c589edf5601ae9ccc27c0d59949226c3/frontend/client/src/autotest/moblab/wizard/WifiCard.java
[modify] https://crrev.com/e8af1693c589edf5601ae9ccc27c0d59949226c3/frontend/client/src/autotest/moblab/rpc/MoblabRpcHelper.java
[add] https://crrev.com/e8af1693c589edf5601ae9ccc27c0d59949226c3/frontend/client/src/autotest/moblab/rpc/WifiInfo.java
[modify] https://crrev.com/e8af1693c589edf5601ae9ccc27c0d59949226c3/frontend/client/src/autotest/moblab/ConfigWizardView.java

Project Member

Comment 4 by bugdroid1@chromium.org, Jul 29 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/5069755ba0d0b8ff58cbd80531fde69daf45d2fa

commit 5069755ba0d0b8ff58cbd80531fde69daf45d2fa
Author: Keith Haddow <haddowk@chromium.org>
Date: Sat Jul 29 17:37:41 2017

[moblab] Enable the precondition suite on the UI.

Add the precondition suite to the UI and also pass the wifi test
parameters along to all suite runs.

BUG= chromium:748702 
TEST=local moblab testing

Change-Id: Id0cba8dbf5067fd5ef28d6057afa0e6663630524
Reviewed-on: https://chromium-review.googlesource.com/592072
Commit-Ready: Keith Haddow <haddowk@chromium.org>
Tested-by: Keith Haddow <haddowk@chromium.org>
Reviewed-by: Keith Haddow <haddowk@chromium.org>
Reviewed-by: Michael Tang <ntang@chromium.org>

[modify] https://crrev.com/5069755ba0d0b8ff58cbd80531fde69daf45d2fa/frontend/client/src/autotest/moblab/SuiteRunnerView.java
[modify] https://crrev.com/5069755ba0d0b8ff58cbd80531fde69daf45d2fa/frontend/afe/moblab_rpc_interface.py
[modify] https://crrev.com/5069755ba0d0b8ff58cbd80531fde69daf45d2fa/frontend/client/src/autotest/moblab/wizard/WifiCard.java

Project Member

Comment 5 by bugdroid1@chromium.org, Jul 31 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/c5206bc8bded8f669617ded771a8d868b9185ea3

commit c5206bc8bded8f669617ded771a8d868b9185ea3
Author: Keith Haddow <haddowk@chromium.org>
Date: Mon Jul 31 17:25:08 2017

[moblab] Fix issue where wifi password could not be reset to None

If you set the password, then went back and tried to remove the
password it was never updated.

BUG= chromium:748702 
TEST=local moblab testing

Change-Id: Id33e8c495ccff9395cf5037cf76246834a53ed20
Reviewed-on: https://chromium-review.googlesource.com/593187
Reviewed-by: Keith Haddow <haddowk@chromium.org>
Reviewed-by: Michael Tang <ntang@chromium.org>
Commit-Queue: Keith Haddow <haddowk@chromium.org>
Tested-by: Keith Haddow <haddowk@chromium.org>

[modify] https://crrev.com/c5206bc8bded8f669617ded771a8d868b9185ea3/frontend/client/src/autotest/moblab/rpc/WifiInfo.java

Project Member

Comment 6 by bugdroid1@chromium.org, Jul 31 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/b7363cb40fdb59d950013ef0415d5deb44525cbc

commit b7363cb40fdb59d950013ef0415d5deb44525cbc
Author: Keith Haddow <haddowk@chromium.org>
Date: Mon Jul 31 17:25:18 2017

[moblab] Enable the precondition suite on the UI.

Add the precondition suite to the UI and also pass the wifi test
parameters along to all suite runs.

BUG= chromium:748702 
TEST=local moblab testing

Change-Id: Id0cba8dbf5067fd5ef28d6057afa0e6663630524
Reviewed-on: https://chromium-review.googlesource.com/592634
Reviewed-by: Keith Haddow <haddowk@chromium.org>
Reviewed-by: Michael Tang <ntang@chromium.org>
Commit-Queue: Keith Haddow <haddowk@chromium.org>
Tested-by: Keith Haddow <haddowk@chromium.org>

[modify] https://crrev.com/b7363cb40fdb59d950013ef0415d5deb44525cbc/frontend/client/src/autotest/moblab/SuiteRunnerView.java
[modify] https://crrev.com/b7363cb40fdb59d950013ef0415d5deb44525cbc/frontend/afe/moblab_rpc_interface.py
[modify] https://crrev.com/b7363cb40fdb59d950013ef0415d5deb44525cbc/frontend/client/src/autotest/moblab/wizard/WifiCard.java

Status: Fixed (was: Assigned)
Project Member

Comment 8 by bugdroid1@chromium.org, Aug 1 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/94dd4622cdad3a271375d0a0d11d9ce162b44bb9

commit 94dd4622cdad3a271375d0a0d11d9ce162b44bb9
Author: Keith Haddow <haddowk@chromium.org>
Date: Tue Aug 01 09:39:58 2017

[moblab] Fix issue where wifi password could not be reset to None

If you set the password, then went back and tried to remove the
password it was never updated.

BUG= chromium:748702 
TEST=local moblab testing

Change-Id: Id33e8c495ccff9395cf5037cf76246834a53ed20
Reviewed-on: https://chromium-review.googlesource.com/592850
Commit-Ready: Keith Haddow <haddowk@chromium.org>
Tested-by: Keith Haddow <haddowk@chromium.org>
Reviewed-by: Keith Haddow <haddowk@chromium.org>
Reviewed-by: Michael Tang <ntang@chromium.org>

[modify] https://crrev.com/94dd4622cdad3a271375d0a0d11d9ce162b44bb9/frontend/client/src/autotest/moblab/rpc/WifiInfo.java

Project Member

Comment 9 by bugdroid1@chromium.org, Aug 1 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/195ab06a43441e5b7518f4ab829383e7b743d207

commit 195ab06a43441e5b7518f4ab829383e7b743d207
Author: Keith Haddow <haddowk@chromium.org>
Date: Tue Aug 01 16:29:33 2017

[moblab] Enable the precondition suite on the UI.

Add the precondition suite to the UI and also pass the wifi test
parameters along to all suite runs.

BUG= chromium:748702 
TEST=local moblab testing

Change-Id: Id0cba8dbf5067fd5ef28d6057afa0e6663630524
Reviewed-on: https://chromium-review.googlesource.com/592636
Reviewed-by: Keith Haddow <haddowk@chromium.org>
Reviewed-by: Michael Tang <ntang@chromium.org>
Commit-Queue: Keith Haddow <haddowk@chromium.org>
Tested-by: Keith Haddow <haddowk@chromium.org>

[modify] https://crrev.com/195ab06a43441e5b7518f4ab829383e7b743d207/frontend/client/src/autotest/moblab/SuiteRunnerView.java
[modify] https://crrev.com/195ab06a43441e5b7518f4ab829383e7b743d207/frontend/afe/moblab_rpc_interface.py
[modify] https://crrev.com/195ab06a43441e5b7518f4ab829383e7b743d207/frontend/client/src/autotest/moblab/wizard/WifiCard.java

Project Member

Comment 10 by bugdroid1@chromium.org, Aug 1 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/third_party/autotest/+/79a07a17ed6397c891f99eea840c75d17175fa53

commit 79a07a17ed6397c891f99eea840c75d17175fa53
Author: Keith Haddow <haddowk@chromium.org>
Date: Tue Aug 01 16:31:29 2017

[moblab] Fix issue where wifi password could not be reset to None

If you set the password, then went back and tried to remove the
password it was never updated.

BUG= chromium:748702 
TEST=local moblab testing

Change-Id: Id33e8c495ccff9395cf5037cf76246834a53ed20
Reviewed-on: https://chromium-review.googlesource.com/594511
Reviewed-by: Keith Haddow <haddowk@chromium.org>
Reviewed-by: Michael Tang <ntang@chromium.org>
Commit-Queue: Keith Haddow <haddowk@chromium.org>
Tested-by: Keith Haddow <haddowk@chromium.org>

[modify] https://crrev.com/79a07a17ed6397c891f99eea840c75d17175fa53/frontend/client/src/autotest/moblab/rpc/WifiInfo.java

Comment 11 by dchan@chromium.org, Jan 22 2018

Status: Archived (was: Fixed)

Sign in to add a comment