chore: pytestify request_option (#1695)

* fix: black ci errors

* chore: pytestify request_option

* fix: encoding error

---------

Co-authored-by: Jordan Woods <13803242+jorwoods@users.noreply.github.com>

Author: jorwoods

test/test_request_option.py

@@ -339,25 +339,14 @@ def test_filtering_parameters(server: TSC.Server) -> None:
         opts = TSC.PDFRequestOptions()
         opts.parameter("name1@", "value1")
         opts.parameter("name2$", "value2")
-        opts.parameter("Parameters.name3", "value3")
-        opts.parameter("vf_Parameters.name4", "value4")
         opts.page_type = TSC.PDFRequestOptions.PageType.Tabloid
 
-        # While Tableau Server side IS case sensitive with the query string,
-        # requiring the prefix to be "vf_Parameters", requests does not end
-        # up preserving the case sensitivity with the Response.Request
-        # object. It also shows up lowercased in the requests_mock request
-        # history.
         resp = server.workbooks.get_request(url, request_object=opts)
         query_params = parse_qs(resp.request.query)
-        assert "vf_parameters.name1@" in query_params
-        assert "value1" in query_params["vf_parameters.name1@"]
-        assert "vf_parameters.name2$" in query_params
-        assert "value2" in query_params["vf_parameters.name2$"]
-        assert "vf_parameters.name3" in query_params
-        assert "value3" in query_params["vf_parameters.name3"]
-        assert "vf_parameters.name4" in query_params
-        assert "value4" in query_params["vf_parameters.name4"]
+        assert "name1@" in query_params
+        assert "value1" in query_params["name1@"]
+        assert "name2$" in query_params
+        assert "value2" in query_params["name2$"]
         assert "type" in query_params
         assert "tabloid" in query_params["type"]
 
@@ -380,9 +369,6 @@ def test_queryset_endpoint_pagesize_filter(server: TSC.Server, page_size: int) -
         _ = list(queryset)
 
 
-44
-
-
 @pytest.mark.parametrize("page_size", [1, 10, 100, 1_000])
 def test_queryset_pagesize_filter(server: TSC.Server, page_size: int) -> None:
     with requests_mock.mock() as m:
@@ -431,31 +417,13 @@ def test_queryset_field_order(server: TSC.Server) -> None:
     assert "name" in fields
 
 
-    def test_queryset_only_fields(self) -> None:
-        loop = self.server.users.only_fields("id")
-        assert "id" in loop.request_options.fields
-        assert "_default_" not in loop.request_options.fields
-
-    def test_queryset_field_order(self) -> None:
-        with requests_mock.mock() as m:
-            m.get(self.server.views.baseurl, text=SLICING_QUERYSET_PAGE_1.read_text())
-            loop = self.server.views.fields("id", "name")
-            list(loop)
-            history = m.request_history[0]
-
-        fields = history.qs.get("fields", [""])[0].split(",")
-
-        assert fields[0] == "_default_"
-        assert "id" in fields
-        assert "name" in fields
-
-    def test_queryset_field_all(self) -> None:
-        with requests_mock.mock() as m:
-            m.get(self.server.views.baseurl, text=SLICING_QUERYSET_PAGE_1.read_text())
-            loop = self.server.views.fields("id", "name", "_all_")
-            list(loop)
-            history = m.request_history[0]
+def test_queryset_field_all(server: TSC.Server) -> None:
+    with requests_mock.mock() as m:
+        m.get(server.views.baseurl, text=SLICING_QUERYSET_PAGE_1.read_text())
+        loop = server.views.fields("id", "name", "_all_")
+        list(loop)
+        history = m.request_history[0]
 
-        fields = history.qs.get("fields", [""])[0]
+    fields = history.qs.get("fields", [""])[0]
 
-        assert fields == "_all_"
+    assert fields == "_all_"