All of lore.kernel.org
 help / color / mirror / Atom feed
From: Denis Kenzior <denkenz@gmail.com>
To: ofono@ofono.org
Subject: Re: [PATCH 1/2] Allow users to specify dbus name replacement behaviour.
Date: Thu, 03 Apr 2014 19:25:29 -0500	[thread overview]
Message-ID: <533DFBF9.9050906@gmail.com> (raw)
In-Reply-To: <533DFB3F.6080802@canonical.com>

[-- Attachment #1: Type: text/plain, Size: 1179 bytes --]

Hi Antti,

> What we are trying to do is a full stack system testing where the
> individual components that are being tested don't have any modifications
> or adaptations when they are run under test suite.

Okay that sort of makes sense

> 
> The number of modems returned by org.ofono.Manager.GetModems() must
> represent the total number of modems available to the system and thus we
> need to have exactly two modems when testing dual-sim features etc.
> 

Sounds like your system is a bit inflexible, but okay.

> The need to change the number of modems during testing comes from the
> fact that our test suite has the single and multimodem tests together
> and runs them one after another and we need to be able to set up the
> environment appropriately in between individual test cases.
> 

Why don't you simply write a plugin that handles all of this?  E.g.
canonical_tester that creates two modem instances.  If you must insist
on having exactly 1 or 2 modems, then just add a DBus interface to
switch between modes.

Adding a control mechanism for controlling the number of phonesim
instances seems like total overkill.

Regards,
-Denis

  reply	other threads:[~2014-04-04  0:25 UTC|newest]

Thread overview: 17+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-04-02 13:56 [PATCH 1/2] Allow users to specify dbus name replacement behaviour jussi.pakkanen
2014-04-02 13:56 ` [PATCH 2/2] Can set name replacement with command line arguments jussi.pakkanen
2014-04-02 18:39 ` [PATCH 1/2] Allow users to specify dbus name replacement behaviour Marcel Holtmann
2014-04-03  7:38   ` Jussi Pakkanen
2014-04-03  9:41     ` Antti =?unknown-8bit?q?Kaijanm=C3=A4ki?=
2014-04-03 17:55       ` Marcel Holtmann
2014-04-03 23:45         ` Antti =?unknown-8bit?q?Kaijanm=C3=A4ki?=
2014-04-04  0:09           ` Antti =?unknown-8bit?q?Kaijanm=C3=A4ki?=
2014-04-04  0:13             ` Denis Kenzior
2014-04-04  0:11           ` Denis Kenzior
2014-04-04  0:50             ` Antti =?unknown-8bit?q?Kaijanm=C3=A4ki?=
2014-04-03 21:40       ` Denis Kenzior
2014-04-04  0:00         ` Antti =?unknown-8bit?q?Kaijanm=C3=A4ki?=
2014-04-04  0:22         ` Antti =?unknown-8bit?q?Kaijanm=C3=A4ki?=
2014-04-04  0:25           ` Denis Kenzior [this message]
2014-04-04  1:16             ` Antti =?unknown-8bit?q?Kaijanm=C3=A4ki?=
2014-04-03 17:50     ` Marcel Holtmann

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=533DFBF9.9050906@gmail.com \
    --to=denkenz@gmail.com \
    --cc=ofono@ofono.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.