fix: add missing functional test

tests/functional/L1_Functional_Tests_GPU.sh

@@ -23,32 +23,37 @@ cd ${PROJECT_ROOT}
 time bash ./tests/functional/grpo_frozen_env.sh
 time bash ./tests/functional/test_frozen_env.sh
 
-time uv run --no-sync bash ./tests/functional/sft.sh
-time uv run --no-sync bash ./tests/functional/sft_resume_diamond.sh
+time uv run --no-sync bash ./tests/functional/distillation.sh
+time uv run --no-sync bash ./tests/functional/distillation_megatron.sh
+time uv run --no-sync bash ./tests/functional/dpo.sh
+time uv run --no-sync bash ./tests/functional/dpo_automodel_lora.sh
+time uv run --no-sync bash ./tests/functional/dpo_megatron.sh
+time uv run --no-sync bash ./tests/functional/eval.sh
+time uv run --no-sync bash ./tests/functional/eval_async.sh
 time uv run --no-sync bash ./tests/functional/grpo.sh
-time uv run --no-sync bash ./tests/functional/prorlv2.sh
 time uv run --no-sync bash ./tests/functional/grpo_async.sh
+time uv run --no-sync bash ./tests/functional/grpo_automodel_lora.sh
 time uv run --no-sync bash ./tests/functional/grpo_automodel_lora_async.sh
 time uv run --no-sync bash ./tests/functional/grpo_automodel_lora_non_colocated.sh
-time uv run --no-sync bash ./tests/functional/grpo_automodel_lora.sh
 time uv run --no-sync bash ./tests/functional/grpo_megatron.sh
 time uv run --no-sync bash ./tests/functional/grpo_megatron_generation.sh
+time uv run --no-sync bash ./tests/functional/grpo_multiple_datasets.sh
 time uv run --no-sync bash ./tests/functional/grpo_multiturn.sh
 time uv run --no-sync bash ./tests/functional/grpo_non_colocated.sh
+time uv run --no-sync bash ./tests/functional/grpo_rm_env.sh
 # time uv run --no-sync bash ./tests/functional/grpo_sglang.sh
-time uv run --no-sync bash ./tests/functional/grpo_multiple_datasets.sh
-time uv run --no-sync bash ./tests/functional/dpo_automodel_lora.sh
-time uv run --no-sync bash ./tests/functional/dpo.sh
+time uv run --no-sync bash ./tests/functional/prorlv2.sh
 time uv run --no-sync bash ./tests/functional/rm.sh
-time uv run --no-sync bash ./tests/functional/eval.sh
-time uv run --no-sync bash ./tests/functional/eval_async.sh
-time uv run --no-sync bash ./tests/functional/test_mcore_extra_installed_correctly.sh
-time uv run --no-sync bash ./tests/functional/test_automodel_extra_installed_correctly.sh
-time uv run --no-sync bash ./tests/functional/vlm_grpo.sh
-time uv run --no-sync bash ./tests/functional/distillation.sh
-time uv run --no-sync bash ./tests/functional/distillation_megatron.sh
+time uv run --no-sync bash ./tests/functional/sft.sh
 time uv run --no-sync bash ./tests/functional/sft_automodel_lora.sh
+time uv run --no-sync bash ./tests/functional/sft_megatron.sh
 time uv run --no-sync bash ./tests/functional/sft_megatron_lora.sh
+time uv run --no-sync bash ./tests/functional/sft_resume_diamond.sh
+time uv run --no-sync bash ./tests/functional/test_automodel_extra_installed_correctly.sh
+# Re-enable once DTensor v2 converter is fixed.
+# time uv run --no-sync bash ./tests/functional/test_converters.sh
+time uv run --no-sync bash ./tests/functional/test_mcore_extra_installed_correctly.sh
+time uv run --no-sync bash ./tests/functional/vlm_grpo.sh
 
 # Research functional tests (self-discovery)
 for test_script in research/*/tests/functional/*.sh; do

tests/functional/dpo_megatron.sh

@@ -22,24 +22,26 @@ mkdir -p $EXP_DIR $LOG_DIR
 
 cd $PROJECT_ROOT
 uv run $PROJECT_ROOT/examples/run_dpo.py \
-    --config $PROJECT_ROOT/examples/configs/recipes/llm/dpo-llama3.1-8b-instruct-4n8g-megatron.yaml \
+    --config $PROJECT_ROOT/examples/configs/recipes/llm/dpo-llama3.1-8b-instruct-4n8g-megatron.v2.yaml \
     policy.model_name=Qwen/Qwen3-0.6B \
-    cluster.gpus_per_node=2 \
     dpo.max_num_steps=3 \
     dpo.val_batches=1 \
     dpo.val_period=3 \
+    policy.train_global_batch_size=8 \
+    policy.megatron_cfg.tensor_model_parallel_size=1 \
+    policy.megatron_cfg.sequence_parallel=false \
     logger.tensorboard_enabled=true \
     logger.log_dir=$LOG_DIR \
     logger.wandb_enabled=false \
     logger.monitor_gpus=true \
     checkpointing.enabled=false \
-    policy.megatron_cfg.tensor_model_parallel_size=1 \
-    policy.train_global_batch_size=8 \
+    cluster.gpus_per_node=2 \
+    cluster.num_nodes=1 \
     $@ \
     2>&1 | tee $RUN_LOG
 
 uv run tests/json_dump_tb_logs.py $LOG_DIR --output_path $JSON_METRICS
 
 uv run tests/check_metrics.py $JSON_METRICS \
-  'data["train/loss"]["3"] < 5' \
+  'data["train/loss"]["3"] < 6' \
 

tests/unit/test_recipes_and_test_suites.py

@@ -310,3 +310,28 @@ def test_all_recipes_start_with_algo_hyphen(all_recipe_yaml_rel_paths):
         assert algo in expected_algos, (
             f"Recipe {recipe_yaml} has unexpected algo {algo}"
         )
+
+
+def test_functional_tests_exist():
+    functional_tests_dir = os.path.join(project_root, "tests", "functional")
+
+    test_list = []
+    with open(
+        os.path.join(functional_tests_dir, "L1_Functional_Tests_GPU.sh"), "r"
+    ) as f:
+        for line in f:
+            line = line.strip()
+            if line and "./tests/functional" in line:
+                test_list.append(line.split(" ")[-1].split("/")[-1])
+
+    missing_list = []
+    for filename in os.listdir(functional_tests_dir):
+        if filename.endswith(".sh"):
+            if filename == "L1_Functional_Tests_GPU.sh":
+                continue
+            if filename not in test_list:
+                missing_list.append(f"./tests/functional/{filename}")
+
+    assert len(missing_list) == 0, (
+        f"Missing functional test scripts in ./tests/functional/L1_Functional_Tests_GPU.sh:\n{'\n'.join(missing_list)}"
+    )