Addressing issues 77, 78, and 81, where leading spaces and empty environment variables cause issues

[mc-etcher]

Addressing issues 77, 78, and 81, where leading spaces and empty environment variables cause issues

Issue #, if available:

• 77
• 78
• 81

Description of changes: Details in the CHANGELOG

• Updated VPC Handler:
  • Removes leading spaces for each item in the ListOfCustomCidrBlocks and CustomerManagedPrefixListIds variables
  • Handles empty environment variables
• Added tests to account for leading spaces in the ListOfCustomCidrBlocks and CustomerManagedPrefixListIds variables
• Addressed a case where the test_vpc_default_route_crud_operations_custom_destinations wasn't adding a subnet
• Changed default values for tests in override_environment_variables()

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

[groverlalit]

Thanks for opening the pull request and help us improve the codebase. We have updated the code to handle this scenario. We should not have empty strings in the list. Handling this use case in the new util where we remove white spaces in the values in the string.