Remove VegaConverter TODO and update misleading comment

Author: mattseddon

dvc/render/converter/vega.py

@@ -114,10 +114,6 @@ def __init__(
         self.plot_id = plot_id
         self.inferred_properties: Dict = {}
 
-        # TODO we should be handling that in `convert`,
-        #      to avoid stateful `self.inferred_properties`
-        self._infer_x_y()
-
     def _infer_y_from_data(self):
         if self.plot_id in self.data:
             for lst in _lists(self.data[self.plot_id]):
@@ -296,16 +292,11 @@ def convert(
     ):
         """
         Convert the data. Fill necessary fields ('x', 'y') and return both
-        generated datapoints and updated properties. If `x` is not provided,
-        leave it as None, fronteds should handle it.
-
-        NOTE: Studio uses this method.
-              The only thing studio FE handles is filling `x` and `y`.
-              `x/y_label` should be filled here.
-
-              Datapoints are not stripped according to config, because users
-              might be utilizing other fields in their custom plots.
+        generated datapoints and updated properties. `x`, `y` values and labels
+        are inferred and always provided.
         """
+        self._infer_x_y()
+
         datapoints = self._find_datapoints()
         properties = {**self.properties, **self.inferred_properties}
 

tests/unit/render/test_vega_converter.py

@@ -24,6 +24,7 @@ def test_finding_lists(dictionary, expected_result):
     assert list(result) == expected_result
 
 
+@pytest.mark.studio
 @pytest.mark.parametrize(
     "input_data,properties,expected_datapoints,expected_properties",
     [