Closed Bug 1554707 Opened 5 years ago Closed 5 years ago

[meta] Refactor existing folder structure in /test

Categories

(DevTools :: Application Panel, task, P1)

task

Tracking

(firefox70 fixed)

RESOLVED FIXED
Firefox 70
Tracking Status
firefox70 --- fixed

People

(Reporter: ogasidlo, Assigned: ogasidlo)

References

(Regressed 1 open bug)

Details

(Keywords: meta, Whiteboard: foundation-work)

Attachments

(1 file, 1 obsolete file)

Our folder structure right now for tests is as follows:

test

|_ service-workers (contains mocks + helpers for SW tests)
|_ browser_application_panel_**.js (several test files)
|_ browser. ini
|_ head.js

I suggest aligning the structure with remote debugging as far as possible as we have soon very similar features.

|_ browser
--|_ mocks (move service-workers mocks in here - needs a review of what we need to refactor)
--|_ helpers (move service-workers helpers in here - needs a review of what we need to refactor)
--|_ browser_application_panel_**.js (several test files)
|_ components (new)
--|_ component_application_panel_**.js (new)
|_ unit (new)
--|_ unit_application_panel_**.js (new)
|_ initializer.js (for base urls + require of App.js component)
|_ application.js (require of all modules)
|_ application.css (require of all styles)
|_ index.html
|_ brower.ini
Priority: P3 → P2
Type: defect → task
Keywords: meta
Summary: Refactor existing folder structure in /test → [meta] Refactor existing folder structure in /test
Assignee: nobody → ogasidlo
Status: NEW → ASSIGNED
Priority: P2 → P1

+1 to this structure (basically what we have in remote debugging plus renaming jest to components). We should add this to our doc, so people know how to run every type of tests.

Refactor test folder structure to align with remote debugging

Attachment #9072112 - Attachment is obsolete: true
Pushed by ogasidlo@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/c022b93b0777
Refactor test folder structure, r=jdescottes,ladybenko
Pushed by archaeopteryx@coole-files.de:
https://hg.mozilla.org/integration/autoland/rev/e633c74fdd10
Refactor test folder structure: adjust code for js Prettier. a=eslint-fix CLOSED TREE
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 70
Regressions: 1559591
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: