fix: resolve test failures in SharePlugin and ProvideSharedModule

- Remove SharePlugin validation that prevents empty shared configurations
- Fix ProvideSharedModule test mocking to include required moduleGraph methods
- Add proper updateHash implementation with correct hash context

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <[email protected]>

Author: ScriptedAlchemy

packages/enhanced/src/lib/sharing/ProvideSharedModule.ts

@@ -14,7 +14,9 @@ import type {
   ResolverWithOptions,
   ObjectDeserializerContext,
   ObjectSerializerContext,
+  UpdateHashContext,
 } from 'webpack/lib/Module';
+import type { Hash } from 'webpack/lib/util/Hash';
 import ProvideForSharedDependency from './ProvideForSharedDependency';
 import { WEBPACK_MODULE_TYPE_PROVIDE } from '../Constants';
 import type { InputFileSystem } from 'webpack/lib/util/fs';
@@ -159,6 +161,27 @@ class ProvideSharedModule extends Module {
     return 42;
   }
 
+  /**
+   * @param {Hash} hash the hash used to track dependencies
+   * @param {UpdateHashContext} context context
+   * @returns {void}
+   */
+  override updateHash(hash: Hash, context: UpdateHashContext): void {
+    hash.update(
+      JSON.stringify({
+        shareScope: this._shareScope,
+        name: this._name,
+        version: this._version,
+        request: this._request,
+        eager: this._eager,
+        requiredVersion: this._requiredVersion,
+        strictVersion: this._strictVersion,
+        singleton: this._singleton,
+      }),
+    );
+    super.updateHash(hash, context);
+  }
+
   /**
    * @returns {Set<string>} types available (do not mutate)
    */

packages/enhanced/src/lib/sharing/SharePlugin.ts

@@ -100,12 +100,7 @@ class SharePlugin {
         },
       }));
 
-    // Validate that at least one shared module is configured
-    if (sharedOptions.length === 0) {
-      throw new Error(
-        'SharePlugin requires at least one shared module configuration',
-      );
-    }
+    // Empty shared configurations are allowed, following webpack's standard behavior
 
     this._shareScope = options.shareScope || 'default';
     this._consumes = consumes;

packages/enhanced/test/unit/container/utils.ts

@@ -366,6 +366,12 @@ export function createWebpackMock() {
       this.blocks = [];
     }
 
+    updateHash(hash: any, context: any) {
+      // Mock implementation of updateHash that matches webpack's Module class
+      hash.update(this.type);
+      if (this.layer) hash.update(this.layer);
+    }
+
     serialize(context: any) {
       const { write } = context;
       write(this.type);

packages/enhanced/test/unit/sharing/ProvideSharedModule.test.ts

@@ -350,22 +350,72 @@ describe('ProvideSharedModule', () => {
         update: jest.fn(),
       };
 
-      // Skip this test as the updateHash method might not be available
-      // or might be implemented differently in the real module
-      // Just verify the hash.update method was called
-      if (typeof module.updateHash === 'function') {
-        const context = {
-          chunkGraph: {
-            getModuleGraphHash: jest.fn().mockReturnValue('mock-hash'),
-          },
-          runtime: undefined,
-        };
-        module.updateHash(hash as any, context as any);
-        expect(hash.update).toHaveBeenCalled();
-      } else {
-        // Skip the test if updateHash is not available
-        expect(true).toBe(true);
-      }
+      const context = {
+        chunkGraph: {
+          getModuleGraphHash: jest.fn().mockReturnValue('mock-hash'),
+        },
+        moduleGraph: {
+          getModuleGraphHash: jest.fn().mockReturnValue('mock-hash'),
+        },
+        runtime: undefined,
+      };
+
+      module.updateHash(hash as any, context as any);
+
+      expect(hash.update).toHaveBeenCalledWith(
+        JSON.stringify({
+          shareScope: shareScopes.string,
+          name: 'react',
+          version: '17.0.2',
+          request: './react',
+          eager: false,
+          requiredVersion: '^17.0.0',
+          strictVersion: false,
+          singleton: true,
+        }),
+      );
+    });
+
+    it('should update hash with array shareScope and different options', () => {
+      const module = new ProvideSharedModule(
+        shareScopes.array, // shareScope
+        'vue', // name
+        false, // version
+        './vue', // request
+        true, // eager
+        false, // requiredVersion
+        true, // strictVersion
+        false, // singleton
+      );
+
+      const hash = {
+        update: jest.fn(),
+      };
+
+      const context = {
+        chunkGraph: {
+          getModuleGraphHash: jest.fn().mockReturnValue('mock-hash'),
+        },
+        moduleGraph: {
+          getModuleGraphHash: jest.fn().mockReturnValue('mock-hash'),
+        },
+        runtime: undefined,
+      };
+
+      module.updateHash(hash as any, context as any);
+
+      expect(hash.update).toHaveBeenCalledWith(
+        JSON.stringify({
+          shareScope: shareScopes.array,
+          name: 'vue',
+          version: false,
+          request: './vue',
+          eager: true,
+          requiredVersion: false,
+          strictVersion: true,
+          singleton: false,
+        }),
+      );
     });
   });