swapTestCases and containsTest methods

Author: corevo

packages/selenium-ide/src/neo/__test__/models/Suite.spec.js

@@ -120,6 +120,26 @@ describe("Suite model", () => {
     suite.removeTestCase(new TestCase());
     expect(suite.tests.length).toBe(1);
   });
+  it("shoul tell if a test exist in the suite", () => {
+    const suite = new Suite();
+    const exists = new TestCase();
+    const nonExistent = new TestCase();
+    suite.addTestCase(exists);
+    expect(suite.containsTest(exists)).toBeTruthy();
+    expect(suite.containsTest(nonExistent)).toBeFalsy();
+  });
+  it("should swap the test cases", () => {
+    const suite = new Suite();
+    const test1 = new TestCase();
+    const test2 = new TestCase();
+    suite.addTestCase(test1);
+    suite.addTestCase(test2);
+    expect(suite.tests[0]).toBe(test1);
+    expect(suite.tests[1]).toBe(test2);
+    suite.swapTestCases(0, 1);
+    expect(suite.tests[1]).toBe(test1);
+    expect(suite.tests[0]).toBe(test2);
+  });
   it("should replace the tests in the suite", () => {
     const store = new ProjectStore();
     const suite = new Suite();

packages/selenium-ide/src/neo/models/Suite.js

@@ -36,6 +36,7 @@ export default class Suite {
   }
 
   @computed get tests() {
+    return this._tests;
     return this._tests.sort((t1, t2) => (
       naturalCompare(t1.name, t2.name)
     ));
@@ -71,6 +72,10 @@ export default class Suite {
     }
   }
 
+  @action.bound containsTest(test) {
+    return this._tests.includes(test);
+  }
+
   @action.bound addTestCase(test) {
     if (!this.isTest(test)) {
       throw new Error(`Expected to receive TestCase instead received ${test ? test.constructor.name : test}`);
@@ -79,6 +84,14 @@ export default class Suite {
     }
   }
 
+  @action.bound insertTestCaseAt(test, index) {
+    if (!this.isTest(test)) {
+      throw new Error(`Expected to receive TestCase instead received ${test ? test.constructor.name : test}`);
+    } else {
+      this._tests.splice(index, 0, test);
+    }
+  }
+
   @action.bound removeTestCase(test) {
     if (!this.isTest(test)) {
       throw new Error(`Expected to receive TestCase instead received ${test ? test.constructor.name : test}`);
@@ -87,6 +100,11 @@ export default class Suite {
     }
   }
 
+  @action.bound swapTestCases(from, to) {
+    const test = this._tests.splice(from, 1)[0];
+    this.insertTestCaseAt(test, to);
+  }
+
   @action.bound replaceTestCases(tests) {
     if (tests.filter(test => !this.isTest(test)).length) {
       throw new Error("Expected to receive array of TestCase");