diff --git a/dbt_sugar/core/clients/dbt.py b/dbt_sugar/core/clients/dbt.py index 415c85da..e794e890 100644 --- a/dbt_sugar/core/clients/dbt.py +++ b/dbt_sugar/core/clients/dbt.py @@ -187,7 +187,6 @@ def read_profile(self): _profile_dict = open_yaml(self.profiles_dir / "profiles.yml") _profile_dict = _profile_dict.get(self._profile_name, _profile_dict.get(self._profile_name)) if _profile_dict: - # read target name from args or try to get it from the dbt_profile `target:` field. _target_profile = self._get_target_profile(profile_dict=_profile_dict) diff --git a/dbt_sugar/core/connectors/snowflake_connector.py b/dbt_sugar/core/connectors/snowflake_connector.py index a1f19001..dfa5486e 100644 --- a/dbt_sugar/core/connectors/snowflake_connector.py +++ b/dbt_sugar/core/connectors/snowflake_connector.py @@ -55,7 +55,6 @@ def __init__( def get_columns_from_table( self, target_table: str, target_schema: str, use_describe: bool = False ) -> Sequence[str]: - # if user wants to use describe (more preformant but with caveat) method # we re-implement column describe since snowflake.sqlalchemy is shit. if use_describe: diff --git a/dbt_sugar/core/ui/cli_ui.py b/dbt_sugar/core/ui/cli_ui.py index 4182a3db..8ac1d33a 100644 --- a/dbt_sugar/core/ui/cli_ui.py +++ b/dbt_sugar/core/ui/cli_ui.py @@ -323,7 +323,6 @@ def _document_undocumented_cols( self, question_payload: Sequence[Mapping[str, Any]], ) -> Mapping[str, Mapping[str, Union[str, List[str]]]]: - columns_to_document = question_payload[0].get("choices", list()) quantifier_word = self._set_quantifier_word() # check if user wants to document all columns diff --git a/tests/cli_ui_test.py b/tests/cli_ui_test.py index 5887f479..530aceba 100644 --- a/tests/cli_ui_test.py +++ b/tests/cli_ui_test.py @@ -53,7 +53,6 @@ def unsafe_ask(self): ], ) def test__document_model(mocker, question_payload, questionary_outputs, expected_results): - mocker.patch("questionary.prompt", return_value=questionary_outputs) results = UserInputCollector(question_type="model", question_payload=question_payload).collect() if questionary_outputs.get("wants_to_document_model") is False: