Fixes CB-7781

[aetchegoyen]

With this change the issue reported in https://issues.apache.org/jira/browse/CB-7781.
Tested and working perfectly now while uninstalling plugin

[dseravalli]

Is this going to be merged in?

[mreinstein]

why do the tests fail with this patch?

[TooTallNate]

Needs a regression test.

[Bnaya]

waiting forward for this,
How's the regression test should looks like?

[absynce]

FWIW I manually applied this patch and it allowed me to remove a plugin in Ionic/Cordova.

[Bnaya]

same here

[mreinstein]

I'm glad that manually applying the patch fixes your problem, but it would be nice if this pull request had at least one test demonstrating the fix.

Submitting patches without tests isn't acceptable.

[benallfree]

@mreinstein Here you go, for test/build.js:

    it('should create an empty plist XML string from null input', function () {
      var xml = build();
      assert.strictEqual(xml, multiline(function () {/*
<?xml version="1.0" encoding="UTF-8"?>
<!DOCTYPE plist PUBLIC "-//Apple//DTD PLIST 1.0//EN" "http://www.apple.com/DTDs/PropertyList-1.0.dtd">
<plist version="1.0"/>
*/}));
    });

[mreinstein]

I've finally got the testing infrastructure somewhat working again. If you re-submit a clean PR with tests included we can merge this now.

Sorry for the painfully long response time on this. 🤦