Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

test for nested if inside of else failing #418

Closed
wants to merge 1 commit into from
Closed

test for nested if inside of else failing #418

wants to merge 1 commit into from

Conversation

dishad
Copy link
Contributor

@dishad dishad commented Sep 25, 2020

Altered the original html template for templates/nestedConditional.html to include an <if> tag inside of an <else> tag that shouldn't be getting parsed since it should evaluate the first <if> to be true.

Teddy produces as output for this



      <p>The variable 'something' and 'somethingElse' are both present</p>


  <else>

      <p>The variable 'something' is not present and the variable 'somethingElse' is not present</p>

instead of the correct

<p>The variable 'something' and 'somethingElse' are both present</p>

@kethinov kethinov mentioned this pull request Sep 26, 2020
kethinov added a commit to kethinov/teddy that referenced this pull request Sep 26, 2020
- Closed rooseveltframework#412
- Closed rooseveltframework#418
- Closed many unreported bugs.
- Wrote a test for rooseveltframework#404
- Wrote a test for rooseveltframework#357 but upon further investigation now consider it a wontfix.
- Various dependencies bumped.
kethinov added a commit that referenced this pull request Sep 26, 2020
- Closed #412
- Closed #418
- Closed many unreported bugs.
- Wrote a test for #404
- Wrote a test for #357 but upon further investigation now consider it a wontfix.
- Various dependencies bumped.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant