chore: set keepAspectRatio to true as default in image-asset (NativeScript#5613)

tbozhikov · SvetoslavTsenov · commit c06d5bae63e5 · 2018-04-10T00:46:27.000+03:00
* chore: set keepAspectRatio to true as default in image-asset

* fix: remove empty lines
diff --git a/tests/app/image-source/image-source-tests.ts b/tests/app/image-source/image-source-tests.ts
@@ -15,7 +15,7 @@ export function testFromResource() {
     // >> imagesource-resname
     const img = imageSource.fromResource("icon");
     // << imagesource-resname
-    
+
     TKUnit.assert(img.height > 0, "image.fromResource failed");
 }
 
@@ -149,6 +149,22 @@ export function testFromAssetWithScalingAndAspectRatio(done) {
     });
 }
 
+export function testFromAssetWithScalingAndDefaultAspectRatio(done) {
+    let asset = new imageAssetModule.ImageAsset(splashscreenPath);
+    let scaleWidth = 10;
+    let scaleHeight = 11;
+    asset.options.width = scaleWidth;
+    asset.options.height = scaleHeight;
+
+    let img = imageSource.fromAsset(asset).then((source) => {
+        TKUnit.assertEqual(source.width, 18);
+        TKUnit.assertEqual(source.height, scaleHeight);
+        done();
+    }, (error) => {
+        done(error);
+    });
+}
+
 export function testFromAssetWithBiggerScaling(done) {
     let asset = new imageAssetModule.ImageAsset(splashscreenPath);
     let scaleWidth = 600;
diff --git a/tns-core-modules/image-asset/image-asset-common.ts b/tns-core-modules/image-asset/image-asset-common.ts
@@ -9,6 +9,11 @@ export class ImageAsset  extends observable.Observable implements definition.Ima
     ios: PHAsset;
     android: string;
 
+    constructor () {
+        super();
+        this._options = { keepAspectRatio: true };
+    }
+
     get options(): definition.ImageAssetOptions {
         return this._options;
     }
