mirror of
https://github.com/LorenEteval/Furious.git
synced 2026-09-22 23:08:08 +03:00
Improve logical spacing in profile testing
Signed-off-by: Loren Eteval <loren.eteval@proton.me>
This commit is contained in:
@@ -531,6 +531,7 @@ class _LatencyScheduler(QtCore.QObject):
|
||||
continue
|
||||
|
||||
job.state = ProfileTestJobState.Running
|
||||
|
||||
worker = self.pingWorkerFactory(job, self)
|
||||
|
||||
self.activeJobs[id(job)] = (job, worker)
|
||||
@@ -1269,6 +1270,7 @@ class _DownloadSpeedScheduler(QtCore.QObject):
|
||||
return
|
||||
|
||||
_, job, port = active
|
||||
|
||||
lease = worker.takeRuntimeLease()
|
||||
|
||||
# Completion may notify reentrant consumers. Establish the durable
|
||||
|
||||
@@ -428,6 +428,7 @@ class ProfileTestServiceTest(unittest.TestCase):
|
||||
self.assertEqual(runtime.isRunning(), failure == 'stop')
|
||||
|
||||
runtime.failure = None
|
||||
|
||||
worker._releaseRuntime()
|
||||
|
||||
self.assertIsNone(worker._runtimeLease)
|
||||
@@ -446,16 +447,19 @@ class ProfileTestServiceTest(unittest.TestCase):
|
||||
if concurrent
|
||||
else manager._serialDownloadScheduler
|
||||
)
|
||||
|
||||
results = []
|
||||
manager.resultApplied.connect(
|
||||
lambda _profile, result: results.append(result)
|
||||
)
|
||||
|
||||
manager.testDownloadSpeed(profiles, concurrent=concurrent)
|
||||
processQtEvents()
|
||||
|
||||
worker, runtime = workers[0], runtimes[0]
|
||||
port = worker.port
|
||||
lease = worker._runtimeLease
|
||||
|
||||
destroyed = []
|
||||
worker.destroyed.connect(lambda: destroyed.append(True))
|
||||
results.clear()
|
||||
@@ -479,9 +483,11 @@ class ProfileTestServiceTest(unittest.TestCase):
|
||||
|
||||
runtime.publishExit(RuntimeExit(1, RuntimeExitReason.Unexpected))
|
||||
processQtEvents()
|
||||
|
||||
self.assertEqual(len(results), 1)
|
||||
|
||||
runtime.failure = None
|
||||
|
||||
scheduler.scheduleDrain()
|
||||
processQtEvents()
|
||||
|
||||
@@ -500,7 +506,9 @@ class ProfileTestServiceTest(unittest.TestCase):
|
||||
manager.testDownloadSpeed(profiles[:1], concurrent=False)
|
||||
manager.testDownloadSpeed(profiles[1:], concurrent=True)
|
||||
processQtEvents()
|
||||
|
||||
self.assertEqual(len(workers), 2)
|
||||
|
||||
runtimes[1].failure = None
|
||||
|
||||
with self.assertLogs('Furious.Service.RuntimeLease', level='ERROR'):
|
||||
@@ -516,9 +524,11 @@ class ProfileTestServiceTest(unittest.TestCase):
|
||||
|
||||
manager.testDownloadSpeed(profiles)
|
||||
processQtEvents()
|
||||
|
||||
self.assertEqual(len(workers), 2)
|
||||
|
||||
runtimes[0].failure = None
|
||||
|
||||
manager.shutdown()
|
||||
processQtEvents()
|
||||
|
||||
|
||||
Reference in New Issue
Block a user