Skip to content
This repository has been archived by the owner on Jun 17, 2024. It is now read-only.

Testable #14

Merged
merged 5 commits into from
Mar 21, 2016
Merged

Testable #14

merged 5 commits into from
Mar 21, 2016

Conversation

jpgarcia
Copy link
Contributor

Changes in this PR:

Typescript

  • Moved from internal modules to external modules (more info: http://www.typescriptlang.org/Handbook#modules-pitfalls-of-modules)
  • Added kurve.ts entry point to wrap the public exports in a single object (this ensures we are keeping the same interface as the previous version but making the inner objects testable)
  • Remove tsd in favor of typings
  • Updated to the latest version (1.8.9)

Unit testing

  • Added the following dependencies to setup the infrastructure: mocha, chai, sinon, ts-node
  • Implemented the very first unit test just as starting point

Build process

  • Typescript build, minification, sourcemaps and bundle gulp tasks were replaced with Webpack
  • Single Typescript definition file for Kuve created with dts-generator instead of gulp/merge2 task (according the this thread typescript will support this OOB in a future release)
  • Added npm task for running tests

Pending Enhancements

  • Remove all dist-* folders and keep a single dist folder with latest binaries. old distributions could be tracked as Github releases
  • Make sample easy to run with another npm task
  • Refactor Typescript classes in separate files
  • Add Tslint for source code static analysis

@billba
Copy link
Contributor

billba commented Mar 21, 2016

FYI I asked the SW folks to post this as a work in progress.

billba added a commit that referenced this pull request Mar 21, 2016
@billba billba merged commit b438e58 into MicrosoftDX:testable Mar 21, 2016
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