Fix a regression in Subversion diff parsing.

Review Request #11724 — Created July 15, 2021 and submitted

Information

Review Board
release-4.0.x

Reviewers

A recent change intended to fix a deleted files indicator in Subversion
diffs regressed some parsing. We weren't checking whether the field in
question was None. This actually impacted a unit test, but it seems we
missed it (as CI is temporarily busted).

This change adds the necessary bullet-proofing, fixing the test.

Applicable unit tests pass.

Tested with a customer.

Summary ID
Fix a regression in Subversion diff parsing.
A recent change intended to fix a deleted files indicator in Subversion diffs regressed some parsing. We weren't checking whether the field in question was `None`. This actually impacted a unit test, but it seems we missed it (as CI is temporarily busted). This change adds the necessary bullet-proofing, fixing the test.
9482edca9d63a638597eb498ebef187bcdcb278b
david
  1. Ship It!
  2. 
      
chipx86
Review request changed
Status:
Completed
Change Summary:
Pushed to release-4.0.x (e18d3b6)