Note: There are a few cases of duplicates in user autocompletion which are being worked on.

Librarify the jsrefest suite

RESOLVED FIXED in mozilla15

Status

()

Core
JavaScript Engine
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: terrence, Assigned: terrence)

Tracking

(Blocks: 1 bug)

Trunk
mozilla15
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

5 years ago
Created attachment 621161 [details] [diff] [review]
v0

Put all the random .py files in tests into a lib dir.  Totally trivial.
Attachment #621161 - Flags: review?(dmandelin)
Comment on attachment 621161 [details] [diff] [review]
v0

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

Missing the moved files?
Attachment #621161 - Flags: review?(dmandelin)
(Assignee)

Comment 2

5 years ago
Wow, that's odd.  The "Details" link shows:

diff --git a/js/src/tests/lib/__init__.py b/js/src/tests/lib/__init__.py
new file mode 100644
diff --git a/js/src/tests/manifest.py b/js/src/tests/lib/manifest.py
rename from js/src/tests/manifest.py
rename to js/src/tests/lib/manifest.py
diff --git a/js/src/tests/progressbar.py b/js/src/tests/lib/progressbar.py
rename from js/src/tests/progressbar.py
rename to js/src/tests/lib/progressbar.py
diff --git a/js/src/tests/results.py b/js/src/tests/lib/results.py
rename from js/src/tests/results.py
rename to js/src/tests/lib/results.py
diff --git a/js/src/tests/tasks_unix.py b/js/src/tests/lib/tasks_unix.py
rename from js/src/tests/tasks_unix.py
rename to js/src/tests/lib/tasks_unix.py
diff --git a/js/src/tests/tasks_win.py b/js/src/tests/lib/tasks_win.py
rename from js/src/tests/tasks_win.py
rename to js/src/tests/lib/tasks_win.py
diff --git a/js/src/tests/tests.py b/js/src/tests/lib/tests.py
rename from js/src/tests/tests.py
rename to js/src/tests/lib/tests.py

at the bottom, but the Diff and Splinter Review links don't mention these files at all.
(Assignee)

Updated

5 years ago
Attachment #621161 - Flags: review?(dmandelin)
Comment on attachment 621161 [details] [diff] [review]
v0

Oh, OK, it looks like Splinter wasn't smart enough to pick up the moves.
Attachment #621161 - Flags: review?(dmandelin) → review+
(Assignee)

Comment 4

5 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/6e821143ee3e
https://hg.mozilla.org/mozilla-central/rev/6e821143ee3e
Target Milestone: --- → mozilla15
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.