fixed a bug where the same baseUrl could appear in the autocomplete more than once

Author: corevo

packages/selenium-ide/src/neo/__test__/stores/domain/ProjectStore.spec.js

@@ -167,7 +167,7 @@ describe("Project Store", () => {
     const projectRep = {
       id: "1",
       name: "my project",
-      url: "https://en.wikipedia.org",
+      url: "https://en.wikipedia.org/",
       tests: [
         {
           id: "1",
@@ -203,8 +203,8 @@ describe("Project Store", () => {
         }
       ],
       urls: [
-        "https://en.wikipedia.org",
-        "http://www.seleniumhq.org"
+        "https://en.wikipedia.org/",
+        "http://www.seleniumhq.org/"
       ].sort(),
       plugins: [
         {
@@ -242,6 +242,19 @@ describe("Project Store", () => {
     project.fromJS(projectRep);
     expect(project.id).toBeDefined();
   });
+  it("should remove duplicate urls when loaded", () => {
+    const projectRep = {
+      name: "my project",
+      url: "",
+      tests: [],
+      suites: [],
+      urls: ["https://seleniumhq.org/", "https://seleniumhq.org/"]
+    };
+
+    const project = new ProjectStore();
+    project.fromJS(projectRep);
+    expect(project.urls.length).toBe(1);
+  });
   it("should have a list of loaded plugins", () => {
     const project = new ProjectStore();
     expect(project.plugins).toBeDefined();

packages/selenium-ide/src/neo/stores/domain/ProjectStore.js

@@ -140,7 +140,10 @@ export default class ProjectStore {
     this.setUrl(jsRep.url);
     this._tests.replace(jsRep.tests.map(TestCase.fromJS));
     this._suites.replace(jsRep.suites.map((suite) => Suite.fromJS(suite, this.tests)));
-    this._urls.replace(jsRep.urls);
+    this._urls.clear();
+    jsRep.urls.forEach((url) => {
+      this.addUrl(url);
+    });
     this.plugins.replace(jsRep.plugins);
     this.id = jsRep.id || uuidv4();
     this.setModified(false);