Bug 466093 - [tern] Migrate the rest of the content assist tests and provide more worker-based tests
Summary: [tern] Migrate the rest of the content assist tests and provide more worker-b...
Status: RESOLVED FIXED
Alias: None
Product: Orion (Archived)
Classification: ECD
Component: JS Tools (show other bugs)
Version: 8.0   Edit
Hardware: All All
: P2 normal (vote)
Target Milestone: 12.0   Edit
Assignee: Michael Rennie CLA
QA Contact:
URL:
Whiteboard:
Keywords: test
Depends on:
Blocks:
 
Reported: 2015-05-01 11:54 EDT by Michael Rennie CLA
Modified: 2016-05-16 11:37 EDT (History)
2 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Rennie CLA 2015-05-01 11:54:55 EDT
We need to finish migrating the content assist tests (there are a few hundred of them) to the new worker-based Tern suite.

We also need to provide worker-based tests for:

1. finding decls
2. computing hovers
3. renaming
Comment 1 Michael Rennie CLA 2015-07-30 14:21:44 EDT
migrated the @type JSDoc tag usages tests:

http://git.eclipse.org/c/orion/org.eclipse.orion.client.git/commit/?id=ca606596f5eb3d42a8bd01b571c954a5cf1bf6b8
Comment 2 Eric Moffatt CLA 2015-09-29 13:55:49 EDT
Michael, is this done yet ? If not how far along are we ?
Comment 3 Michael Rennie CLA 2015-09-29 14:01:12 EDT
(In reply to Eric Moffatt from comment #2)
> Michael, is this done yet ? If not how far along are we ?

Any test left in the the file js-tests/javascript/contentAssistTests.js should be migrated to the new tern-worker-based tests.

That being said, I have been putting emphasis on tests that are in there due to bug fixes (look for @since or @see tags, or bug URLs).
Comment 4 Michael Rennie CLA 2016-05-16 11:37:05 EDT
We have all the test coverage form the class that we want.

I removed the rest of the class:

http://git.eclipse.org/c/orion/org.eclipse.orion.client.git/commit/?id=3f7faec0be16926091dfd118ed51c24357f3ed85