Merge pull request #583 from tweenjs/convert-tests-to-typescript

chore: convert test code to TypeScript

Author: trusktr

README.md

@@ -239,15 +239,13 @@ You need to install `npm` first--this comes with node.js, so install that one fi
 npm install
 ```
 
-if running the tests for the first time, to install additional dependencies for running tests, and then run
+To run the tests run:
 
 ```bash
 npm test
 ```
 
-every time you want to run the tests.
-
-If you want to add any feature or change existing features, you _must_ run the tests to make sure you didn't break anything else. If you send a pull request (PR) to add something new and it doesn't have tests, or the tests don't pass, the PR won't be accepted. See [contributing](CONTRIBUTING.md) for more information.
+If you want to add any feature or change existing features, you _must_ run the tests to make sure you didn't break anything else. Any pull request (PR) needs to have updated passing tests for feature changes (or new passing tests for new features) in `src/tests.ts`, otherwise the PR won't be accepted. See [contributing](CONTRIBUTING.md) for more information.
 
 ## People
 

package.json

@@ -31,7 +31,7 @@
 		"tsc": "tsc",
 		"tsc-watch": "tsc --watch",
 		"examples": "npx serve .",
-		"test": "npm run build && npm run test-unit && npm run test-lint",
+		"test": "npm run build && npm run test-lint && npm run test-unit",
 		"test-unit": "nodeunit test/unit/nodeunitheadless.js",
 		"test-lint": "npm run prettier -- --check && eslint 'src/**/*.ts'",
 		"lint": "npm run prettier -- --write && eslint 'src/**/*.ts' --fix",

rollup.config.js

@@ -30,6 +30,12 @@ export default [
 			},
 		],
 	},
+	{
+		input: '.tmp/tests.js',
+		context: 'this',
+		watch: {clearScreen: false},
+		output: [{file: '.tmp/tests.cjs.js', format: 'cjs', exports: 'named'}],
+	},
 	{
 		input: './.tmp/Index.d.ts',
 		watch: {clearScreen: false},

src/Tween.ts

@@ -34,7 +34,8 @@ export class Tween<T extends UnknownProps> {
 	private _startTime = 0
 	private _easingFunction: EasingFunction = Easing.Linear.None
 	private _interpolationFunction: InterpolationFunction = Interpolation.Linear
-	private _chainedTweens: Array<Tween<UnknownProps>> = []
+	// eslint-disable-next-line
+	private _chainedTweens: Array<Tween<any>> = []
 	private _onStartCallback?: (object: T) => void
 	private _onStartCallbackFired = false
 	private _onUpdateCallback?: (object: T, elapsed: number) => void
@@ -72,7 +73,7 @@ export class Tween<T extends UnknownProps> {
 		return this
 	}
 
-	duration(d: number): this {
+	duration(d = 1000): this {
 		this._duration = d
 		return this
 	}
@@ -256,68 +257,69 @@ export class Tween<T extends UnknownProps> {
 		return this
 	}
 
-	group(group: Group): this {
+	group(group = mainGroup): this {
 		this._group = group
 		return this
 	}
 
-	delay(amount: number): this {
+	delay(amount = 0): this {
 		this._delayTime = amount
 		return this
 	}
 
-	repeat(times: number): this {
+	repeat(times = 0): this {
 		this._initialRepeat = times
 		this._repeat = times
 		return this
 	}
 
-	repeatDelay(amount: number): this {
+	repeatDelay(amount?: number): this {
 		this._repeatDelayTime = amount
 		return this
 	}
 
-	yoyo(yoyo: boolean): this {
+	yoyo(yoyo = false): this {
 		this._yoyo = yoyo
 		return this
 	}
 
-	easing(easingFunction: EasingFunction): this {
+	easing(easingFunction: EasingFunction = Easing.Linear.None): this {
 		this._easingFunction = easingFunction
 		return this
 	}
 
-	interpolation(interpolationFunction: InterpolationFunction): this {
+	interpolation(interpolationFunction: InterpolationFunction = Interpolation.Linear): this {
 		this._interpolationFunction = interpolationFunction
 		return this
 	}
 
-	chain(...tweens: Array<Tween<UnknownProps>>): this {
+	// eslint-disable-next-line
+	chain(...tweens: Array<Tween<any>>): this {
 		this._chainedTweens = tweens
 		return this
 	}
 
-	onStart(callback: (object: T) => void): this {
+	onStart(callback?: (object: T) => void): this {
 		this._onStartCallback = callback
 		return this
 	}
 
-	onUpdate(callback: (object: T, elapsed: number) => void): this {
+	onUpdate(callback?: (object: T, elapsed: number) => void): this {
 		this._onUpdateCallback = callback
 		return this
 	}
 
-	onRepeat(callback: (object: T) => void): this {
+	onRepeat(callback?: (object: T) => void): this {
 		this._onRepeatCallback = callback
 		return this
 	}
 
-	onComplete(callback: (object: T) => void): this {
+	onComplete(callback?: (object: T) => void): this {
 		this._onCompleteCallback = callback
 		return this
 	}
 
-	onStop(callback: (object: T) => void): this {
+	onStop(callback?: (object: T) => void): this {
 		this._onStopCallback = callback
 		return this
 	}