#2112 closed defect (fixed)
Automated Tests missing data
Reported by: | J. Max Mena | Owned by: | Smo |
---|---|---|---|
Priority: | major | Milestone: | 2.5 |
Component: | tests | Version: | trunk |
Keywords: | Cc: |
Description
As mentioned in #619, #1840, and #2064, the automated tests are missing a handful of data points. From ticket:1840#comment:6, most of the useful data points such as batch delay, quality, et al aren't included in the CSV. As far as I can tell, there's no error or traceback printed when the tests run xpra info
, so it's likely that the Xpra Info page has changed, as is mentioned. The test code will likely need to be updated to reflect these changes.
Attachments (1)
Change History (12)
comment:1 Changed 2 years ago by
Status: | new → assigned |
---|
comment:2 Changed 2 years ago by
Wow, this whole thing is in terrible shape.
Minor cleanup in r21958.
Bug fix in r21959. Now at least some of the tests do run.
Now the real problem with the missing data is the same thing that required fixes to the session info dialog (r21340).
As of r21960, we capture the batch delay again, more fixes needed.
comment:3 Changed 2 years ago by
As of r21962, the actual batch delay is back (but now this is a normalized value... and maybe this should not be compared?), as well as the latency values.
Still need to restore:
- packets/s in and out
- application packets/s
- speed and quality
comment:4 Changed 2 years ago by
comment:5 Changed 2 years ago by
Owner: | changed from Antoine Martin to Jonathan Anthony |
---|---|
Status: | assigned → new |
comment:6 Changed 19 months ago by
Owner: | changed from Jonathan Anthony to Smo |
---|
comment:7 Changed 19 months ago by
I attached some data from some browser tests. Still seems to be some missing data points.
download limit (KB), upload limit (KB), latency (ms) all read 0 for example
Can you take a look and let me know?
comment:8 Changed 19 months ago by
Owner: | changed from Smo to Antoine Martin |
---|
comment:9 Changed 19 months ago by
Owner: | changed from Antoine Martin to Smo |
---|
Still seems to be some missing data points.
download limit (KB), upload limit (KB), latency (ms) all read 0 for example
Those are test configuration options for trickle.
They should be showing up if you enable it.
comment:10 Changed 19 months ago by
Resolution: | → fixed |
---|---|
Status: | new → closed |
Confirmed the missing data does show up when you turn on the trickle options.
comment:11 Changed 5 weeks ago by
this ticket has been moved to: https://github.com/Xpra-org/xpra/issues/2112
I'm pretty sure the data collection breakage is fairly recent, whereas these tickets are not, so you should be able to test the version they were completed for: