Merge lp:~camptocamp/oerpscenario/add-module-steps into lp:oerpscenario

Proposed by Nicolas Bessi - Camptocamp
Status: Needs review
Proposed branch: lp:~camptocamp/oerpscenario/add-module-steps
Merge into: lp:oerpscenario
Diff against target: 23 lines (+16/-0)
1 file modified
features/steps/module_config.py (+16/-0)
To merge this branch: bzr merge lp:~camptocamp/oerpscenario/add-module-steps
Reviewer Review Type Date Requested Status
Yannick Vaucher @ Camptocamp Approve
Alexandre Fayolle - camptocamp code review, no test Needs Fixing
Review via email: mp+160056@code.launchpad.net

Description of the change

Add new module helper to update or uninstall addons

To post a comment you must log in.
Revision history for this message
Alexandre Fayolle - camptocamp (alexandre-fayolle-c2c) wrote :

The my module must be "{state}" implementation does not work after "I update the {module} module".

review: Needs Fixing (code review, no test)
322. By openerp <openerp@openerp-vagrant>

[FIX] updated module state can be tested

Revision history for this message
Nicolas Bessi - Camptocamp (nbessi-c2c-deactivatedaccount) wrote :

Fix committed

Revision history for this message
Nicolas Bessi - Camptocamp (nbessi-c2c-deactivatedaccount) wrote :

Alexandre if you can have a look at it

Revision history for this message
Yannick Vaucher @ Camptocamp (yvaucher-c2c) wrote :
Revision history for this message
Yannick Vaucher @ Camptocamp (yvaucher-c2c) :
review: Approve

Unmerged revisions

322. By openerp <openerp@openerp-vagrant>

[FIX] updated module state can be tested

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'features/steps/module_config.py'
--- features/steps/module_config.py 2013-02-08 09:11:32 +0000
+++ features/steps/module_config.py 2013-05-22 09:30:38 +0000
@@ -62,3 +62,19 @@
62def impl(ctx):62def impl(ctx):
63 assert ctx.found_item63 assert ctx.found_item
64 ctx.found_item.execute()64 ctx.found_item.execute()
65
66@given(u'I update the "{oe_module}" module')
67def impl(ctx, oe_module):
68 ctx.client.upgrade(oe_module)
69 ctx.altered_module = oe_module
70
71@given(u'I uninstall module "{oe_module}"')
72def impl(ctx, oe_module):
73 ctx.client.uninstall(oe_module)
74 ctx.altered_module = oe_module
75
76@then(u'my module must be "{state}"')
77def impl(ctx, state):
78 assert ctx.altered_module
79 mod = ctx.altered_module
80 assert mod in ctx.client.module(mod)[state]

Subscribers

People subscribed via source and target branches

to all changes: