fix failing test

.github/workflows/ci.yml

@@ -34,12 +34,6 @@ jobs:
     - uses: actions/setup-node@v4
       with:
         node-version-file: '.nvmrc'
-    - uses: actions/cache@v1
-      with:
-        path: ~/.npm
-        key: ${{ runner.os }}-node-${{ hashFiles('**/package-lock.json') }}
-        restore-keys: |
-          ${{ runner.os }}-node-
     - run: npm ci
     - run: npm run build
     - run: npm run lint
@@ -57,12 +51,6 @@ jobs:
     - uses: actions/setup-node@v4
       with:
         node-version-file: '.nvmrc'
-    - uses: actions/cache@v1
-      with:
-        path: ~/.npm
-        key: ${{ runner.os }}-node-${{ hashFiles('**/package-lock.json') }}
-        restore-keys: |
-          ${{ runner.os }}-node-
     - run: npm ci
     - run: npm run build
     - run: npm run build-storybook

cypress/e2e/accessibility.cy.ts

@@ -14,7 +14,7 @@ describe("accessibility", () => {
       const selector = "root:skip:layer-list";
       then(get.elementByTestId(selector)).shouldExist();
       when.tab();
-      then(get.skipTargetLayerList()).shouldBeFocused();
+      then(get.elementByTestId(selector)).shouldBeFocused();
       when.click(selector);
       then(get.skipTargetLayerList()).shouldBeFocused();
     });

cypress/e2e/maputnik-driver.ts

@@ -87,18 +87,18 @@ export default class MaputnikDriver {
     ) => {
       let url = "?debug";
       switch (styleProperties) {
-        case "geojson":
-          url += `&style=${baseUrl}geojson-style.json`;
-          break;
-        case "raster":
-          url += `&style=${baseUrl}raster-style.json`;
-          break;
-        case "both":
-          url += `&style=${baseUrl}geojson-raster-style.json`;
-          break;
-        case "layer":
-          url += `&style=${baseUrl}/example-layer-style.json`;
-          break;
+      case "geojson":
+        url += `&style=${baseUrl}geojson-style.json`;
+        break;
+      case "raster":
+        url += `&style=${baseUrl}raster-style.json`;
+        break;
+      case "both":
+        url += `&style=${baseUrl}geojson-raster-style.json`;
+        break;
+      case "layer":
+        url += `&style=${baseUrl}/example-layer-style.json`;
+        break;
       }
       if (zoom) {
         url += `#${zoom}/41.3805/2.1635`;
@@ -111,7 +111,7 @@ export default class MaputnikDriver {
       this.helper.get.elementByTestId("toolbar:link").should("be.visible");
     },
 
-    typeKeys: (keys: string, selector?: string) =>
+    typeKeys: (keys: string) =>
       this.helper.get.element("body").type(keys),
 
     clickZoomIn: () => {
@@ -192,9 +192,9 @@ export default class MaputnikDriver {
     },
 
     styleStoreEqualToExampleFileData: () => {
-      cy.window().then((win: any) => {
-        const obj = this.get.styleFromWindow(win);
-        const bobj = this.get.maputnikStyleFromLocalStorageObj();
+      cy.window().then((_win: any) => {
+        //const obj = this.get.styleFromWindow(win);
+        const obj = this.get.maputnikStyleFromLocalStorageObj();
         this.helper.given
           .fixture("example-style.json", "file:example-style.json")
           .should("deep.equal", obj);

cypress/e2e/modals.cy.ts

@@ -96,7 +96,7 @@ describe("modals", () => {
       });
     });
 
-    describe("when click owner", () => {
+    describe("when set name and click owner", () => {
       beforeEach(() => {
         when.setValue("modal:settings.name", "foobar");
         when.click("modal:settings.owner");
@@ -110,13 +110,19 @@ describe("modals", () => {
       });
     });
 
-    it("owner", () => {
-      when.setValue("modal:settings.owner", "foobar");
-      when.click("modal:settings.name");
-      then(get.maputnikStyleFromLocalStorage()).shouldDeepNestedInclude({
-        owner: "foobar",
+    describe("when set owner and click name", () => {
+      beforeEach(() => {
+        when.setValue("modal:settings.owner", "foobar");
+        when.click("modal:settings.name");
+        when.wait(200);
+      });
+      it("should update owner in local storage", () => {
+        then(get.maputnikStyleFromLocalStorage()).shouldDeepNestedInclude({
+          owner: "foobar",
+        });
       });
     });
+
     it("sprite url", () => {
       when.setValue("modal:settings.sprite", "http://example.com");
       when.click("modal:settings.name");