Fix for rwaldron#1499

If an array of keyFrames are passed to Servos, those keyFrames should be used as values for the whole collection and should not be normalized as a keyFrameSet.
dtex · Sep 11, 2018 · bff5d8a · bff5d8a
1 parent 210aaf4
commit bff5d8a
Show file tree

Hide file tree

Showing 3 changed files with 61 additions and 8 deletions.
diff --git a/lib/servo.js b/lib/servo.js
@@ -326,7 +326,7 @@ Servo.prototype.to = function(degrees, time, rate) {
  */
 
 Servo.prototype[Animation.normalize] = function(keyFrames) {
-
+  
   var last = this.last ? this.last.target : this.startAt;
 
   // If user passes null as the first element in keyFrames use current position
@@ -618,9 +618,9 @@ Collection.installMethodForwarding(
  */
 Servos.prototype[Animation.normalize] = function(keyFrameSet) {
   return keyFrameSet.map(function(keyFrames, index) {
-    if (keyFrames !== null) {
+    if (keyFrames !== null && Array.isArray(keyFrames)) {
       var servo = this[index];
-
+     
       // If servo is a servoArray then user servo[0] for default values
       if (servo instanceof Servos) {
         servo = servo[0];
@@ -642,7 +642,7 @@ Servos.prototype[Animation.normalize] = function(keyFrameSet) {
           };
         }
       }
-
+      
       return this[index][Animation.normalize](keyFrames);
     }
     return keyFrames;

diff --git a/test/extended/servo.js b/test/extended/servo.js
@@ -144,7 +144,7 @@ exports["Servo"] = {
 
     this.servo.on("move:complete", function() {
       test.equal(this.servo.position, 0);
-      test.ok(this.servoWrite.callCount === 101);
+      test.ok(this.servoWrite.callCount - 100 <= 1);
       test.done();
     }.bind(this));
   },
@@ -163,7 +163,7 @@ exports["Servo"] = {
 
     this.servo.on("move:complete", function() {
       test.equal(this.servo.position, 180);
-      test.ok(this.servoWrite.callCount === 101);
+      test.ok(this.servoWrite.callCount - 100 <= 1);
       test.done();
     }.bind(this));
   },
@@ -217,7 +217,7 @@ exports["Servo"] = {
 
     this.servo.on("move:complete", function() {
       test.equal(this.servo.value, 80);
-      test.equal(this.servoWrite.lastCall.args[1], 70);
+      test.equal(this.servoWrite.lastCall.args[1], 1300);
       test.done();
     }.bind(this));
 
@@ -237,7 +237,7 @@ exports["Servo"] = {
 
     this.servo.on("move:complete", function() {
       test.equal(this.servo.value, 80);
-      test.equal(this.servoWrite.lastCall.args[1], 110);
+      test.equal(this.servoWrite.lastCall.args[1], 1700);
       test.done();
     }.bind(this));
 

diff --git a/test/servo.collection.js b/test/servo.collection.js
@@ -21,6 +21,11 @@ exports["Servo.Collection"] = {
       pin: 9,
       board: this.board
     });
+
+    this.d = new Servo({
+      pin: 11,
+      board: this.board
+    });
 
     this.spies = [
       "to", "stop"
@@ -127,6 +132,54 @@ exports["Servo.Collection"] = {
     test.done();
   },
 
+  "Animation.normalize-nested": function(test) {
+    test.expect(1);
+
+    var group1 = new Servos([
+      this.a, this.b
+    ]);
+
+    var group2 = new Servos([
+      this.c, this.d
+    ]);
+
+    var bothGroups = new Servos([
+      group1, group2
+    ]);
+
+    var normalized = bothGroups[Animation.normalize]([
+      [
+        [
+          null,
+          10,
+        ]
+      ],
+      [
+        [
+          null,
+          20,
+        ]
+      ]
+    ]);
+
+    test.deepEqual(normalized, [
+      [ 
+        [ 
+          { value: 90, easing: "linear" }, 
+          { step: 10, easing: "linear" }
+        ]
+      ],
+      [ 
+        [ 
+          { value: 90, easing: "linear" },
+          { step: 20, easing: "linear" }
+        ]
+      ]
+    ]);
+
+    test.done();
+  },
+
   "Animation.normalize": function(test) {
     test.expect(3);