ubuntu-system-tests

Merge lp://staging/~canonical-platform-qa/ubuntu-system-tests/config-handle-special-characters into lp://staging/ubuntu-system-tests

  1. config-handle-special-characters
  2. Merge into trunk
Proposed by Christopher Lee
Status: Work in progress
Proposed branch: lp://staging/~canonical-platform-qa/ubuntu-system-tests/config-handle-special-characters
Merge into: lp://staging/ubuntu-system-tests
Diff against target: 38 lines (+13/-2)
1 file modified
ubuntu_sanity_tests/selftests/test_config.py (+13/-2)
To merge this branch: bzr merge lp://staging/~canonical-platform-qa/ubuntu-system-tests/config-handle-special-characters
Reviewer Review Type Date Requested Status
Canonical Platform QA Team Pending
Review via email: mp+256261@code.staging.launchpad.net

Description of the change

A failing tests highlighting that uciconfig ignores part of a string if it contains a '#' character.

To post a comment you must log in.
Revision history for this message
Christopher Lee (veebers) wrote :

It seems that the complete string is lost in uciconfig.parsers.Parsers.parse_config()

Reply
Revision history for this message
Vincent Ladeuil (vila) wrote :

That's the expected behaviour, '#' starts a comment.

What is missing in uciconfig is a way to quote values.

Reply

Unmerged revisions

108. By Christopher Lee

Failing test for config containing special characters.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to all changes: