1034522 - B2G Emulator: Add console command - sim status

Status: RESOLVED INCOMPLETE

(Firefox OS Graveyard :: Emulator, defect)

Description

[Szu-Yu Chen [:aknow] (inactive)]

Add a console command to alter sim status. It could be used for testing empty sim as following steps:
1. send emulator console command "sim status absent"
2. turn radio off and on

Comment 1

[Szu-Yu Chen [:aknow] (inactive)]

Attachment: platform_external_qemu: PR #106

Comment 2

[Vicamo Yang [:vicamo][:vyang]]

Comment on attachment 8450870 [details] [review]
platform_external_qemu: PR #106

Looks good to me, but I need test cases to see how this works actually before placing a review comment.

Comment 3

[Szu-Yu Chen [:aknow] (inactive)]

ni myself to work on test case.

Comment 4

[Szu-Yu Chen [:aknow] (inactive)]

Attachment: card state test

I modify the card state test to show an example of using this console command.

Comment 5

[Vicamo Yang [:vicamo][:vyang]]

Comment on attachment 8455999 [details] [diff] [review]
card state test

Review of attachment 8455999 [details] [diff] [review]:
-----------------------------------------------------------------

::: dom/icc/tests/marionette/test_icc_card_state.js
@@ +29,5 @@
>  /* Test cardstatechange event by switching radio off */
>  taskHelper.push(function testCardStateChange() {
>    // Turn off radio.
> +  setRadioEnabled(false, function() {
> +    icc.addEventListener("cardstatechange", function oncardstatechange() {

You may lose this event because that addEventListener is not necessarily installed before 'cardstatechange' has ever been emitted.

Basiscally when you do:

  <some request that emits 'event-foo'>();
  yet_another_async_op(function callback() {
    <register_event_foo>();
    ...
  });

That can result in intermittent failure on try server/emulator.

Comment 6

[Szu-Yu Chen [:aknow] (inactive)]

Attachment: #2 card state test

Comment 7

[Vicamo Yang [:vicamo][:vyang]]

Comment on attachment 8456051 [details] [diff] [review]
#2 card state test

Review of attachment 8456051 [details] [diff] [review]:
-----------------------------------------------------------------

Can you have something like:

  function setRadioAndWaitForNamedEvent(aEnable, aEventName, aCallback) {
    setRadioEnabled(false);
    icc.addEventListener(aEventName, function onevent() {
      if (aCallback()) {
        icc.removeEventListener(aEventName, onevent);
      }
    });
  }

  // In that |testCardStateChange()| you have:
  setRadioAndWaitForNamedEvent(false, "cardstatechange" function() {
    if (icc.cardState !== null) {
      return false;
    }

    setRadioAndWaitForNamedEvent(true, "iccdetected", function() {
      window.setTimeout(() => taskHelper.runNext(), 0);

      // Unregister "iccdetected".
      return true;
    });

    // Unregister "cardstatechange".
    return true;
  });

  // In |testRemoveCard()| you have:
  function emulatorChangeSimStatus(aCardStatus, aCallback) {
    setRadioAndWaitForNamedEvent(false, "cardstatechange", function() {
      if (icc.cardState !== null) {
        return false;
      }

      emulatorHelper.sendCommand("sim status " + status, aCallback);

      // Unregister "cardstatechange".
      return true;
    }
  }

  emulatorChangeSimStatus("absent", function() {
    setRadioAndWaitForNamedEvent(true, "cardstatechange", function() {
      if (icc.cardState === null) {
        return false;
      }

      emulatorChangeSimStatus("ready", function() {
        setRadioAndWaitForNamedEvent(true, "iccdetected", function() {
          window.setTimeout(() => taskHelper.runNext(), 0);

          // Unregister "iccdetected".
          return true;
        });
      });

      // Unregister "cardstatechange".
      return true;
    });
  });

::: dom/icc/tests/marionette/test_icc_card_state.js
@@ +5,3 @@
>  MARIONETTE_HEAD_JS = "icc_header.js";
>  
> +connection = navigator.mozMobileConnections[0];

nit: let connection = ...

@@ +81,5 @@
> +      })
> +    }, function() {
> +      let duration = Date.now() - startTime;
> +      // Make sure that the card state is still null after 5s.
> +      return duration >= 5000 && icc.cardState === null;

We'd probably avoid timeouts whenever possible.

Comment 8

[Szu-Yu Chen [:aknow] (inactive)]

Attachment: #3 card state test