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

Fixes for the blockstore API client / XBlock runtime under python 3

Browse Source
This commit is contained in:
Braden MacDonald
2019-12-05 09:24:24 -08:00
parent 0d899a1392
commit 1ea3c032d3
7 changed files with 19 additions and 19 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
openedx/core/djangoapps/content_libraries/tests/test_content_libraries.py
View File

@@ -152,7 +152,7 @@ class ContentLibrariesTest(ContentLibrariesRestApiTest):
handler_url = self._get_block_handler_url(block_id, "xmodule_handler") + "problem_get"
problem_get_response = self.client.get(handler_url)
self.assertEqual(problem_get_response.status_code, 200)
self.assertIn("You have used 0 of 5 attempts", problem_get_response.content)
self.assertIn("You have used 0 of 5 attempts", problem_get_response.content.decode('utf-8'))
# Now delete the block:
self.assertEqual(self._get_library(lib_id)["has_unpublished_deletes"], False)

4
openedx/core/djangoapps/content_libraries/tests/test_runtime.py
View File

@@ -235,7 +235,7 @@ class ContentLibraryXBlockUserStateTest(ContentLibraryContentTestMixin, TestCase
submit_result = client.post(problem_check_url, data={problem_key: "choice_3"})
self.assertEqual(submit_result.status_code, 200)
submit_data = json.loads(submit_result.content)
submit_data = json.loads(submit_result.content.decode('utf-8'))
self.assertDictContainsSubset({
"current_score": 0,
"total_possible": 1,
@@ -251,7 +251,7 @@ class ContentLibraryXBlockUserStateTest(ContentLibraryContentTestMixin, TestCase
# And submit a correct answer:
submit_result = client.post(problem_check_url, data={problem_key: "choice_1"})
self.assertEqual(submit_result.status_code, 200)
submit_data = json.loads(submit_result.content)
submit_data = json.loads(submit_result.content.decode('utf-8'))
self.assertDictContainsSubset({
"current_score": 1,
"total_possible": 1,

8
openedx/core/djangoapps/content_libraries/views.py
View File

@@ -46,11 +46,11 @@ def convert_exceptions(fn):
log.exception("XBlock not found in content library")
raise NotFound
except api.LibraryBlockAlreadyExists as exc:
log.exception(exc.message)
raise ValidationError(exc.message)
log.exception(str(exc))
raise ValidationError(str(exc))
except api.InvalidNameError as exc:
log.exception(exc.message)
raise ValidationError(exc.message)
log.exception(str(exc))
raise ValidationError(str(exc))
return wrapped_fn

4
openedx/core/djangoapps/xblock/runtime/shims.py
View File

@@ -76,8 +76,8 @@ class RuntimeShim(object):
# class in opaque-keys that accepts either old CourseKeys or new
# LearningContextKeys.
hasher = hashlib.md5()
hasher.update(settings.SECRET_KEY)
hasher.update(six.text_type(self.user_id))
hasher.update(settings.SECRET_KEY.encode('utf-8'))
hasher.update(six.text_type(self.user_id).encode('utf-8'))
digest = hasher.hexdigest()
return digest

2
openedx/core/djangolib/blockstore_cache.py
View File

@@ -167,7 +167,7 @@ def get_bundle_draft_files_cached(bundle_uuid, draft_name):
cache_key = ('bundle_draft_files', )
result = bundle_cache.get(cache_key)
if result is None:
result = blockstore_api.get_bundle_files(bundle_uuid, use_draft=draft_name)
result = list(blockstore_api.get_bundle_files(bundle_uuid, use_draft=draft_name))
bundle_cache.set(cache_key, result)
return result

2
openedx/core/lib/blockstore_api/methods.py
View File

@@ -297,7 +297,7 @@ def get_bundle_files_dict(bundle_uuid, use_draft=None):
def get_bundle_files(bundle_uuid, use_draft=None):
"""
Get a flat list of all the files in the specified bundle or draft.
Get an iterator over all the files in the specified bundle or draft.
"""
return get_bundle_files_dict(bundle_uuid, use_draft).values()

16
openedx/core/lib/blockstore_api/tests/test_blockstore_api.py
View File

@@ -112,31 +112,31 @@ class BlockstoreApiClientTest(unittest.TestCase):
self.assertEqual(draft, draft3)
# Write a file into the bundle:
api.write_draft_file(draft.uuid, "test.txt", "initial version")
api.write_draft_file(draft.uuid, "test.txt", b"initial version")
# Now the file should be visible in the draft:
draft_contents = api.get_bundle_file_data(bundle.uuid, "test.txt", use_draft=draft.name)
self.assertEqual(draft_contents, "initial version")
self.assertEqual(draft_contents, b"initial version")
api.commit_draft(draft.uuid)
# Write a new version into the draft:
api.write_draft_file(draft.uuid, "test.txt", "modified version")
api.write_draft_file(draft.uuid, "test.txt", b"modified version")
published_contents = api.get_bundle_file_data(bundle.uuid, "test.txt")
self.assertEqual(published_contents, "initial version")
self.assertEqual(published_contents, b"initial version")
draft_contents2 = api.get_bundle_file_data(bundle.uuid, "test.txt", use_draft=draft.name)
self.assertEqual(draft_contents2, "modified version")
self.assertEqual(draft_contents2, b"modified version")
# Now delete the draft:
api.delete_draft(draft.uuid)
draft_contents3 = api.get_bundle_file_data(bundle.uuid, "test.txt", use_draft=draft.name)
# Confirm the file is now reset:
self.assertEqual(draft_contents3, "initial version")
self.assertEqual(draft_contents3, b"initial version")
# Finaly, test the get_bundle_file* methods:
file_info1 = api.get_bundle_file_metadata(bundle.uuid, "test.txt")
self.assertEqual(file_info1.path, "test.txt")
self.assertEqual(file_info1.size, len("initial version"))
self.assertEqual(file_info1.size, len(b"initial version"))
self.assertEqual(file_info1.hash_digest, "a45a5c6716276a66c4005534a51453ab16ea63c4")
self.assertEqual(api.get_bundle_files(bundle.uuid), [file_info1])
self.assertEqual(list(api.get_bundle_files(bundle.uuid)), [file_info1])
self.assertEqual(api.get_bundle_files_dict(bundle.uuid), {
"test.txt": file_info1,
})