[v3-2-test] airflow-ctl: fix variable import to correctly handle falsy values (#64362) (#64448)

(cherry picked from commit a908450)

Co-authored-by: Henry Chen <henryhenry0512@gmail.com>

Author: 2 people

airflow-ctl/src/airflowctl/ctl/commands/variable_command.py

@@ -51,7 +51,7 @@ def import_(args, api_client=NEW_API_CLIENT) -> list[str]:
     vars_to_update = []
     for k, v in var_json.items():
         value, description = v, None
-        if isinstance(v, dict) and v.get("value"):
+        if isinstance(v, dict) and "value" in v:
             value, description = v["value"], v.get("description")
 
         vars_to_update.append(

airflow-ctl/tests/airflow_ctl/ctl/commands/test_variable_command.py

@@ -83,6 +83,37 @@ def test_import_success(self, api_client_maker, tmp_path, monkeypatch):
         )
         assert response == [self.key]
 
+    @pytest.mark.parametrize(
+        "falsy_value",
+        [
+            "",
+            0,
+            False,
+        ],
+        ids=["empty_string", "zero", "false"],
+    )
+    def test_import_falsy_values(self, api_client_maker, tmp_path, monkeypatch, falsy_value):
+        """Test that falsy values (empty string, 0, False) are correctly imported."""
+        api_client = api_client_maker(
+            path="/api/v2/variables",
+            response_json=self.bulk_response_success.model_dump(),
+            expected_http_status_code=200,
+            kind=ClientKind.CLI,
+        )
+
+        monkeypatch.chdir(tmp_path)
+        expected_json_path = tmp_path / self.export_file_name
+        variable_file = {
+            self.key: {"value": falsy_value, "description": "test falsy value"},
+        }
+
+        expected_json_path.write_text(json.dumps(variable_file))
+        response = variable_command.import_(
+            self.parser.parse_args(["variables", "import", expected_json_path.as_posix()]),
+            api_client=api_client,
+        )
+        assert response == [self.key]
+
     def test_import_error(self, api_client_maker, tmp_path, monkeypatch):
         api_client = api_client_maker(
             path="/api/v2/variables",