datopian / datahub-qa

:package: Bugs, issues and suggestions for datahub.io
https://datahub.io/
32 stars 6 forks source link

[push] show proper error when `resources.path` is invalid #80

Closed AcckiyGerman closed 6 years ago

AcckiyGerman commented 6 years ago

As a user I want to see a readable error when I push the data, so I could fix my dataset. But now, when I push a dataset with invalid resources.path I've got

How to reproduce

Push datasets with following errors in the datapackage.json (use https://github.com/frictionlessdata/test-data/packages/ ...invalid packages):

Expected behavior

Related issues:

AcckiyGerman commented 6 years ago

FIXED: https://github.com/datahq/data-cli/issues/265#issuecomment-366716624 Fix works only on testing server ATM

AcckiyGerman commented 6 years ago

@zelima could you update the production specstore with your "return errors" fix https://github.com/datahq/specstore/commit/fde06cdbbc223b7f33dcae7d4787af439383ea33 and I will close the issue.

zelima commented 6 years ago

@AcckiyGerman updated

AcckiyGerman commented 6 years ago

TESTED & FIXED

invalid-backward-path$ data push
> Error! Unexpected error: Local path "../../files/csv/separators/comma.csv" is not safe