Fix #2333. Remove not none arg parse defaults.

[ilovelinux]

Steps

• Remove not None arg parse defaults
• Check that default values for those options, provided in __main__.Config, are the same.

Fix #2333

Breaking changes

None

[gaborbernat]

Please add tests validating the change.

[codspeed-hq]

CodSpeed Performance Report

Merging #2334 will not alter performance

_{Comparing ilovelinux:fix/2333-ilovelinux-fix-defaults (17b9a3c) with main (ef0c97c)}

Summary

✅ 31 untouched benchmarks

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.39%. Comparing base (ef0c97c) to head (17b9a3c).

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #2334   +/-   ##
=======================================
  Coverage   98.39%   98.39%           
=======================================
  Files          65       65           
  Lines        7792     7798    +6     
  Branches      775      776    +1     
=======================================
+ Hits         7667     7673    +6     
  Misses         91       91           
  Partials       34       34           

Flag	Coverage Δ
unittests	98.39% <100.00%> (+<0.01%)	⬆️

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.

[gaborbernat]

I don't think this is the right test. We want to see that the pyprojct.toml values are respected, not that no values have default values.

[ilovelinux]

I know but... Do you have any idea how to test that for all values without setting them by hand? I can't find a way to test the code in __main__.py easily.

datamodel-code-generator/src/datamodel_code_generator/__main__.py

Lines 366 to 377 in ef0c97c

	pyproject_config = _get_pyproject_toml_config(Path.cwd())
	if pyproject_config is not None:
	pyproject_toml = {k.replace("-", "_"): v for k, v in pyproject_config.items()}
	else:
	pyproject_toml = {}
	try:
	config = Config.parse_obj(pyproject_toml)
	config.merge_args(namespace)
	except Error as e:
	print(e.message, file=sys.stderr) # noqa: T201
	return Exit.ERROR

This tests ensure that if a parameter is not given, then namespace has no parameters set. This will ensure that no values from pyproject.toml (L373) is overwritten (L374)

[ilovelinux]

CC: @koxudaxi