Currently the test case for LayerTreeResourceProvider and DisplayResourceProvider are in ResourceProviderTest.
Some DisplayResourceProvider test case are moved from ResourceProvider, but didn't check if we should provide corresponding LayerTreeResourceProvider. The same rules applies to LayerTreeResourceProvider. Such a way will lead to miss test coverage on DisplayResourceProvider or LayerTreeResourceProvider.
For aforementioned DisplayResourceProvider test case (or LayerTreeResourceProvider test case), we should check carefully if we should provide corresponding LayerTreeResourceProvider test case (or DisplayResourceProvider test case).
My idea is, provide three unittest files:
ResourceProviderTest: tests for resource shared between LayerTreeResourceProvider and DisplayResourceProvider;
LayerTreeResourceProviderTest: tests for LayerTreeResourceProvider;
DisplayResourceProviderTest: tests for DisplayResourceProviderTest;
Currently the test case for LayerTreeResourceProvider and DisplayResourceProvider are in ResourceProviderTest.
Some DisplayResourceProvider test case are moved from ResourceProvider, but didn't check if we should provide corresponding LayerTreeResourceProvider
test case. The same rules applies to LayerTreeResourceProvider. Such a way
will lead to miss test coverage on DisplayResourceProvider or LayerTreeResourceProvider.
For aforementioned DisplayResourceProvider test case (or LayerTreeResourceProvider test case), we should check carefully if we should
provide corresponding LayerTreeResourceProvider test case (or
DisplayResourceProvider test case).
My idea is, provide three unittest files:
ResourceProviderTest: tests for resource shared between
LayerTreeResourceProvider and DisplayResourceProvider;
LayerTreeResourceProviderTest: tests for LayerTreeResourceProvider;
DisplayResourceProviderTest: tests for DisplayResourceProviderTest;
Currently the test case for LayerTreeResourceProvider and DisplayResourceProvider are in ResourceProviderTest.
Some DisplayResourceProvider test case are moved from ResourceProvider, but didn't check if we should provide corresponding LayerTreeResourceProvider test case. The same rules applies to LayerTreeResourceProvider. Such a way will lead to miss test coverage on DisplayResourceProvider or LayerTreeResourceProvider.
For aforementioned DisplayResourceProvider test case (or LayerTreeResourceProvider test case), we should check carefully if we should provide corresponding LayerTreeResourceProvider test case (or
DisplayResourceProvider test case).
My idea is, provide three unittest files: ResourceProviderTest: tests for resource shared between LayerTreeResourceProvider and DisplayResourceProvider; LayerTreeResourceProviderTest: tests for LayerTreeResourceProvider; DisplayResourceProviderTest: tests for DisplayResourceProviderTest;
Currently the test case for LayerTreeResourceProvider and DisplayResourceProvider are in ResourceProviderTest.
Some DisplayResourceProvider test case are moved from ResourceProvider, but didn't check if we should provide corresponding LayerTreeResourceProvider test case. The same rules applies to LayerTreeResourceProvider. Such a way will lead to miss test coverage on DisplayResourceProvider or LayerTreeResourceProvider.
For aforementioned DisplayResourceProvider test case (or LayerTreeResourceProvider test case), we should check carefully if we should provide corresponding LayerTreeResourceProvider test case (or DisplayResourceProvider test case).
My idea is, provide three unittest files: ResourceProviderTest: tests for resource shared between LayerTreeResourceProvider and DisplayResourceProvider; LayerTreeResourceProviderTest: tests for LayerTreeResourceProvider; DisplayResourceProviderTest: tests for DisplayResourceProviderTest;
Comment 1 by xing...@intel.com
, Sep 1 2017