Opened 15 years ago
Closed 14 years ago
#3009 closed Bug (fixed)
Enhance testing framework to support async 'setUp'
| Reported by: | Garry Yao | Owned by: | Garry Yao |
|---|---|---|---|
| Priority: | Normal | Milestone: | CKEditor 3.1 |
| Component: | QA | Version: | 3.0 Beta 2 |
| Keywords: | Confirmed HasTest | Cc: |
Description (last modified by )
In functional tests where always requires to have a fully interacted editor instance as prerequisites, with all tests in a case share a 'playground' instance.
We should provide a customized TestSuite which responsible for setting up a editor instance transparently with all containing TCs share this singleton, with this, it even able to make sure each test 's cleanness in a TC by resetting the editor with TestCase::setUp.
Since the setting up of our editor is always an asynchronous operation, so does cleaning up an editor sometimes, sadly that YUI doesn't support calling wait/resume inside setUp method, we should work around it in order to bring the above functionality into truth.
Attachments (3)
Change History (14)
comment:1 Changed 15 years ago by
| Keywords: | Confirmed added |
|---|---|
| Owner: | set to Garry Yao |
| Status: | new → assigned |
comment:2 Changed 15 years ago by
| Description: | modified (diff) |
|---|
Changed 15 years ago by
| Attachment: | 3009.patch added |
|---|
comment:3 Changed 15 years ago by
| Keywords: | Review? added |
|---|
comment:4 Changed 14 years ago by
| Component: | General → QA |
|---|---|
| Milestone: | CKEditor 3.0 |
comment:5 Changed 14 years ago by
| Version: | SVN (FCKeditor) → CKEditor 3.0 Beta 2 |
|---|
comment:6 Changed 14 years ago by
| Keywords: | HasPatch added; Review? removed |
|---|---|
| Milestone: | → CKEditor 3.1 |
The correct way is having a dedicated 'YAHOO.tool.TestSuite' instead of 'YAHOO.tool.TestCase', any editor based TCs should be grouped into it.
Changed 14 years ago by
| Attachment: | 3009_2.patch added |
|---|
comment:7 Changed 14 years ago by
| Keywords: | HasTest Review? added; HasPatch removed |
|---|
Ticket test at: source:CKEditor/branches/tests/editor/tt/3009/3009.html
comment:8 Changed 14 years ago by
| Description: | modified (diff) |
|---|
Update description according to recent investigation.
Changed 14 years ago by
| Attachment: | 3009_3.patch added |
|---|
comment:9 Changed 14 years ago by
The new path contains the following changes:
- Enable 'wait/resume' support in 'setUp' for both TestSuite/TestCase;
- Decouple YUI Test enhancement to dedicated YUI extension.
- Update TC with [4106].
comment:10 Changed 14 years ago by
| Keywords: | Review? removed |
|---|
comment:11 Changed 14 years ago by
| Resolution: | → fixed |
|---|---|
| Status: | assigned → closed |
Revised to a new approach to work arround this.