@frontify/guideline-blocks-settings 0.31.0 → 0.31.2

package/CHANGELOG.md

@@ -1,5 +1,21 @@
 # @frontify/guideline-blocks-settings
 
+## 0.31.2
+
+### Patch Changes
+
+- Updated dependencies [[`68a9298`](https://github.com/Frontify/brand-sdk/commit/68a9298df9e2177e70333f47dc433a056d76625b)]:
+  - @frontify/app-bridge@3.1.0
+
+## 0.31.1
+
+### Patch Changes
+
+- [#708](https://github.com/Frontify/brand-sdk/pull/708) [`e2a0023`](https://github.com/Frontify/brand-sdk/commit/e2a0023256001dcc64a597f6dcd9df66a1c9f186) Thanks [@ragi96](https://github.com/ragi96)! - feat: add support for @frontify/app-bridge 4.0.0-alpha.0
+
+- Updated dependencies [[`c98d8c4`](https://github.com/Frontify/brand-sdk/commit/c98d8c414b2cdd00d4945f0c29581370b0a7daa0)]:
+  - @frontify/app-bridge@3.0.4
+
 ## 0.31.0
 
 ### Minor Changes

package/dist/index.d.ts

@@ -1,5 +1,3 @@
-/// <reference types="react" />
-
 import { AnyObject } from '@udecode/plate';
 import { AppBridgeBlock } from '@frontify/app-bridge';
 import type { AppBridgeTheme } from '@frontify/app-bridge';

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@frontify/guideline-blocks-settings",
-  "version": "0.31.0",
+  "version": "0.31.2",
   "description": "Provides types and helpers for the guideline block development",
   "sideEffects": false,
   "main": "dist/index.umd.js",
@@ -65,7 +65,7 @@
     "@frontify/sidebar-settings": "^0.9.1"
   },
   "peerDependencies": {
-    "@frontify/app-bridge": "^3.0.0",
+    "@frontify/app-bridge": "^3.0.0 || ^4.0.0-alpha.0",
     "react": "^18",
     "react-dom": "^18"
   },

package/setupTests.ts

@@ -1,11 +1,16 @@
 /* (c) Copyright Frontify Ltd., all rights reserved. */
 
-import { afterEach, vi } from 'vitest';
+import { afterEach, beforeAll, vi } from 'vitest';
+import { configure } from '@testing-library/react';
 
 vi.stubGlobal('crypto', {
     getRandomValues: vi.fn(),
 });
 
+beforeAll(() => {
+    configure({ testIdAttribute: 'data-test-id' });
+});
+
 afterEach(() => {
     vi.restoreAllMocks();
 });

package/src/components/Attachments/Attachments.spec.ct.tsx

@@ -43,6 +43,12 @@ const isPre302Stub = async (appBridge: AppBridgeBlock): Promise<boolean> => {
     return context === undefined;
 };
 
+const hasOpenAssetChooser = (
+    appBridge: AppBridgeBlock,
+): appBridge is AppBridgeBlock & { openAssetChooser: SinonStub } => {
+    return 'openAssetChooser' in appBridge;
+};
+
 describe('Attachments', () => {
     it('renders attachments flyout if it is in edit mode', () => {
         mount(<Attachments appBridge={getAppBridgeBlockStub({ editorState: true })} />);
@@ -94,7 +100,7 @@ describe('Attachments', () => {
             editorState: true,
         });
 
-        if (await isPre302Stub(appBridge)) {
+        if ((await isPre302Stub(appBridge)) && hasOpenAssetChooser(appBridge)) {
             (appBridge.openAssetChooser as SinonStub) = cy.stub().callsArgWith(0, AssetDummy.with(4));
         }
 

package/src/hooks/useAttachments.spec.tsx

@@ -8,11 +8,7 @@ import { AttachmentsProvider, useAttachments, useAttachmentsContext, withAttachm
 
 const MOCK_SETTINGS_ID = 'attachments';
 
-/**
- * @vitest-environment happy-dom
- */
-
-describe('useAttachments', () => {
+describe('useAttachments', async () => {
     it('should have 1 attachment if attachment is added', async () => {
         const STUB_WITH_NO_ASSETS = getAppBridgeBlockStub({
             blockId: 1,
@@ -115,7 +111,7 @@ describe('withAttachmentsProvider', () => {
             return (
                 <ul>
                     {context.attachments.map((attachment) => (
-                        <li key={attachment.id} role="list">
+                        <li key={attachment.id} data-test-id="item-test">
                             {attachment.id}
                         </li>
                     ))}
@@ -123,8 +119,8 @@ describe('withAttachmentsProvider', () => {
             );
         }, MOCK_SETTINGS_ID);
 
-        const { getAllByRole } = render(<Component appBridge={STUB_WITH_THREE_ASSETS} />);
+        const { getAllByTestId } = render(<Component appBridge={STUB_WITH_THREE_ASSETS} />);
 
-        await waitFor(() => expect(getAllByRole('list')).toHaveLength(3));
+        await waitFor(() => expect(getAllByTestId('item-test')).toHaveLength(3));
     });
 });