feat(hydrate): add serializeShadowroot to hydrateDocument

[Armand-Lluka]

What is the current behavior?

GitHub Issue Number: #6240

What is the new behavior?

hydrateDocument has been extended to provide the serializeShadowRoot opt to be inline with what is provided with renderToString

Documentation

Does this introduce a breaking change?

• Yes
• No

Testing

Test was added to test.e2e.ts to verify the new option

Other information

[christian-bromann]

🙌 Awesome, thanks!

[christian-bromann]

@Armand-Lluka it seems like the e2e tests are failing:

Summary of all failing tests
FAIL src/declarative-shadow-dom/test.e2e.ts
  ● renderToString › hydrateDocument › can hydrate components with open shadow dom by default

    expect(received).toContain(expected) // indexOf

    Matcher error: received value must not be null nor undefined

    Received has value: null

      386 |         fullDocument: false,
      387 |       });
    > 388 |       expect(html).toContain('s-id="');
          |                    ^
      389 |       expect(html).toContain('shadowrootmode="open"');
      390 |     });
      391 |

      at Object.<anonymous> (src/declarative-shadow-dom/test.e2e.ts:388:20)

  ● renderToString › hydrateDocument › can hydrate components with scoped shadow dom

    expect(received).toContain(expected) // indexOf

    Matcher error: received value must not be null nor undefined

    Received has value: null

      395 |         fullDocument: false,
      396 |       });
    > 397 |       expect(html).toContain('s-id="');
          |                    ^
      398 |       expect(html).not.toContain('shadowrootmode="open"');
      399 |     });
      400 |   });

      at Object.<anonymous> (src/declarative-shadow-dom/test.e2e.[26:38.6]  screenshot, tests finished, passed: false in 16.88 s MEM: 2634.9MB
[26:38.6]  screenshot, completeTimespan started ... MEM: 2634.9MB
ts:397:20)

[Armand-Lluka]

Looks like I was running the wrong test suite 🙈

I've updated the type in stencil-public-compiler.ts, let me know if anything else needs updating.

@christian-bromann

[christian-bromann]

Last request: can we remove serializeShadowRoot from SerializeDocumentOptions since it extends HydrateDocumentOptions where it is now being defined.

[christian-bromann]

LGTM 👍