[MERGED] osmo-gsm-tester[master]: test.py: Remove unused parameter ofono_client in setup()

2017-11-08 Thread Pau Espin Pedrol 
Pau Espin Pedrol has submitted this change and it was merged.

Change subject: test.py: Remove unused parameter ofono_client in setup()
..


test.py: Remove unused parameter ofono_client in setup()

Change-Id: Ib8983421a7ca5f94fe1363768a12fff63c1fc64a
---
M src/osmo_gsm_tester/suite.py
M src/osmo_gsm_tester/test.py
2 files changed, 2 insertions(+), 2 deletions(-)

Approvals:
  Neels Hofmeyr: Looks good to me, approved
  Jenkins Builder: Verified



diff --git a/src/osmo_gsm_tester/suite.py b/src/osmo_gsm_tester/suite.py
index fc4930d..eead5af 100644
--- a/src/osmo_gsm_tester/suite.py
+++ b/src/osmo_gsm_tester/suite.py
@@ -91,7 +91,7 @@
 log.large_separator(self.suite_run.trial.name(), 
self.suite_run.name(), self.name(), sublevel=3)
 self.status = Test.UNKNOWN
 self.start_timestamp = time.time()
-test.setup(self.suite_run, self, ofono_client, 
sys.modules[__name__], event_loop, sms)
+test.setup(self.suite_run, self, sys.modules[__name__], 
event_loop, sms)
 with self.redirect_stdout():
 util.run_python_file('%s.%s' % 
(self.suite_run.definition.name(), self.basename),
  self.path)
diff --git a/src/osmo_gsm_tester/test.py b/src/osmo_gsm_tester/test.py
index 49911b3..b939054 100644
--- a/src/osmo_gsm_tester/test.py
+++ b/src/osmo_gsm_tester/test.py
@@ -35,7 +35,7 @@
 Timeout = None
 Sms = None
 
-def setup(suite_run, _test, ofono_client, suite_module, event_module, 
sms_module):
+def setup(suite_run, _test, suite_module, event_module, sms_module):
 global trial, suite, test, resources, log, dbg, err, wait, wait_no_raise, 
sleep, poll, prompt, Timeout, Sms
 trial = suite_run.trial
 suite = suite_run

-- 
To view, visit https://gerrit.osmocom.org/4708
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: Ib8983421a7ca5f94fe1363768a12fff63c1fc64a
Gerrit-PatchSet: 2
Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Owner: Pau Espin Pedrol 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: Neels Hofmeyr 
Gerrit-Reviewer: Pau Espin Pedrol

osmo-gsm-tester[master]: test.py: Remove unused parameter ofono_client in setup()

2017-11-07 Thread Neels Hofmeyr 

Patch Set 1: Code-Review+2

-- 
To view, visit https://gerrit.osmocom.org/4708
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ib8983421a7ca5f94fe1363768a12fff63c1fc64a
Gerrit-PatchSet: 1
Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Owner: Pau Espin Pedrol 
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: Neels Hofmeyr 
Gerrit-HasComments: No

[PATCH] osmo-gsm-tester[master]: test.py: Remove unused parameter ofono_client in setup()

2017-11-07 Thread Pau Espin Pedrol 

Review at  https://gerrit.osmocom.org/4708

test.py: Remove unused parameter ofono_client in setup()

Change-Id: Ib8983421a7ca5f94fe1363768a12fff63c1fc64a
---
M src/osmo_gsm_tester/suite.py
M src/osmo_gsm_tester/test.py
2 files changed, 2 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.osmocom.org:29418/osmo-gsm-tester refs/changes/08/4708/1

diff --git a/src/osmo_gsm_tester/suite.py b/src/osmo_gsm_tester/suite.py
index 28bdd69..33a7761 100644
--- a/src/osmo_gsm_tester/suite.py
+++ b/src/osmo_gsm_tester/suite.py
@@ -91,7 +91,7 @@
 log.large_separator(self.suite_run.trial.name(), 
self.suite_run.name(), self.name(), sublevel=3)
 self.status = Test.UNKNOWN
 self.start_timestamp = time.time()
-test.setup(self.suite_run, self, ofono_client, 
sys.modules[__name__], event_loop, sms)
+test.setup(self.suite_run, self, sys.modules[__name__], 
event_loop, sms)
 with self.redirect_stdout():
 util.run_python_file('%s.%s' % 
(self.suite_run.definition.name(), self.basename),
  self.path)
diff --git a/src/osmo_gsm_tester/test.py b/src/osmo_gsm_tester/test.py
index 49911b3..b939054 100644
--- a/src/osmo_gsm_tester/test.py
+++ b/src/osmo_gsm_tester/test.py
@@ -35,7 +35,7 @@
 Timeout = None
 Sms = None
 
-def setup(suite_run, _test, ofono_client, suite_module, event_module, 
sms_module):
+def setup(suite_run, _test, suite_module, event_module, sms_module):
 global trial, suite, test, resources, log, dbg, err, wait, wait_no_raise, 
sleep, poll, prompt, Timeout, Sms
 trial = suite_run.trial
 suite = suite_run

-- 
To view, visit https://gerrit.osmocom.org/4708
To unsubscribe, visit https://gerrit.osmocom.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ib8983421a7ca5f94fe1363768a12fff63c1fc64a
Gerrit-PatchSet: 1
Gerrit-Project: osmo-gsm-tester
Gerrit-Branch: master
Gerrit-Owner: Pau Espin Pedrol