add more tests for new-frameworks utils

code/lib/cli/src/automigrate/fixes/new-frameworks/utils.test.ts

@@ -1,4 +1,4 @@
-import { getBuilderInfo as _getBuilderInfo } from './utils';
+import { getBuilderInfo as _getBuilderInfo, getNextjsAddonOptions } from './utils';
 
 type GetBuilderInfoParams = Parameters<typeof _getBuilderInfo>[0];
 
@@ -11,27 +11,20 @@ describe('getBuilderInfo', () => {
       getBuilderInfo({
         core: { builder: '@storybook/builder-webpack5' },
       })
-    ).toEqual({
+    ).toEqual({ name: 'webpack5', options: {} });
-      name: 'webpack5',
-      options: {},
-    });
 
     expect(
       getBuilderInfo({
         core: {
           builder: {
             name: '@storybook/builder-webpack5',
-            options: {
+            options: { lazyCompilation: true },
-              lazyCompilation: true,
-            },
           },
         },
       })
     ).toEqual({
       name: 'webpack5',
-      options: {
+      options: { lazyCompilation: true },
-        lazyCompilation: true,
-      },
     });
   });
 
@@ -40,10 +33,7 @@ describe('getBuilderInfo', () => {
       getBuilderInfo({
         framework: '@storybook/react-webpack5',
       })
-    ).toEqual({
+    ).toEqual({ name: 'webpack5', options: {} });
-      name: 'webpack5',
-      options: {},
-    });
 
     expect(
       getBuilderInfo({
@@ -58,9 +48,7 @@ describe('getBuilderInfo', () => {
       })
     ).toEqual({
       name: 'webpack5',
-      options: {
+      options: { lazyCompilation: true },
-        lazyCompilation: true,
-      },
     });
   });
 
@@ -69,27 +57,20 @@ describe('getBuilderInfo', () => {
       getBuilderInfo({
         core: { builder: '@storybook/builder-vite' },
       })
-    ).toEqual({
+    ).toEqual({ name: 'vite', options: {} });
-      name: 'vite',
-      options: {},
-    });
 
     expect(
       getBuilderInfo({
         core: {
           builder: {
             name: '@storybook/builder-vite',
-            options: {
+            options: { foo: 'bar' },
-              foo: 'bar',
-            },
           },
         },
       })
     ).toEqual({
       name: 'vite',
-      options: {
+      options: { foo: 'bar' },
-        foo: 'bar',
-      },
     });
   });
 
@@ -98,27 +79,40 @@ describe('getBuilderInfo', () => {
       getBuilderInfo({
         framework: '@storybook/react-vite',
       })
-    ).toEqual({
+    ).toEqual({ name: 'vite', options: {} });
-      name: 'vite',
-      options: {},
-    });
 
     expect(
       getBuilderInfo({
         framework: {
           name: '@storybook/react-vite',
-          options: {
+          options: { builder: { foo: 'bar' } },
-            builder: {
-              foo: 'bar',
-            },
-          },
         },
       })
     ).toEqual({
       name: 'vite',
+      options: { foo: 'bar' },
+    });
+  });
+});
+
+describe('getNextjsAddonOptions', () => {
+  it('should find storybook-addon-next and extract its options', () => {
+    expect(getNextjsAddonOptions(['foo', 'bar'])).toEqual({});
+    expect(getNextjsAddonOptions(['foo', 'storybook-addon-next'])).toEqual({});
+    expect(getNextjsAddonOptions(['foo', { name: 'storybook-addon-next' }])).toEqual({});
+    expect(getNextjsAddonOptions(['foo', { name: 'storybook-addon-next', options: {} }])).toEqual(
+      {}
+    );
+    expect(
+      getNextjsAddonOptions([
+        'foo',
+        {
+          name: 'storybook-addon-next',
           options: {
-        foo: 'bar',
+            nextConfigPath: 'foo/bar',
           },
-    });
+        },
+      ])
+    ).toEqual({ nextConfigPath: 'foo/bar' });
   });
 });