Commit 7180c64b authored by Juan Antonio Navarro Perez's avatar Juan Antonio Navarro Perez Committed by Commit Bot

[results processor] Stop url-decoding test paths

These are no longer url-encoded from Telemetry side.

Bug: 1022380
Change-Id: I8a34a9d3a5b58c3add7aec6fcb7c73bb48d92ef3
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/1943132
Commit-Queue: Juan Antonio Navarro Pérez <perezju@chromium.org>
Reviewed-by: default avatarMikhail Khokhlov <khokhlov@google.com>
Cr-Commit-Position: refs/heads/master@{#720170}
parent 2b0f1d99
...@@ -57,7 +57,7 @@ class Json3OutputTest(unittest.TestCase): ...@@ -57,7 +57,7 @@ class Json3OutputTest(unittest.TestCase):
# url-friendly names without special characters. # url-friendly names without special characters.
def testUrlAsStoryName(self): def testUrlAsStoryName(self):
results = self.Convert([ results = self.Convert([
testing.TestResult('benchmark/http%3A%2F%2Fexample.com') testing.TestResult('benchmark/http://example.com')
]) ])
test_result = self.FindTestResult( test_result = self.FindTestResult(
......
...@@ -7,7 +7,6 @@ import datetime ...@@ -7,7 +7,6 @@ import datetime
import logging import logging
import multiprocessing import multiprocessing
from multiprocessing.dummy import Pool as ThreadPool from multiprocessing.dummy import Pool as ThreadPool
import urllib
TELEMETRY_TEST_PATH_FORMAT = 'telemetry' TELEMETRY_TEST_PATH_FORMAT = 'telemetry'
...@@ -70,9 +69,7 @@ def SplitTestPath(test_result, test_path_format): ...@@ -70,9 +69,7 @@ def SplitTestPath(test_result, test_path_format):
else: else:
raise ValueError('Unknown test path format: %s' % test_path_format) raise ValueError('Unknown test path format: %s' % test_path_format)
# TODO(crbug.com/981349): Remove this after test paths are no longer test_path = test_result['testPath']
# url-quoted.
test_path = urllib.unquote(test_result['testPath'])
if separator not in test_path: if separator not in test_path:
raise ValueError('Invalid test path: %s' % test_path) raise ValueError('Invalid test path: %s' % test_path)
......
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