New issue
Advanced search Search tips

Issue 716338 link

Starred by 1 user

Issue metadata

Status: WontFix
Owner:
Closed: Jul 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: ----
Pri: 3
Type: Feature



Sign in to add a comment

[Upstream WPT] Make subtest names greater than 500 characters a harness error

Project Member Reported by jeffcarp@chromium.org, Apr 28 2017

Issue description

Via foolip's comment in bit.ly/wptdashboard-design-doc

The specific number 500 is arbitrary and is open for debate.

- [ ] Create histogram of subtest name lengths
- [ ] Create bug in upstream WPT
 
The first step is probably making an issue in upstream WPT to discuss.
Description: Show this description
Status: Available (was: Untriaged)
Agreed, this sounds like it may be good idea, and should be discussed in an upstream issue.

Is the reason for this that long subtest names are an issue for App Engine datastore if subtest names are used as StringProperty or used in keys? There's probably some way that we could work around that if we don't end up putting a limit on subtest names. The simplest workaround might be to truncate very long subtest names.
Yes, originally it was motivated by not wanting to break the 1MB limit for datastore objects. However, the design doc has changed a bit and I'm not planning on storing test results in Datastore, so this is no longer a pressing issue. foolip seemed to be interested in limiting subtest name length in general just as a good practice.

The first thing to do is probably generate a histogram of subtest name lengths. I'll update the TODOs.
Description: Show this description
Components: Blink>Infra>Ecosystem
Components: -Blink>Infra>Predictability
Should we WontFix this issue perhaps? I can't fine this issue references in the Chromium code, are there TODOs elsewhere pointing here to update?
Status: WontFix (was: Available)
I wouldn't mind marking WontFix. If we want to pursue this later, as #1 says, this would be a feature request on the wpt GitHub issue tracker.

Sign in to add a comment