Merge lp:~camptocamp/oerpscenario/trunk-add_load-afe into lp:oerpscenario
Proposed by
Alexandre Fayolle - camptocamp
Status: | Needs review |
---|---|
Proposed branch: | lp:~camptocamp/oerpscenario/trunk-add_load-afe |
Merge into: | lp:oerpscenario |
Diff against target: |
111 lines (+107/-0) 1 file modified
features/steps/load.py (+107/-0) |
To merge this branch: | bzr merge lp:~camptocamp/oerpscenario/trunk-add_load-afe |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Leonardo Pistone | code review | Needs Fixing | |
Review via email: mp+206372@code.launchpad.net |
Description of the change
add @given('I load the data file "{filename}.csv" into the model "{model_name}"')
and @given('I load the data file "{filename}.yml"')
the yaml version can process images, which is useful for products.
To post a comment you must log in.
Unmerged revisions
- 330. By Alexandre Fayolle - camptocamp
-
[ADD] phrases to load csv or yaml files. Yaml files support references to images
Thanks!
In fact, we already have a step to import csv files in tools.py, but your one is better, because you check the messages returned by load().
This would have saved me a headache, because load() does not raise on error, so it is important to check the return value.
I suggest you remove the old step in this branch.