Whole file
robotframework/robotframework
The author described this change as “Fix `Var.from_params` with empty `value_separator`”. It counts as a record because the check below fails on the code as it stood at 88058afba and passes on 4e91aa6d7, with nothing else changed between the two runs.
Fix saved2026-07-23
Sharing licenceApache-2.0 · LICENSE.txt
Change size+1 −1
What the code was meant to do, written into the code itself as a save note
Fix `Var.from_params` with empty `value_separator`
The change
| 1559 | 1559 | Token(Token.SEPARATOR, separator), | |
| 1560 | 1560 | Token(Token.OPTION, f"scope={scope}"), | |
| 1561 | 1561 | ] | |
| 1562 | - | if value_separator: | |
| 1562 | + | if value_separator is not None: | |
| 1563 | 1563 | tokens += [ | |
| 1564 | 1564 | Token(Token.SEPARATOR, separator), | |
| 1565 | 1565 | Token(Token.OPTION, f"separator={value_separator}"), |
The check that tells the two apart
fail→pass·utest/parsing/test_statements.py::TestCreateStatementsFromParams::test_Var
Check file utest/parsing/test_statements.py, taken without changes from the fix and copied onto the older code, so the exact same check runs against both versions.
Origin and history
The code before it88058afbaadf36081544ab644bec99b246f404d5
Broken version dated2026-07-17
Modulerobot.parsing.model.statements
Units changedVar
Fingerprinte0dfef74c3cb3bcc
Checked2026-08-18 by goldset/0.1
Every field above is generated by our program. None of it is written by hand.