Copy properties from sql object explicitly

[blakeembrey]

Closes #2253.

[dougwilson]

Do you think using Object.create would be sufficient but still keep from maintaining a list at that location? Adding a test that represents the issue would enable me to answer my own question vs defer to you, ps :)

[blakeembrey]

@dougwilson Yes, pretty sure, assuming no one is iterating over the Object.keys elsewhere within the project. Where should I add a test case? I kind of hoped the existing tests of cover this, since I'm not changing functionality.

[dougwilson]

Where should I add a test case?

So the test files are in the test/ directory. You'll probably wanted to add them under unit since these are likely unit tests.

I kind of hoped the existing tests of cover this, since I'm not changing functionality.

Well, if there was an existing test, then I guess this PR is unneeded? Not sure why the test suite would be passing without your change here if there is already a test in this module's suite?

[blakeembrey]

Well, if there was an existing test, then I guess this PR is unneeded? Not sure why the test suite would be passing without your change here if there is already a test in this module's suite?

Derp, right - let me add this.

[blakeembrey]

Added a test that I believe should work given node 0.6's feature set.

[blakeembrey]

@dougwilson Bump.

[dougwilson]

I plan to have a new minor out next week and will get this in there.

[dougwilson]

My humblest apologies on the delay...