Adds clone tests and updates dataType tests for variables. Closes #55. (

#57)
material-foundation · Jan 9, 2017 · 8949d8d · 8949d8d
1 parent 83e326d
commit 8949d8d
Show file tree

Hide file tree

Showing 5 changed files with 34 additions and 0 deletions.
diff --git a/src/core/__tests__/BooleanVariable_test.ts b/src/core/__tests__/BooleanVariable_test.ts
@@ -54,4 +54,9 @@ describe("BooleanVariable", () => {
     expect(callbackSpy).to.have.been.calledOnce.and.calledWith(variable);
     expect(updatedVariable.selectedValue).to.equal(newValue);
   });
+
+  it("should clone properly", () => {
+    let clone = variable.clone();
+    expect(JSON.stringify(clone)).to.equal(JSON.stringify(variable));
+  });
 });
diff --git a/src/core/__tests__/ColorVariable_test.ts b/src/core/__tests__/ColorVariable_test.ts
@@ -38,6 +38,9 @@ describe("ColorVariable", () => {
 
   it("have the correct contraintType", () => {
     expect(variable.constraintType).to.equal(ConstraintType.LIST);
+
+    variable.possibleValues = [];
+    expect(variable.constraintType).to.equal(ConstraintType.NONE);
   });
 
   it("have the correct controlType", () => {
@@ -73,4 +76,9 @@ describe("ColorVariable", () => {
     expect(callbackSpy).to.have.been.calledOnce.and.calledWith(variable);
     expect(updatedVariable.selectedValue).to.equal(newValue);
   });
+
+  it("should clone properly", () => {
+    let clone = variable.clone();
+    expect(JSON.stringify(clone)).to.equal(JSON.stringify(variable));
+  });
 });
diff --git a/src/core/__tests__/NumberVariable_test.ts b/src/core/__tests__/NumberVariable_test.ts
@@ -38,6 +38,9 @@ describe("NumberVariable", () => {
 
   it("have the correct contraintType", () => {
     expect(variable.constraintType).to.equal(ConstraintType.LIST);
+
+    variable.possibleValues = [];
+    expect(variable.constraintType).to.equal(ConstraintType.NONE);
   });
 
   it("should have correct controlType based on number of allowed values", () => {
@@ -73,4 +76,9 @@ describe("NumberVariable", () => {
     expect(callbackSpy).to.have.been.calledOnce.and.calledWith(variable);
     expect(updatedVariable.selectedValue).to.equal(newValue);
   });
+
+  it("should clone properly", () => {
+    let clone = variable.clone();
+    expect(JSON.stringify(clone)).to.equal(JSON.stringify(variable));
+  });
 });
diff --git a/src/core/__tests__/RangeVariable_test.ts b/src/core/__tests__/RangeVariable_test.ts
@@ -70,4 +70,9 @@ describe("RangeVariable", () => {
     expect(callbackSpy).to.have.been.calledOnce.and.calledWith(variable);
     expect(updatedVariable.selectedValue).to.equal(newValue);
   });
+
+  it("should clone properly", () => {
+    let clone = variable.clone();
+    expect(JSON.stringify(clone)).to.equal(JSON.stringify(variable));
+  });
 });
diff --git a/src/core/__tests__/StringVariable_test.ts b/src/core/__tests__/StringVariable_test.ts
@@ -38,6 +38,9 @@ describe("StringVariable", () => {
 
   it("have the correct contraintType", () => {
     expect(variable.constraintType).to.equal(ConstraintType.LIST);
+
+    variable.possibleValues = [];
+    expect(variable.constraintType).to.equal(ConstraintType.NONE);
   });
 
   it("should have correct controlType based on number of allowed values", () => {
@@ -73,4 +76,9 @@ describe("StringVariable", () => {
     expect(callbackSpy).to.have.been.calledOnce.and.calledWith(variable);
     expect(updatedVariable.selectedValue).to.equal(newValue);
   });
+
+  it("should clone properly", () => {
+    let clone = variable.clone();
+    expect(JSON.stringify(clone)).to.equal(JSON.stringify(variable));
+  });
 });