Add unit tests for `Container`

[huong-li-nguyen]

Description

• Added unit tests for instantiation and build of Container
• Added unit tests for all helper functions in _models_utils
• Refactored shared tests between Container and Page

Screenshot

Notice

• [x] I acknowledge and agree that, by checking this box and clicking "Submit Pull Request":

  • I submit this contribution under the Apache 2.0 license and represent that I am entitled to do so on behalf of myself, my employer, or relevant third parties, as applicable.
  • I certify that (a) this contribution is my original creation and / or (b) to the extent it is not my original creation, I am authorized to submit this contribution on behalf of the original creator(s) or their licensees.
  • I certify that the use of this contribution as authorized by the Apache 2.0 license does not violate the intellectual property rights of anyone else.
  • I have not referenced individuals, products or companies in any commits, directly or indirectly.
  • I have not added data or restricted code in any commits, directly or indirectly.

[petar-qb]

petar-qb - do any tests have to be added regarding the actions code? Or is most of it covered by existing tests? Please also align with l0uden which ones we will cover in terms of unit tests and which ones l0uden should cover :) Thanks!

We'll probably need to add new unit tests for the newly added "search" helper methods, but let's wait for the final form of https://github.com/mckinsey/vizro/pull/254 and then I'll make these changes.

[huong-li-nguyen]

petar-qb - do any tests have to be added regarding the actions code? Or is most of it covered by existing tests? Please also align with l0uden which ones we will cover in terms of unit tests and which ones l0uden should cover :) Thanks!

We'll probably need to add new unit tests for the newly added "search" helper methods, but let's wait for the final form of #254 and then I'll make these changes.

Sounds good to me! 👍

I'll leave this PR open for you to wrap up then. In general, these PRs will be merged in this order to main:

1. https://github.com/mckinsey/vizro/pull/254
2. https://github.com/mckinsey/vizro/pull/262
3. https://github.com/mckinsey/vizro/pull/265

[petar-qb]

@huong-li-nguyen With a small change made in the https://github.com/mckinsey/vizro/pull/265/commits/ad3f55b47f5924724e1f95e8fc580cbc500399ab, containers are fully tested with actions.

Unit tests for the newly added search helper methods in ModelManager have not yet been implemented as ModelManager will be completely overwritten soon (see https://github.com/mckinsey/vizro/issues/111).