Passed in: 5
" ); @@ -199,6 +250,29 @@ component extends="coldbox.system.testing.BaseTestCase" { expect( result ).toInclude( "UUID2: #firstUUID#
" ); } ); + it( "should correctly mount with _lazyMount() using a valid snapshot", function() { + // Create a mock snapshot + var mountParams = { "foo": "bar", "baz": 123 }; + var snapshotStruct = { + "data": { + "forMount": [ mountParams ] + } + }; + var encodedSnapshot = toBase64( serializeJson( snapshotStruct ) ); + // Setup test component and event + testComponent._withEvent( getRequestContext() ); + // Spy on onMount to capture params + var calledParams = {}; + + // Call _lazyMount + testComponent._lazyMount( encodedSnapshot ); + // If we got here that means there's no error NOT calling onMount + testComponent.$( "onMount" ); + + testComponent._lazyMount( encodedSnapshot ); + + } ); + it( "should accept false flag for computed properties to prevent caching", function() { var result = CBWIREController.wire( "test.should_accept_false_flag_for_computed_properties_to_prevent_caching" ); var firstUUID = reFindNoCase( "UUID: ([A-Za-z0-9-]+)", result, 1, true ).match[ 2 ]; @@ -224,9 +298,6 @@ component extends="coldbox.system.testing.BaseTestCase" { xit( "should support deep nesting with correct count of children", function() { var result = CBWIREController.wire( "test.should_support_deep_nesting" ); var parent = parseRendering( result, 1 ); - writeDump( result ); - writeDump( parent ); - abort; var child1 = parseRendering( result, 2 ); var child2 = parseRendering( result, 3 ); expect( parent.snapshot.memo.children.count() ).toBe( 2 ); @@ -236,17 +307,17 @@ component extends="coldbox.system.testing.BaseTestCase" { it( "shouldn't isolate by default", function() { var result = CBWIREController.wire( "test.shouldnt_isolate_by_default" ); - expect( result ).toInclude( ""isolate":false" ); + expect( result ).toInclude( ""isolate"&##x3a;false" ); } ); it( "should isolate when using isolate=true", function() { var result = CBWIREController.wire( "test.should_isolate_when_using_isolate_true" ); - expect( result ).toInclude( ""isolate":true" ); + expect( result ).toInclude( ""isolate"&##x3a;true" ); } ); it( "should isolate when using lazyLoad=true", function() { var result = CBWIREController.wire( "test.should_isolate_when_using_lazyLoad_true" ); - expect( result ).toInclude( ""isolate":true" ); + expect( result ).toInclude( ""isolate"&##x3a;true" ); } ); it( "should support hasErrors(), hasError( prop ), and getError( prop ) for validation", function() { @@ -453,7 +524,7 @@ component extends="coldbox.system.testing.BaseTestCase" { it( "should include listeners within wire:effects on initial render", function() { var result = testComponent._render( testComponent.template( "wires.TestComponent" ) ); - expect( result ).toInclude( "wire:effects=""{"listeners":["someEvent"]}""" ); + expect( result ).toInclude( "wire:effects=""&##x7b;"listeners"&##x3a;&##x5b;"someEvent"&##x5d;&##x7d;""" ); } ); it( "should support single file components", function() { @@ -470,6 +541,18 @@ component extends="coldbox.system.testing.BaseTestCase" { } } ); + it( "should correctly encode snapshot data containing HTML with quotes", function() { + local.htmlWithQuotes = 'Some text with "quotes" inside & special chars >.
'; + local.componentName = "test.html_with_quotes"; // Use the new component + + var renderedHtml = CBWIREController.wire( + name = local.componentName, + params = { content = local.htmlWithQuotes } + ); + + expect( renderedHtml ).toInclude( 'Some&##x20;text&##x20;with&##x20;&##x5c;"quotes' ); + }); + }); describe("Incoming Requests", function() { @@ -507,7 +590,7 @@ component extends="coldbox.system.testing.BaseTestCase" { it( "should trim string values if enabled on component", () => { var settings = getInstance( "coldbox:modulesettings:cbwire" ); - settings.trimStringValues = false; + settings.trimStringValues = true; var payload = incomingRequest( memo = { "name": "test.should_trim_string_values_if_enabled_on_component", @@ -1264,6 +1347,42 @@ component extends="coldbox.system.testing.BaseTestCase" { expect( reFindNoCase( "/cbwire/upload\?expires=[0-9]+&signature=[A-Za-z0-9]+$", response.components[1].effects.dispatches[1].params.url ) ).toBeGT( 0 ); } ); + it( "should _startUpload() and honor custom updateEndpoint in upload URL", function() { + var settings = getInstance( "coldbox:modulesettings:cbwire" ); + // Set custom updateEndpoint + settings.updateEndpoint = "/index.bxm/cbwire/update"; + + var payload = incomingRequest( + memo = { + "name": "TestComponent", + "id": "Z1Ruz1tGMPXSfw7osBW2", + "children": [] + }, + data = {}, + calls = [ + { + "path": "", + "method": "_startUpload", + "params": [ + "testFile", + [ "name": "image.png", "size": 118672, "type": "image/png" ], + false + ] + } + ], + updates = {} + ); + var response = cbwireController.handleRequest( payload, event ); + expect( response.components[1].effects.dispatches[1].name ).toBe( "upload:generatedSignedUrl" ); + expect( response.components[1].effects.dispatches[1].self ).toBeTrue(); + expect( response.components[1].effects.dispatches[1].params.name ).toBe( "testFile" ); + expect( response.components[1].effects.dispatches[1].params.url ).toInclude( "/index.bxm/cbwire/upload" ); + expect( reFindNoCase( "/index\.bxm/cbwire/upload\?expires=[0-9]+&signature=[A-Za-z0-9]+$", response.components[1].effects.dispatches[1].params.url ) ).toBeGT( 0 ); + + // Clean up - restore default + settings.delete( "updateEndpoint" ); + } ); + xit( "should _finishUpload()", function() { var payload = incomingRequest( memo = { @@ -1306,7 +1425,7 @@ component extends="coldbox.system.testing.BaseTestCase" { key="", lazy=true ); - expect( lazyHtml ).toInclude( ""isolate":true" ); + expect( lazyHtml ).toInclude( ""isolate"&##x3a;true" ); } ); it( "should lazy load a component using the original outer element", function() { @@ -1372,6 +1491,77 @@ component extends="coldbox.system.testing.BaseTestCase" { } ); }); + describe("CBWIREController getComponentDSL", function() { + + beforeEach(function(currentSpec) { + setup(); + cbwireController = getInstance("CBWIREController@cbwire"); + prepareMock(cbwireController); + }); + + it("should return component name with wires prefix when name doesn't contain wires", function() { + var result = cbwireController.getComponentDSL("TestComponent"); + expect(result).toBe("wires.TestComponent"); + }); + + it("should use wiresLocation setting when provided and name doesn't contain wires", function() { + var settings = getInstance("coldbox:modulesettings:cbwire"); + settings.wiresLocation = "customWires"; + var result = cbwireController.getComponentDSL("TestComponent"); + expect(result).toBe("customWires.TestComponent"); + settings.wiresLocation = ""; + }); + + it("should fallback to wires prefix when wiresLocation is not set", function() { + var settings = getInstance("coldbox:modulesettings:cbwire"); + if (settings.keyExists("wiresLocation")) { + structDelete(settings, "wiresLocation"); + } + var result = cbwireController.getComponentDSL("TestComponent"); + expect(result).toBe("wires.TestComponent"); + }); + + it("should return component name as-is when it already contains wires", function() { + var result = cbwireController.getComponentDSL("wires.TestComponent"); + expect(result).toBe("wires.TestComponent"); + }); + + it("should handle module reference with @ symbol", function() { + //cbwireController.$("getModuleComponentPath", "modules.testModule.wires.TestComponent"); + var result = cbwireController.getComponentDSL("TestComponent@testingmodule"); + expect(result).toBe("modules_app.testingmodule.wires.TestComponent"); + }); + + it("should throw ModuleNotFound exception when module reference has invalid format", function() { + expect(function() { + cbwireController.getComponentDSL("TestComponent@module@extra"); + }).toThrow(type="ModuleNotFound"); + }); + + it("should throw ModuleNotFound exception when module reference has only one part", function() { + expect(function() { + cbwireController.getComponentDSL("TestComponent@"); + }).toThrow(type="ModuleNotFound"); + }); + + it("should throw ModuleNotFound exception when module reference has just @ symbol", function() { + expect(function() { + cbwireController.getComponentDSL("@"); + }).toThrow(type="ModuleNotFound"); + }); + + it("should handle nested component paths with wires prefix", function() { + var result = cbwireController.getComponentDSL("nested.TestComponent"); + expect(result).toBe("wires.nested.TestComponent"); + }); + + it("should handle component with existing wires in middle of path", function() { + var result = cbwireController.getComponentDSL("some.wires.path.TestComponent"); + expect(result).toBe("some.wires.path.TestComponent"); + }); + + }); + describe("CBWIREController", function() { beforeEach(function(currentSpec) { @@ -1380,6 +1570,7 @@ component extends="coldbox.system.testing.BaseTestCase" { setup(); cbwireController = getInstance("CBWIREController@cbwire"); prepareMock( cbwireController ); + }); it( "should return an object", function() { @@ -1457,15 +1648,21 @@ component extends="coldbox.system.testing.BaseTestCase" { } ).toThrow( type="ModuleNotFound" ); } ); - it( "can render component from nested module using default wires location", function() { + it( "can render component from module using default wires location", function() { var result = cbwireController.wire( "NestedModuleDefaultComponent@testingmodule" ); expect( result ).toContain( "Nested module component using default wires location" ); } ); + it( "can render component from module using nested folder", function() { + var result = cbwireController.wire( "wires.nestedComponent.NestedFolderComponent@testingmodule" ); + expect( result ).toContain( "Nested folder component" ); + } ); + it( "can load components from an external modules folder", function() { var result = cbwireController.wire( "should_load_external_modules@ExternalModule" ); expect( result ).toInclude( "External Module Loaded" ); } ); + }); describe( "Preprocessors", function() { @@ -1612,7 +1809,8 @@ component extends="coldbox.system.testing.BaseTestCase" { "calls": arguments.calls, "snapshot": { "data": arguments.data, - "memo": arguments.memo + "memo": arguments.memo, + "checksum": "" }, "updates": arguments.updates } @@ -1621,7 +1819,7 @@ component extends="coldbox.system.testing.BaseTestCase" { }; response.content.components = response.content.components.map( function( _comp ) { - _comp.snapshot = serializeJson( _comp.snapshot ); + _comp.snapshot = getInstance( "ChecksumService@cbwire" ).calculateChecksum( _comp.snapshot ); return _comp; } ); @@ -1649,29 +1847,96 @@ component extends="coldbox.system.testing.BaseTestCase" { } /** - * Parse the snapshot from a rendered HTML component + * Parse the snapshot from a rendered HTML component by decoding + * ALL HTML entities before attempting JSON deserialization. * - * @return struct + * @html string | The rendered HTML containing the component. + * @index numeric | The index of the component if multiple match (usually 1). + * + * @return struct The deserialized snapshot struct. + * + * @throws Error if parsing or deserialization fails. */ - private function parseSnapshot( html, index = 1 ) { - local.match = reMatchNoCase( "wire:snapshot=""([^""]+)", html )[ index ]; - local.regexMatches = reFindNoCase( "wire:snapshot=""([^""]+)", local.match, 1, true ); - local.snapshot = local.regexMatches.match[ 2 ]; - local.snapshot = replaceNoCase( local.snapshot, """, """", "all" ); - return deserializeJSON( local.snapshot ); + private function parseSnapshot( required string html, numeric index = 1 ) { + // Use single quotes for regex literals to avoid excessive escaping + local.match = reMatchNoCase( 'wire:snapshot="([^"]+)"', arguments.html ); + if ( !arrayLen( local.match ) >= arguments.index ) { + throw( message="Snapshot attribute not found at index #arguments.index# in provided HTML.", detail=arguments.html ); + } + local.snapshotAttributeMatch = local.match[ arguments.index ]; + + // Extract the encoded value + local.regexMatches = reFindNoCase( 'wire:snapshot="([^"]+)"', local.snapshotAttributeMatch, 1, true ); + if ( !arrayLen( local.regexMatches.match ) == 2 ) { + throw( message="Could not extract snapshot value using regex from attribute match.", detail=local.snapshotAttributeMatch ); + } + local.snapshotEncoded = local.regexMatches.match[ 2 ]; + + // Decode ALL HTML entities (handles ", ", ", <, &, etc.) + local.snapshotDecoded = canonicalize( local.snapshotEncoded, true, true ); // Key change! + + try { + return deserializeJSON( local.snapshotDecoded ); + } catch ( any e ) { + // Provide more context on failure for easier debugging + var errorMsg = "Failed to deserialize snapshot JSON after decoding HTML entities."; + errorMsg &= " Decoded JSON string was: [#encodeForHtml(local.snapshotDecoded)#]."; // Encode for safe display + errorMsg &= " Original Error: #e.message# #e.detail#"; + throw( message=errorMsg, detail=local.snapshotDecoded ); + } } /** - * Parse the effects from a rendered HTML component + * Parse the effects from a rendered HTML component by decoding + * ALL HTML entities before attempting JSON deserialization. * - * @return struct + * @html The rendered HTML containing the component. + * @index The index of the component if multiple match (usually 1). + * + * @return any The deserialized effects (usually struct or array). + * + * @throws Error if parsing or deserialization fails. + */ + private function parseEffects( required string html, numeric index = 1 ) { + // Use single quotes for regex literals + local.match = reMatchNoCase( 'wire:effects="([^"]+)"', arguments.html ); + if ( !arrayLen( local.match ) >= arguments.index ) { + throw( message="Effects attribute not found at index #arguments.index# in provided HTML.", detail=arguments.html ); + } + local.effectsAttributeMatch = local.match[ arguments.index ]; + + // Extract the encoded value + local.regexMatches = reFindNoCase( 'wire:effects="([^"]+)"', local.effectsAttributeMatch, 1, true ); + if ( !arrayLen( local.regexMatches.match ) == 2 ) { + throw( message="Could not extract effects value using regex from attribute match.", detail=local.effectsAttributeMatch ); + } + local.effectsEncoded = local.regexMatches.match[ 2 ]; + + // Decode ALL HTML entities + local.effectsDecoded = canonicalize( local.effectsEncoded, true, true ); // Key change! + + try { + if ( isJSON( local.effectsDecoded ) ) { + return deserializeJSON( local.effectsDecoded ); + } else { + return {}; + } + } catch ( any e ) { + // Provide more context on failure + var errorMsg = "Failed to deserialize effects JSON after decoding HTML entities."; + errorMsg &= " Decoded JSON string was: [#encodeForHtml(local.effectsDecoded)#]."; // Encode for safe display + errorMsg &= " Original Error: #e.message# #e.detail#"; + throw( message=errorMsg, detail=local.effectsDecoded ); + } + } + + /** + * Check if the current environment is a BoxLang environment + * + * @return boolean */ - private function parseEffects( html, index = 1 ) { - local.match = reMatchNoCase( "wire:effects=""([^""]+)", html )[ index ]; - local.regexMatches = reFindNoCase( "wire:effects=""([^""]+)", local.match, 1, true ); - local.effects = local.regexMatches.match[ 2 ]; - local.effects = replaceNoCase( local.effects, """, """", "all" ); - return deserializeJSON( local.effects ); + private function isBoxLang() { + return server.keyExists( "boxlang" ); } }