Commit f3cc7bd5 by Calen Pennington

Mark some side-effecting actions so that they follow the --dry-run paver flag

parent d8defaa4
...@@ -77,6 +77,9 @@ def wait_for_server(server, port): ...@@ -77,6 +77,9 @@ def wait_for_server(server, port):
) )
) )
if tasks.environment.dry_run:
return True
attempts = 0 attempts = 0
server_ok = False server_ok = False
......
...@@ -7,7 +7,7 @@ from urllib import urlencode ...@@ -7,7 +7,7 @@ from urllib import urlencode
from common.test.acceptance.fixtures.course import CourseFixture, FixtureError from common.test.acceptance.fixtures.course import CourseFixture, FixtureError
from path import Path as path from path import Path as path
from paver.easy import sh, BuildFailure, cmdopts, task, needs, might_call, call_task from paver.easy import sh, BuildFailure, cmdopts, task, needs, might_call, call_task, dry
from pavelib.utils.test.suites.suite import TestSuite from pavelib.utils.test.suites.suite import TestSuite
from pavelib.utils.envs import Env from pavelib.utils.envs import Env
from pavelib.utils.test.bokchoy_utils import ( from pavelib.utils.test.bokchoy_utils import (
...@@ -233,7 +233,10 @@ class BokChoyTestSuite(TestSuite): ...@@ -233,7 +233,10 @@ class BokChoyTestSuite(TestSuite):
# Create course in order to seed forum data underneath. This is # Create course in order to seed forum data underneath. This is
# a workaround for a race condition. The first time a course is created; # a workaround for a race condition. The first time a course is created;
# role permissions are set up for forums. # role permissions are set up for forums.
CourseFixture('foobar_org', '1117', 'seed_forum', 'seed_foo').install() dry(
"Installing course fixture for forums",
CourseFixture('foobar_org', '1117', 'seed_forum', 'seed_foo').install
)
print 'Forums permissions/roles data has been seeded' print 'Forums permissions/roles data has been seeded'
except FixtureError: except FixtureError:
# this means it's already been done # this means it's already been done
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment