morphologyfactory: move empty stratum check here

It is better to test whether a straum is empty here, since it will be
noticed earlier in the build, as soon as the morphologies are loaded,
rather than after they have all been parsed.

It is also conceptually nicer to put it here, since the morphologyfactory
was written to perform this kind of validation.

On a more practical note, the validation is moved here so that the test
for this error isn't masked by the test for no build dependencies.

To ensure tests still pass, we alter the stratum morphology used by
other unit tests to no longer be empty, and add an empty one to test.

1 files changed, 0 insertions, 9 deletions

diff --git a/morphlib/builder2.py b/morphlib/builder2.py
index daf50b56..e1eaaa86 100644
--- a/morphlib/builder2.py
+++ b/morphlib/builder2.py
@@ -469,8 +469,6 @@ class StratumBuilder(BuilderBase):
         with self.build_watch('overall-build'):
             constituents = [d for d in self.artifact.dependencies
                             if self.is_constituent(d)]
-            if len(constituents) == 0:
-                raise EmptyStratumError(self.artifact.name)
 
             # the only reason the StratumBuilder has to download chunks is to
             # check for overlap now that strata are lists of chunks
@@ -707,10 +705,3 @@ class Builder(object):  # pragma: no cover
         built_artifacts = o.build_and_cache()
         logging.debug('Builder.build: done')
         return built_artifacts
-
-
-class EmptyStratumError(cliapp.AppException):
-
-    def __init__(self, stratum_name): # pragma: no cover
-        cliapp.AppException.__init__(self,
-            "Stratum %s is empty (has no dependencies)" % stratum_name)