Closed Bug 1598323 Opened 5 years ago Closed 5 years ago

TypeError: unorderable types: NoneType() > int()

Categories

(Remote Protocol :: Marionette, defect, P3)

Version 3
defect

Tracking

(Not tracked)

RESOLVED DUPLICATE of bug 1640249

People

(Reporter: rgpt, Unassigned)

References

Details

Steps to reproduce:
$ python3 mach marionette-test testing/marionette/harness/marionette_harness/tests/unit/test_accessibility.py

Actual results:
TypeError: unorderable types: NoneType() > int()

File "/home/rgpt/Desktop/mozilla/mozilla-unified/testing/marionette/mach_commands.py", line 109, in marionette_test
return run_marionette(tests, topsrcdir=self.topsrcdir, **kwargs)
File "/home/rgpt/Desktop/mozilla/mozilla-unified/testing/marionette/mach_commands.py", line 58, in run_marionette
failed = MarionetteHarness(MarionetteTestRunner, args=vars(args)).run()
File "/home/rgpt/Desktop/mozilla/mozilla-unified/testing/marionette/harness/marionette_harness/runtests.py", line 71, in run
runner.run_tests(tests)
File "/home/rgpt/Desktop/mozilla/mozilla-unified/testing/marionette/harness/marionette_harness/runner/base.py", line 917, in run_tests
self.run_test_sets()
File "/home/rgpt/Desktop/mozilla/mozilla-unified/testing/marionette/harness/marionette_harness/runner/base.py", line 1105, in run_test_sets
elif self.total_chunks > len(self.tests):

Expected results:
It should have worked with py3 also as it does on py2

Blocks: 1597773

@whimboo could you please help me on this ? the default value of total_chunks in init must have been 1.

No longer blocks: 1597773
Flags: needinfo?(ahal)

It's me you want to have feedback from. I will have to look into that, and may not get to that today.

Flags: needinfo?(ahal) → needinfo?(hskupin)
Flags: needinfo?(hskupin)
Priority: -- → P3

If total_chunks is of type None it would mean that None is passed into BaseMarionetteTestRunner.__init__(). Please check the caller tree by going up level by level where this wrong value comes from.

I believe this was fixed by Bug 1640249

Indeed, I can remember to have reviewed that part.

Status: NEW → RESOLVED
Closed: 5 years ago
Resolution: --- → DUPLICATE
Product: Testing → Remote Protocol
You need to log in before you can comment on or make changes to this bug.