ckanext-validation icon indicating copy to clipboard operation
ckanext-validation copied to clipboard

Site-wide validation overrides

Open wardi opened this issue 2 years ago • 3 comments

We use this to implement a "UTF-8-only" policy, but the same could be used for standardizing other options across a site with a configuration option.

wardi avatar Jul 17 '23 12:07 wardi

failure is unrelated to this change.

The problem is our tests are creating a resource with ckan's factory, which creates a dataset, then passes the resource to a new dataset factory, but that resource already belongs to a dataset.

wardi avatar Jul 19 '23 03:07 wardi

It may be unrelated, but it still means this change isn't properly tested.

How easily can the tests be fixed?

ThrawnCA avatar Jul 19 '23 03:07 ThrawnCA

test resources should be created like this https://github.com/frictionlessdata/ckanext-validation/blob/1073c80dace453a404df3d5f1ac1ae86a88b5029/ckanext/validation/tests/test_logic.py#L235-L237 instead of like https://github.com/frictionlessdata/ckanext-validation/blob/1073c80dace453a404df3d5f1ac1ae86a88b5029/ckanext/validation/tests/test_logic.py#L349-L351

wardi avatar Jul 19 '23 12:07 wardi