pub/python-betterproto
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Cleanup

Browse Source
This commit is contained in:
boukeversteegh
2020-05-22 21:16:27 +02:00
parent c3e3837f71
commit 92088ebda8
1 changed files with 3 additions and 8 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
betterproto/tests/test_inputs.py
View File

@@ -3,6 +3,8 @@ import json
import os import os
import sys import sys
import pytest import pytest
import betterproto
from betterproto.tests.util import get_directories, inputs_path
# Force pure-python implementation instead of C++, otherwise imports # Force pure-python implementation instead of C++, otherwise imports
# break things because we can't properly reset the symbol database. # break things because we can't properly reset the symbol database.
@@ -12,8 +14,6 @@ from google.protobuf import symbol_database
from google.protobuf.descriptor_pool import DescriptorPool from google.protobuf.descriptor_pool import DescriptorPool
from google.protobuf.json_format import Parse from google.protobuf.json_format import Parse
import betterproto
from betterproto.tests.util import get_directories, inputs_path
excluded_test_cases = {'googletypes_response', 'service'} excluded_test_cases = {'googletypes_response', 'service'}
test_case_names = {*get_directories(inputs_path)} - excluded_test_cases test_case_names = {*get_directories(inputs_path)} - excluded_test_cases
@@ -50,13 +50,8 @@ def test_message_json(test_case_name: str) -> None:
message.from_json(reference_json_data) message.from_json(reference_json_data)
message_json = message.to_json(0) message_json = message.to_json(0)
print(reference_json_data)
print(message_json)
assert json.loads(reference_json_data) == json.loads(message_json) assert json.loads(reference_json_data) == json.loads(message_json)
# todo: handle -negative
@pytest.mark.parametrize("test_case_name", test_case_names) @pytest.mark.parametrize("test_case_name", test_case_names)
def test_binary_compatibility(test_case_name: str) -> None: def test_binary_compatibility(test_case_name: str) -> None:
@@ -67,7 +62,6 @@ def test_binary_compatibility(test_case_name: str) -> None:
reference_module_root = os.path.join(*reference_output_package.split('.'), test_case_name) reference_module_root = os.path.join(*reference_output_package.split('.'), test_case_name)
print(f'Appending {reference_module_root}')
sys.path.append(reference_module_root) sys.path.append(reference_module_root)
# import reference message # import reference message
@@ -95,6 +89,7 @@ def test_binary_compatibility(test_case_name: str) -> None:
helper methods helper methods
''' '''
def get_test_case_json_data(test_case_name): def get_test_case_json_data(test_case_name):
test_data_path = os.path.join(inputs_path, test_case_name, f'{test_case_name}.json') test_data_path = os.path.join(inputs_path, test_case_name, f'{test_case_name}.json')
if not os.path.exists(test_data_path): if not os.path.exists(test_data_path):