[Transpose & generate_tests.py]: Added join
env Function & Changed Template
#3760
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
As a result of the discussion here in the forum and the previously logged issue here in GitHub, decided to alter the
JinJa2
template for this exercise, so that the test case input was "pre joined" prior to verifying it in the test case.This is to avoid any ambiguity around whether the function input is intended to be a
list
or astr
(the intent from problem specs is that the inputs arestr
s).[x] Changed
generate_tests.py
to add thejoin_test_inputs()
function & make it available in the template environment.[x] Changed the
JinJa2
template to use the pre-join functionality, and to preserve the\n
characters.[x] Changed the stub to reflect renaming
lines
totext
for the input parameter.[x] Regenerated a new test file & tested
example.py
against it.