refactor: removed the tree height parameter to rust and added a regression test

[nimrod-starkware]

…ssion test

---

This change is 

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 0% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 62.67%. Comparing base (5ca884e) to head (32e1fcd).
Report is 1 commits behind head on main.

Files	Patch %	Lines
crates/committer/src/patricia_merkle_tree/types.rs	0.00%	0 Missing and 1 partial ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #189      +/-   ##
==========================================
+ Coverage   62.06%   62.67%   +0.60%     
==========================================
  Files          36       36              
  Lines        1629     1677      +48     
  Branches     1629     1677      +48     
==========================================
+ Hits         1011     1051      +40     
- Misses        568      574       +6     
- Partials       50       52       +2     

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

[nimrod-starkware]

Reviewable status: 0 of 6 files reviewed, 1 unresolved discussion

a discussion (no related file):
python side PR

---

[dorimedini-starkware]

Reviewed 6 of 6 files at r1, all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @nimrod-starkware)

---

crates/committer_cli/src/tests/python_tests.rs line 155 at r1 (raw file):

fn get_tree_height() -> String {
    format!("{}", u8::from(TreeHeight::MAX))
}

wouldn't it be better to add derive_more::Display in the #[derive] attribute of TreeHeight? then you could do something like this

Suggestion:

fn get_tree_height() -> String {
    TreeHeight::MAX.into()
}

[nimrod-starkware]

Reviewable status: 5 of 7 files reviewed, 1 unresolved discussion (waiting on @dorimedini-starkware)

---

crates/committer_cli/src/tests/python_tests.rs line 155 at r1 (raw file):

Previously, dorimedini-starkware wrote…

wouldn't it be better to add derive_more::Display in the #[derive] attribute of TreeHeight? then you could do something like this

Done.

[dorimedini-starkware]

Reviewed 2 of 2 files at r2, all commit messages.
Reviewable status: complete! all files reviewed, all discussions resolved (waiting on @nimrod-starkware)