#3 Add tests for the OpamString type
Merged by jjames. Opened by defolos.
defolos/opam2rpm more_tests  into  master

Download 3.patch

I have added type hints and tests to the OpamString type and while doing that realized that decoding of escape strings does not work properly. I have therefore changed lines 80 & 89 in opamparser.py to properly handle escape sequences.

However, I am not entirely sure whether that is the correct result. Please take a look at the unit tests, whether the results are what you expect.

Yes, that looks correct. Thanks for the tests and bug fixes!

Pull-Request has been merged by jjames

Metadata