[dataformats] Remove dependency check and add push failure test

The check is done prior to the dependency list enumeration.

beat/cmdline/dataformats.py

@@ -39,8 +39,6 @@ import click
 
 from beat.core import dataformat
 
-from beat.backend.python.dataformat import DataFormat
-
 from . import common
 from . import commands
 
@@ -135,9 +133,7 @@ def pull_impl(webapi, prefix, names, force, indentation, cache):
 
 def get_dependencies(ctx, asset_name):
     prefix = ctx.meta["config"].path
-    df = DataFormat(prefix, asset_name)
-    if not df.valid:
-        raise RuntimeError("Invalid dataformat: {}".format(df.errors))
+    df = dataformat.DataFormat(prefix, asset_name)
 
     dependencies = {}
 

beat/cmdline/test/test_dataformats.py

@@ -54,6 +54,7 @@ class TestOnline(core.OnlineAssetTestCase):
         "fork": "user/forked_obj/1",
         "push": "user/composed/1",
         "not_owner_push": "system/chart/1",
+        "push_invalid": "errors/description_too_long/1",
     }
 
     def _modify_asset(self, asset_name):