Implemented the use of the SkyDynamicComponentService

[Blackbaud-TrevorBurch]

No description provided.

[codecov-io]

Codecov Report

Merging #9 into master will increase coverage by 0.28%.
The diff coverage is 100%.

@@           Coverage Diff            @@
##           master     #9      +/-   ##
========================================
+ Coverage   99.71%   100%   +0.28%     
========================================
  Files          10     10              
  Lines         350    338      -12     
  Branches       53     52       -1     
========================================
- Hits          349    338      -11     
+ Misses          1      0       -1

Impacted Files	Coverage Δ
...pp/public/modules/flyout/flyout-adapter.service.ts	100% <ø> (ø)	⬆️
src/app/public/modules/flyout/flyout.module.ts	100% <ø> (ø)	⬆️
src/app/public/modules/flyout/flyout.service.ts	100% <100%> (+1.53%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update fa8a51c...6e6cdbb. Read the comment docs.

[Blackbaud-AlexKingman]

@Blackbaud-TrevorBurch what about these items in flyout.component.ts?

    private adapter: SkyFlyoutAdapterService,
    private changeDetector: ChangeDetectorRef,
    private injector: Injector,
    private resolver: ComponentFactoryResolver

[Blackbaud-TrevorBurch]

@Blackbaud-AlexKingman the dynamic service only adds things to the top and bottom of the body. Those imports are used for rendering the consumer's passed in components in the flyout host which is not supported by the dynamic service. If we wanted to have that functionality we would need another enhancement to the dynamic service.

[Blackbaud-SteveBrush]

Your changes to the service unit test work well, but we definitely need to come back later on and mock everything out a little better.
https://angular.io/guide/testing#service-tests