From 010060d932e236c7eb6802d696c68be8ac0d91ec Mon Sep 17 00:00:00 2001 From: SerenaFeng Date: Tue, 13 Mar 2018 11:33:27 +0800 Subject: format URLs 1. unify URL query 2. unify URL format Change-Id: I4bdf13067c4f726df42a8e34b4365fed4f58924e Signed-off-by: SerenaFeng --- testapi/testapi-client/testapiclient/utils/command.py | 8 -------- .../testapi-client/testapiclient/utils/url_parse.py | 18 ++++++++++++++++++ 2 files changed, 18 insertions(+), 8 deletions(-) (limited to 'testapi/testapi-client/testapiclient/utils') diff --git a/testapi/testapi-client/testapiclient/utils/command.py b/testapi/testapi-client/testapiclient/utils/command.py index c99a3f3..9614acf 100644 --- a/testapi/testapi-client/testapiclient/utils/command.py +++ b/testapi/testapi-client/testapiclient/utils/command.py @@ -7,7 +7,6 @@ from cliff import show import six from testapiclient import utils -from testapiclient.utils import url_parse class CommandMeta(abc.ABCMeta): @@ -38,13 +37,6 @@ class Command(command.Command): class Lister(Command, lister.Lister): - @staticmethod - def filter_by_name(url, parsed_args): - def query_url(): - return url_parse.query_join(url, name=parsed_args.name) - - return query_url() if parsed_args.name else url - @staticmethod def format_output(columns, data): return (columns, diff --git a/testapi/testapi-client/testapiclient/utils/url_parse.py b/testapi/testapi-client/testapiclient/utils/url_parse.py index 08f7a63..fbc503a 100644 --- a/testapi/testapi-client/testapiclient/utils/url_parse.py +++ b/testapi/testapi-client/testapiclient/utils/url_parse.py @@ -20,3 +20,21 @@ def query_join(base, **queries): def resource_join(url): testapi_url = os.environ.get('testapi_url') return path_join(testapi_url, url) + + +def get_queries(queries, parsed_args): + if not isinstance(queries, list): + queries = [queries] + + return {query: getattr(parsed_args, query) + for query in queries + if hasattr(parsed_args, query)} + + +def query_by(base, queries, parsed_args): + return query_join(base, + **get_queries(queries, parsed_args)) + + +def url_format(base, parsed_args): + return base.format(**(parsed_args.__dict__)) -- cgit 1.2.3-korg