Closed Bug 1609392 Opened 3 years ago Closed 3 years ago

Split Raptor and Browsertime classes from raptor.py

Categories

(Testing :: Raptor, enhancement, P1)

Version 3
enhancement

Tracking

(firefox74 fixed)

RESOLVED FIXED
mozilla74
Tracking Status
firefox74 --- fixed

People

(Reporter: Bebe, Assigned: Bebe)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

The planning doc states we should split Raptor classes in multiple files:

../testing/
raptor/
   ...
   raptor/
       ...
	raptor/
		__init__.py
		base.py
			-Raptor
		android.py
			-RaptorAndroid
		desktop.py
			-RaptorDesktop

After discussing with Tarek we decided to rename all Raptor classes to WebExtension.

This will make things more clear and readable in the project.
Also we will have a better visibility and difference between the 2 testing tools used BrowserTime and webextension

Flags: needinfo?(rwood)

the project classes will be:

../testing/
raptor/
   ...
   raptor/
       ...
	webextension/
		__init__.py
		base.py
			-WebExtension
		android.py
			-WebExtensionAndroid
		desktop.py
			-WebExtensionDesktop
Summary: Split Raptor class from raptor.py → Split Raptor and Browsertime classes from raptor.py

the project classes will be:

../testing/
raptor/
   ...
   raptor/
       ...
	browsertime/
		__init__.py
		base.py
			-Browsertime
		android.py
			-BrowsertimeAndroid
		desktop.py
			-BrowsertimeDesktop
	webextension/
		__init__.py
		base.py
			-WebExtension
		android.py
			-WebExtensionAndroid
		desktop.py
			-WebExtensionDesktop

Also I will update the unit tests and some comments to reflect the renaming

Assignee: nobody → fstrugariu
Status: NEW → ASSIGNED

(In reply to Florin Strugariu [:Bebe] (needinfo me) from comment #1)

After discussing with Tarek we decided to rename all Raptor classes to WebExtension.

This will make things more clear and readable in the project.
Also we will have a better visibility and difference between the 2 testing tools used BrowserTime and webextension

Sounds good, thanks.

Flags: needinfo?(rwood)
Attachment #9121078 - Attachment description: Bug 1609392 - Split Raptor class from raptor.py → Bug 1609392 - Split Raptor and Browsertime classes from raptor.py
Pushed by fstrugariu@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/2f3960b4e42d
Split Raptor and Browsertime classes from raptor.py r=tarek,AlexandruIonescu,octavian_negru,sparky,perftest-reviewers,alexandru.irimovici
Priority: -- → P1
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla74
You need to log in before you can comment on or make changes to this bug.