1338320 - annotate testing/web-platform/moz.build with BUG_COMPONENT for tests/ directory

Status: RESOLVED FIXED

(Testing :: web-platform-tests, defect)

Description

[Joel Maher ( :jmaher ) (UTC -8)]

there are many directories of tests underneath testing/web-platform/tests/, we need to match these up to bugzilla-components to help find the right people to debug issues.

Comment 1

[Joel Maher ( :jmaher ) (UTC -8)]

I have 8 questions on specific directories, otherwise I have mediacapture -> workers annotated.  upcoming questions/patch tomorrow.

Comment 2

[Joel Maher ( :jmaher ) (UTC -8)]

:tnguyen, can you help me determine which bugzilla component bugs for these referrer-policy tests should go:
https://dxr.mozilla.org/mozilla-central/source/testing/web-platform/tests/referrer-policy

I suspect it would be a single component for all tests.

Comment 3

[Joel Maher ( :jmaher ) (UTC -8)]

Simon, could you help me figure out which bugzilla component bugs related to the preload tests would go in:
https://dxr.mozilla.org/mozilla-central/source/testing/web-platform/tests/preload

I suspect these will all fall in the same component.

Comment 4

[Joel Maher ( :jmaher ) (UTC -8)]

:ato, can you help me figure out what bugzilla component I would file bugs for old-tests/webdriver/*:
https://dxr.mozilla.org/mozilla-central/source/testing/web-platform/tests/old-tests/webdriver

would it be testing::marionette?

Comment 5

[Joel Maher ( :jmaher ) (UTC -8)]

I still need to figure out:
testing/web-platform/tests/old-tests/submission/Infraware/Session_History/contents/Session_History/
testing/web-platform/tests/old-tests/submission/Opera/script_scheduling/
testing/web-platform/tests/old-tests/submission/Microsoft/foreigncontent
testing/web-platform/tests/old-tests/submission/Microsoft/sandbox
testing/web-platform/tests/old-tests/submission/Microsoft/structuredclones
testing/web-platform/tests/old-tests/submission/Microsoft/xhtml5
testing/web-platform/tests/page-visibility/
testing/web-platform/tests/payment-request/
testing/web-platform/tests/presentation-api/

open to any drive by comments on these as well.

Comment 6

[Andreas Tolfsen ❲:ato❳]

(In reply to Joel Maher ( :jmaher) from comment #4)
> :ato, can you help me figure out what bugzilla component I would file bugs
> for old-tests/webdriver/*:
> https://dxr.mozilla.org/mozilla-central/source/testing/web-platform/tests/
> old-tests/webdriver
> 
> would it be testing::marionette?

They should be filed with the "webdriver" label on https://github.com/w3c/web-platform-tests/.

Comment 7

[Joel Maher ( :jmaher ) (UTC -8)]

interesting, but I think it works:
 ./mach file-info bugzilla-component testing/web-platform/tests/webdriver/**
https://github.com/w3c/web-platform-tests/ :: webdriver
  testing/web-platform/tests/webdriver/OWNERS
  testing/web-platform/tests/webdriver/README.md
  testing/web-platform/tests/webdriver/actions.py
  testing/web-platform/tests/webdriver/contexts.py
  testing/web-platform/tests/webdriver/interface.html
  testing/web-platform/tests/webdriver/navigation.py
  testing/web-platform/tests/webdriver/support

Comment 8

[Thomas Nguyen (:tnguyen)]

(In reply to Joel Maher ( :jmaher) from comment #2)
> :tnguyen, can you help me determine which bugzilla component bugs for these
> referrer-policy tests should go:
> https://dxr.mozilla.org/mozilla-central/source/testing/web-platform/tests/
> referrer-policy
> 
> I suspect it would be a single component for all tests.

they are under dom:sercurity

Comment 9

[Simon Pieters [:zcorpan]]

(In reply to Joel Maher ( :jmaher) from comment #3)
> Simon, could you help me figure out which bugzilla component bugs related to
> the preload tests would go in:
> https://dxr.mozilla.org/mozilla-central/source/testing/web-platform/tests/
> preload
> 
> I suspect these will all fall in the same component.

I don't know. James?

Comment 10

[James Graham [:jgraham]]

> testing/web-platform/tests/old-tests/submission/Infraware/Session_History/contents/Session_History/

Presumably either Core::History: Global or Core::DOM: Core & HTML

> testing/web-platform/tests/old-tests/submission/Opera/script_scheduling/

Core::DOM: Core & HTML

> testing/web-platform/tests/old-tests/submission/Microsoft/foreigncontent

Maybe Core::SVG (these tests look like junk anyway)

> testing/web-platform/tests/old-tests/submission/Microsoft/sandbox

Either Core::DOM: Security or Core::DOM: Core & HTML or Core::Security

> testing/web-platform/tests/old-tests/submission/Microsoft/structuredclones

Core::Javascript Engine

> testing/web-platform/tests/old-tests/submission/Microsoft/xhtml5

These tests are junk, but might become Core::Layout if they were fixed

> testing/web-platform/tests/page-visibility/

Core::DOM I think

> testing/web-platform/tests/payment-request/

Core::DOM

> testing/web-platform/tests/presentation-api/

Core::DOM

Comment 11

[James Graham [:jgraham]]

> testing/web-platform/tests/preload

Core::DOM: Core & HTML 

> testing/web-platform/tests/old-tests/webdriver

testing::marionette is fine

Comment 12

[Joel Maher ( :jmaher ) (UTC -8)]

Attachment: add BUG_COMPONENT to testing/web-platform/tests/* (part 1 of 2)

:overholt, I am asking you for review  as much of these fall in Core::DOM, please feel free to redirect as you see fit.

Comment 13

[Joel Maher ( :jmaher ) (UTC -8)]

Attachment: add BUG_COMPONENT to testing/web-platform/tests/* (part 1 of 2)

updated typedarrays to be Core::Javascript: Standard Library.

Comment 14

[Andrew Overholt [:overholt]]

Comment on attachment 8838486 [details] [diff] [review]
add BUG_COMPONENT to testing/web-platform/tests/* (part 1 of 2)

Review of attachment 8838486 [details] [diff] [review]:
-----------------------------------------------------------------

With the changes below (above?), r=overholt. Sorry for the delay here!

::: testing/web-platform/moz.build
@@ +55,5 @@
> +with Files("tests/mixed-content/**"):
> +    BUG_COMPONENT = ("Core", "DOM: Security")
> +
> +with Files("tests/navigation-timing/**"):
> +    BUG_COMPONENT = ("Core", "Document Navigation")

Just Core::DOM here.

@@ +58,5 @@
> +with Files("tests/navigation-timing/**"):
> +    BUG_COMPONENT = ("Core", "Document Navigation")
> +
> +with Files("tests/notifications/**"):
> +    BUG_COMPONENT = ("Core", "DOM: Workers")

Toolkit::Notifications and Alerts

@@ +65,5 @@
> +    BUG_COMPONENT = ("Core", "Canvas: 2D")
> +
> +# No tests are run here
> +with Files("tests/old-tests/submission/Infraware/Session_History/**"):
> +    BUG_COMPONENT = ("Core", "History: Global")

Core::Document Navigation

@@ +71,5 @@
> +with Files("tests/old-tests/submission/Infraware/forms/**"):
> +    BUG_COMPONENT = ("Core", "DOM: Core & Html")
> +
> +with Files("tests/old-tests/Opera/**"):
> +    BUG_COMPONENT = ("Core", "DOM: Core & Html")

s/old-tests\/Opera/old-tests\/submission\/Opera/ . These are a real mix ... what do you want to do with them?

@@ +75,5 @@
> +    BUG_COMPONENT = ("Core", "DOM: Core & Html")
> +
> +# No tests are run here
> +with Files("tests/old-tests/Microsoft/dragdrop/**"):
> +    BUG_COMPONENT = ("Core", "Drag & Drop")

s/old-tests\/Microsoft/old-tests\/submission\/Microsoft/

@@ +82,5 @@
> +with Files("tests/old-tests/Microsoft/foreigncontent/**"):
> +    BUG_COMPONENT = ("Core", "SVG")
> +
> +with Files("tests/old-tests/Microsoft/history/**"):
> +    BUG_COMPONENT = ("Core", "History: Global")

Core::Document Navigation is more applicable, I think.

@@ +108,5 @@
> +with Files("tests/payment-request/**"):
> +    BUG_COMPONENT = ("Core", "DOM")
> +
> +with Files("tests/performance-timeline/**"):
> +    BUG_COMPONENT = ("Core", "DOM: Service Workers")

Core::DOM

@@ +142,5 @@
> +    BUG_COMPONENT = ("Core", "DOM")
> +
> +with Files("tests/secure-contexts/**"):
> +    BUG_COMPONENT = ("Core", "DOM: Workers")
> +

While there are a bunch of worker things in this directory I think this is more "https" so let's just go with Core::DOM (we can always fix it or move bugs later).

@@ +171,5 @@
> +with Files("tests/svg-aam/**"):
> +    BUG_COMPONENT = ("Core", "SVG")
> +
> +with Files("tests/touchevents/**"):
> +    BUG_COMPONENT = ("Core", "Panning and Zooming")

Core::DOM

@@ +177,5 @@
> +with Files("tests/typedarrays/**"):
> +    BUG_COMPONENT = ("Core", "Javascript: Standard Library")
> +
> +with Files("tests/uievents/**"):
> +    BUG_COMPONENT = ("Core", "DOM")

Let's go with DOM: Events

Comment 15

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/mozilla-inbound/rev/41d509e4b368
annotate testing/web-platform/moz.build with BUG_COMPONENT: part1. r=overholt

Comment 16

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/41d509e4b368