Skip to content
This repository was archived by the owner on Nov 25, 2020. It is now read-only.

Test stubs for Core modules #4

Merged
merged 4 commits into from
Mar 20, 2014
Merged

Test stubs for Core modules #4

merged 4 commits into from
Mar 20, 2014

Conversation

wookiehangover
Copy link
Contributor

What does it do?

  • Adds basic test structure for modules in Core, with basic existence and instantiation tests
  • moves support code in core into its own module

Where should the reviewer start?

Look at the test files in src/core/test

How should this be manually tested?

Run gulp and then go to http://localhost:8080/test/index.html to run the test suite :feelsgood:

@@ -12,6 +12,9 @@

module.exports = class KDEventEmitter

# FIXME (sb) - this class has no knowledge of @Wildcard (and requiring it
# would make a circular dependency). Also, this doesn't appear to be used
# anywhere in the framework.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@humanchimp any note here for an improvement?

sinan added a commit that referenced this pull request Mar 20, 2014
Test stubs for Core modules
@sinan sinan merged commit 4625df7 into master Mar 20, 2014
@sinan sinan deleted the core-tests branch March 20, 2014 00:45
szkl pushed a commit that referenced this pull request May 10, 2014
sinan pushed a commit to sinan/kd that referenced this pull request Feb 27, 2015
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants