Only allow nonnegative version numbers

[sbadithe]

• The test in question was testing invalid version numbers in the case where the minor version number was 0. The latest release of the packaging library started throwing an error for this case. This PR resolves that by taking the max of (0, minor-1) to prevent negative version numbers.

[codecov]

Codecov Report

Merging #2389 (de947f7) into main (722fbe8) will decrease coverage by 0.05%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #2389      +/-   ##
==========================================
- Coverage   99.50%   99.44%   -0.06%     
==========================================
  Files         322      326       +4     
  Lines       20161    20205      +44     
==========================================
+ Hits        20061    20093      +32     
- Misses        100      112      +12     

Impacted Files	Coverage Δ
featuretools/entityset/deserialize.py	100.00% <100.00%> (ø)
featuretools/entityset/serialize.py	100.00% <100.00%> (ø)
featuretools/feature_base/features_deserializer.py	100.00% <100.00%> (ø)
featuretools/feature_base/features_serializer.py	100.00% <100.00%> (ø)
...etools/tests/entityset_tests/test_serialization.py	100.00% <100.00%> (ø)
...ests/primitive_tests/test_feature_serialization.py	100.00% <100.00%> (ø)
...ests/primitive_tests/test_features_deserializer.py	100.00% <100.00%> (ø)
.../tests/primitive_tests/test_features_serializer.py	100.00% <100.00%> (ø)
featuretools/utils/gen_utils.py	100.00% <100.00%> (ø)
featuretools/utils/schema_utils.py	100.00% <100.00%> (ø)
... and 4 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

[rwedge]

Does it make sense for these tests to be tied to the actual schema version number? Should we mock the schema version instead to get consistent test coverage?

[thehomebrewnerd]

@sbadithe When you pull in the latest changes from main you should also update pyproject.toml to remove the upper bound restriction on packaging.

[sbadithe]

With this refactor, we no longer have to import the schema version conditionally inside the function to avoid a circular import error -- we can just do it at the top

[rwedge]

remove in favor of function in schema_utils.py