Andal.LND/edx-platform
4
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Merge pull request #21292 from cpennington/correct-laboratory-tracebacks

Browse Source 
Remove publishing for access control experimnent for now
This commit is contained in:
Calen Pennington
2019-08-05 15:24:33 -04:00
committed by GitHub
parent b57c1cdd0d f319d20d6d
commit b77793da5f
1 changed files with 5 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
lms/djangoapps/courseware/rules.py
View File

@@ -4,6 +4,7 @@ django-rules and Bridgekeeper rules for courseware related features
from __future__ import absolute_import from __future__ import absolute_import
import logging import logging
import traceback
import laboratory import laboratory
import rules import rules
@@ -63,15 +64,16 @@ class HasAccessRule(Rule):
class StaffAccessExperiment(laboratory.Experiment): class StaffAccessExperiment(laboratory.Experiment):
def compare(self, control, candidate): def compare(self, control, candidate):
return bool(control) == candidate return bool(control.value) == candidate.value
def publish(self, result): def publish(self, result):
if not result.match: if not result.match:
LOG.warning( LOG.warning(
u"StaffAccessExperiment: control=%r, candidate=%r", u"StaffAccessExperiment: control=%r, candidate=%r\n%s",
result.control, result.control,
result.candidates[0], result.candidates[0],
exc_info=True "".join(traceback.format_stack(limit=10))
) )