Closed
Bug 1433947
Opened 8 years ago
Closed 8 years ago
Enable Unit Tests for Firefox and FirefoxBeta schemas
Categories
(Firefox for iOS :: Build & Test, enhancement)
Tracking
()
RESOLVED
FIXED
| Tracking | Status | |
|---|---|---|
| fxios | 11.0 | --- |
People
(Reporter: isabel_rios, Unassigned)
Details
Attachments
(1 file)
Unit Tests are running on master only on Fennec_Enterprise.
So when a release branch is created, and PRs run on Firefox and FirefoxBeta no tests run there.
| Reporter | ||
Comment 1•8 years ago
|
||
As we commented, with this PR the unit tests that run on v10.x are enabled for master too for the same schemas so that when a release branch is created it will have the unit tests running on Firefox and FirefoxBeta schemas
Attachment #8946275 -
Flags: review?(npark)
Comment 2•8 years ago
|
||
Comment on attachment 8946275 [details] [review]
Pull Request
Noticed Stefan was listed in the Github reviewer list, but not here. Should he be added as well?
Attachment #8946275 -
Flags: review?(npark) → review+
| Reporter | ||
Comment 3•8 years ago
|
||
Since he mentioned about creating the PR on slack I added him to review so he knew it was done and also to check if he was fine with the tests enabled and having them enabled in master, not only on v11.x. I guess that is fine since those tests do not run unless the corresponding schemas are selected.
Lets ask him just in case to be sure he is fine with this PR too.
So Stefan, ni you ^^ Thanks!
Flags: needinfo?(sarentz)
| Reporter | ||
Updated•8 years ago
|
Whiteboard: [needsuplift]
| Reporter | ||
Comment 4•8 years ago
|
||
Merged master: https://github.com/mozilla-mobile/firefox-ios/commit/e2153c31fcd32779f6b4b724382038e97dd2b4cb
Status: NEW → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
| Reporter | ||
Comment 5•8 years ago
|
||
Uplifted v11.x https://github.com/mozilla-mobile/firefox-ios/commit/0f87b86c2418694319e1311f247a34cae7bb7f81
Whiteboard: [needsuplift]
You need to log in
before you can comment on or make changes to this bug.
Description
•