add AgentOptimizer test in CI

[skzhang1]

Why are these changes needed?

Related issue number

Closes #2353

Checks

• I've included any doc changes needed for https://microsoft.github.io/autogen/. See https://microsoft.github.io/autogen/docs/Contribute#documentation to build and test documentation locally.
• I've added tests (if relevant) corresponding to the changes introduced in this PR.
• I've made sure all auto checks have passed.

[codecov-commenter]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 37.02%. Comparing base (4a44093) to head (f5edbe8).
Report is 18 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #2380      +/-   ##
==========================================
- Coverage   38.14%   37.02%   -1.12%     
==========================================
  Files          78       81       +3     
  Lines        7865     8084     +219     
  Branches     1683     1724      +41     
==========================================
- Hits         3000     2993       -7     
- Misses       4615     4840     +225     
- Partials      250      251       +1     

Flag	Coverage Δ
unittests	37.02% <ø> (-1.11%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[WaelKarkoub]

--skip-docker

Thanks for the suggestion. Actually, the docker is optional.

I'm going through the AgentOptimizer implementation, and it seems like it's using docker to execute code. We might need to add a --skip-docker flag for those tests in case it wasn't installed

Thanks for the suggestion. Docker is not mandatory for AgentOptimizer. It is optional.

Moving the convo to this new PR. In test_record_conversation and test_step, they set up a user proxy with docker code execution

autogen/test/agentchat/contrib/test_agent_optimizer.py

Lines 32 to 41 in d473dee

	user_proxy = UserProxyAgent(
	name="user_proxy",
	human_input_mode="NEVER",
	code_execution_config={
	"work_dir": f"{here}/test_agent_scripts",
	"use_docker": "python:3",
	"timeout": 60,
	},
	max_consecutive_auto_reply=3,
	)

autogen/test/agentchat/contrib/test_agent_optimizer.py

Lines 77 to 86 in d473dee

	user_proxy = UserProxyAgent(
	name="user_proxy",
	human_input_mode="NEVER",
	code_execution_config={
	"work_dir": f"{here}/test_agent_scripts",
	"use_docker": "python:3",
	"timeout": 60,
	},
	max_consecutive_auto_reply=3,
	)

If they're not necessary, I would recommend removing them as they could cause the tests to be flaky.