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

Invalid test after config added support for multilines #9631

Closed
berland opened this issue Jan 1, 2025 · 0 comments · Fixed by #9711
Closed

Invalid test after config added support for multilines #9631

berland opened this issue Jan 1, 2025 · 0 comments · Fixed by #9711

Comments

@berland
Copy link
Contributor

berland commented Jan 1, 2025

As the config parser now supports statements over multiple lines through the backslash operator, the following code:

def test_without_gen_kw(prior_ensemble, run_args, run_paths):
with fileinput.input("snake_oil.ert", inplace=True) as fin:
for line in fin:
if line.startswith("GEN_KW"):
continue
print(line, end="")

does not work as intended on the input file:

GEN_KW \
SNAKE_OIL_PARAM \
templates/snake_oil_template.txt \
snake_oil_params.txt \
parameters/snake_oil_parameters.txt

@berland berland moved this to Todo in SCOUT Jan 1, 2025
@eivindjahren eivindjahren moved this from Todo to In Progress in SCOUT Jan 13, 2025
@eivindjahren eivindjahren self-assigned this Jan 13, 2025
@eivindjahren eivindjahren mentioned this issue Jan 13, 2025
9 tasks
@eivindjahren eivindjahren moved this from In Progress to Ready for Review in SCOUT Jan 13, 2025
@github-project-automation github-project-automation bot moved this from Ready for Review to Done in SCOUT Jan 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

Successfully merging a pull request may close this issue.

2 participants