fix: allow single quotes in key=value pairs

[RodEsp]

What does this PR do?

Allows using ' in the value of name-value pairs for flags such as force:data:record:create's --values.

What issues does this PR fix or reference?

forcedotcom/cli#1820
@W-12105706@

[peternhale]

I know there are not a lot UTs in this modules, but maybe one or two more around this change.

[RodEsp]

Heya @peternhale, I created #428 so that we can test the behavior of the existing method and make sure that the changes in this PR aren't introducing any regressions.

Once we merge #428 I'll merge main into this PR's branch and use the new unit tests to continue tweaking the regex here.

[RodEsp]

Heya @peternhale, this is ready for another look.

Unfortunately, I could not get the regex to work with both weird Unicode characters and values like key="leftvalue rightvalue" at the same time, so it looks like we're stuck with the current lengthy code that checks for the existence of 's and "s.

[peternhale]

@RodEsp I found another valid string that does not make it through the function intact, but it probably would of failed in the current code too.

  it('should allow double and single quotes in key=value pairs', () => {
    const dict = dataCommand.testStringToDictionary("key='39° 44' 31.3548\" N'");

    expect(dict).to.deep.equal({
      key: '39° 44\' 31.3548" N',
    });
  });

[RodEsp]

Yeah, that didn't work without the changes in this PR either. The problem is with values that have quoted strings inside quoted strings, e.g. key="this is a 'quote'" or key='this is a "quote"'. I think it'll be a greater effort to get that to work, but for the sake of fixing forcedotcom/cli#1820 asap I'd like to merge this now and I created forcedotcom/cli#1837 to address strings like the one in your unit test later.