[app-configuration] Fix selecting ConfigurationSetting fields when th…

…ey contain an underscore (Azure#13258)

Fixing problem where some fields could not be properly `select`'d for ConfigurationSettings because they had an underscore.

This revealed some other underlying issues (like inconsistency in how the `select` field was populated in GetConfigurationSetting vs the List*Setting operations). Tests have been added, code refactored and hopefully things look better after this.

Fixes Azure#13126

sdk/appconfiguration/app-configuration/CHANGELOG.md

@@ -2,6 +2,9 @@
 
 ## 1.1.1 (Unreleased)
 
+- Fix issues with `select`ing fields to be returned from `listConfigurationSettings`, `listConfigurationRevisions`
+  and `getConfigurationSetting` where `last_modified` and `content_type` could not properly be passed in.
+  [PR #13258](https://github.com/Azure/azure-sdk-for-js/pull/13258)
 
 ## 1.1.0 (2020-07-07)
 

sdk/appconfiguration/app-configuration/package.json

@@ -62,9 +62,9 @@
     "integration-test:node": "npm run build:test:node && nyc mocha -r esm --require source-map-support/register --reporter ../../../common/tools/mocha-multi-reporter.js --timeout 180000 --full-trace dist-esm/test/*.spec.js dist-esm/test/**/*.spec.js",
     "test:browser": "npm run clean && npm run build:test:browser && npm run unit-test:browser",
     "test:node": "npm run clean && npm run build:test:node && npm run unit-test:node",
-    "test": "npm run clean && npm run build:test && npm run unit-test",
+    "test": "npm run test:node && npm run test:browser",
     "unit-test:browser": "",
-    "unit-test:node": "mocha --require source-map-support/register --reporter ../../../common/tools/mocha-multi-reporter.js --timeout 180000 --full-trace test-dist/index.node.js",
+    "unit-test:node": "npm run build:test:node && mocha --require source-map-support/register --reporter ../../../common/tools/mocha-multi-reporter.js --timeout 180000 --full-trace test-dist/index.node.js",
     "unit-test": "npm run unit-test:node && npm run unit-test:browser",
     "docs": "typedoc --excludePrivate --excludeNotExported --excludeExternals --stripInternal --mode file --out ./dist/docs ./src"
   },