TINYINT-3435: Add more tests

src/main/component/Utils.ts

@@ -81,15 +81,4 @@ export const bindHandlers = (editor: Editor, eventHandlers: Partial<EventHandler
       eventHandlers[eventName.toLowerCase()]?.(e, editor);
     });
   });
-};
-
-export const injectTiny = (doc: any, url: string, cb: any): void => {
-  const script = doc.createElement('script');
-  script.referrerPolicy = 'origin';
-  script.type = 'application/javascript';
-  script.src = url;
-  script.onload = cb;
-  if (doc.head) {
-    doc.head.appendChild(script);
-  }
 };

src/test/ts/alien/Loader.ts

@@ -38,6 +38,7 @@ export interface SvelteEditorContext extends Disposable {
   componentInstance: Record<string, any>;
   /** Update any props on the live component instance and flush Svelte reactivity synchronously. */
   setProps(patch: Partial<EditorProps>): void;
+  getProps(name: keyof EditorProps): unknown;
   remove(): void;
 }
 
@@ -86,6 +87,8 @@ export const render = async (props: EditorProps = {}): Promise<SvelteEditorConte
     flushSync();
   };
 
+  const getProps = (name: string): any => reactiveProps[name];
+
   const remove = () => {
     unmount(componentInstance).catch((reason) => {
       // eslint-disable-next-line no-console
@@ -99,6 +102,7 @@ export const render = async (props: EditorProps = {}): Promise<SvelteEditorConte
     DOMNode,
     componentInstance,
     setProps,
+    getProps,
     remove,
     [Symbol.dispose]: remove
   };

src/test/ts/browser/BindingTest.ts

@@ -0,0 +1,23 @@
+import { Assertions } from '@ephox/agar';
+import { describe, it } from '@ephox/bedrock-client';
+
+import * as Loader from '../alien/Loader';
+import { VALID_API_KEY, VERSIONS } from '../alien/TestHelpers';
+import { TinyAssertions } from '@ephox/mcagar';
+
+describe('BindingTest', () => {
+  VERSIONS.forEach((version) =>
+    Loader.withVersion(version, (render) => {
+      const defaultProps: Loader.EditorProps = { apiKey: VALID_API_KEY };
+
+      it('TINYINT-3435: value prop is bound to the editor\'s content', async () => {
+        const initialValue = '<p>Hello, World!</p>';
+        const newValue = '<p>New Content</p>';
+        using ctx = await render({ ...defaultProps, value: initialValue });
+        TinyAssertions.assertContent(ctx.editor, initialValue);
+        ctx.editor.execCommand('mceSetContent', false, newValue);
+        Assertions.assertEq('value prop should be updated', newValue, ctx.getProps('value'));
+      });
+    })
+  );
+});

src/test/ts/browser/EditorInitTest.ts

@@ -51,6 +51,16 @@ describe('EditorInitTest', () => {
         using ctx = await render({ ...defaultProps, value: '' });
         TinyAssertions.assertContent(ctx.editor, '');
       });
+
+      it('TINYINT-3435: cssClass is set to the container if provided', async () => {
+        using ctx = await render({ ...defaultProps, cssClass: 'test-class' });
+        Assertions.assertEq('className of containershould contain test-class', true, ctx.DOMNode.parentElement?.className === 'test-class');
+      });
+
+      it('TINYINT-3435: conf is passed as options to the editor', async () => {
+        using ctx = await render({ ...defaultProps, conf: { branding: false }});
+        Assertions.assertEq('branding option should be false', false, ctx.editor.options.get('branding'));
+      });
     })
   );
 });

src/test/ts/browser/EventsTest.ts

@@ -0,0 +1,31 @@
+import { TestStore } from '@ephox/agar';
+import { describe, it } from '@ephox/bedrock-client';
+
+import * as Loader from '../alien/Loader';
+import { VALID_API_KEY, VERSIONS } from '../alien/TestHelpers';
+
+describe('EventTest', () => {
+  VERSIONS.forEach((version) =>
+    Loader.withVersion(version, (render) => {
+      const store = TestStore<string>();
+      const eventHandlers = {
+        init: () => {
+          store.add('init');
+        },
+        loadcontent: () => {
+          store.add('loadcontent');
+        }
+      };
+      const defaultProps: Loader.EditorProps = { apiKey: VALID_API_KEY, ...eventHandlers };
+
+      it('TINYINT-3435: event handlers are handled correctly', async () => {
+        // eslint-disable-next-line @typescript-eslint/no-unused-vars
+        using ctx = await render({ ...defaultProps });
+        store.assertEq('Events should be fired', [
+          'loadcontent',
+          'init',
+        ]);
+      });
+    })
+  );
+});