Merge pull request #1 from mastilver/passes-values-through

feat: passes values through

index.js

@@ -3,5 +3,16 @@ var isObservable = require('is-observable');
 var symbolObservable = require('symbol-observable');
 
 module.exports = function (x) {
-	return isObservable(x) ? x[symbolObservable]().forEach(function () {}) : x;
+	if (isObservable(x)) {
+		var result = [];
+
+		return x[symbolObservable]().forEach(function (value) {
+			result.push(value);
+		})
+		.then(function () {
+			return result;
+		});
+	}
+
+	return Promise.resolve(x);
 };

readme.md

@@ -17,8 +17,9 @@ $ npm install --save observable-to-promise
 ```js
 const observableToPromise = require('observable-to-promise');
 
-observableToPromise(Observable.of(1, 2)).then(() => {
-	console.log('done');
+observableToPromise(Observable.of(1, 2)).then((result) => {
+	console.log(result);
+	// => [1, 2]
 });
 ```
 

test.js

@@ -10,3 +10,18 @@ test(t => {
 	t.true(isPromise(fn(zenObservable.of(1, 2))));
 	t.end();
 });
+
+test(t => {
+	fn(2).then(x => {
+		t.is(x, 2);
+		t.end();
+	});
+});
+
+test(t => {
+	fn(zenObservable.of(1, 2))
+	.then(result => {
+		t.same(result, [1, 2]);
+		t.end();
+	});
+});