Update deps to [email protected]

Author: frankfarzan

.github/workflows/ci.yaml

@@ -72,7 +72,7 @@ jobs:
     needs: [node-ci, go-ci]
     strategy:
       matrix:
-        platform: [ubuntu-latest]
+        platform: [ubuntu-latest, macos-latest]
         node-version: [12.x]
     runs-on: ${{ matrix.platform }}
     steps:

ts/demo-functions/package-lock.json

@@ -22,7 +22,7 @@
     "kpt:type-create": "kpt type-create"
   },
   "dependencies": {
-    "@googlecontainertools/kpt-functions": "^0.11.0",
+    "@googlecontainertools/kpt-functions": "^0.12.0",
     "glob": "^7.1.3",
     "js-yaml": "^3.13.1"
   },

ts/demo-functions/package.json

@@ -14,7 +14,7 @@
  * limitations under the License.
  */
 
-import { Configs, TestRunner } from '@googlecontainertools/kpt-functions';
+import { Configs, TestRunner, ConfigError } from '@googlecontainertools/kpt-functions';
 import { labelNamespace } from './label_namespace';
 import { Namespace, ConfigMap } from './gen/io.k8s.api.core.v1';
 
@@ -30,7 +30,7 @@ const FUNC_CONFIG: ConfigMap = new ConfigMap({
 describe('labelNamespace', () => {
   it('empty input ok', RUNNER.assertCallback(new Configs(undefined, FUNC_CONFIG)));
 
-  it('requires functionConfig', RUNNER.assertCallback(undefined, undefined, TypeError));
+  it('requires functionConfig', RUNNER.assertCallback(undefined, undefined, ConfigError));
 
   it('adds label namespace when metadata.labels is undefined', async () => {
     const input = new Configs(undefined, FUNC_CONFIG);

ts/demo-functions/src/label_namespace_test.ts

@@ -44,7 +44,7 @@ describe('readYaml', () => {
   it('replicates test dir', async () => {
     const sourceDir = path.resolve(__dirname, '../test-data/source/foo-yaml');
     const expectedIntermediateFile = path.resolve(__dirname, '../test-data/intermediate/foo.yaml');
-    const expectedConfigs = readConfigs(expectedIntermediateFile, FileFormat.YAML);
+    const expectedConfigs = await readConfigs(expectedIntermediateFile, FileFormat.YAML);
     functionConfig.data![SOURCE_DIR] = sourceDir;
     const actualConfigs = new Configs(undefined, functionConfig);
 

ts/demo-functions/src/read_yaml_test.ts

@@ -14,7 +14,7 @@
  * limitations under the License.
  */
 
-import { Configs, TestRunner } from '@googlecontainertools/kpt-functions';
+import { Configs, TestRunner, ConfigError } from '@googlecontainertools/kpt-functions';
 import { ClusterRoleBinding, RoleBinding, Subject } from './gen/io.k8s.api.rbac.v1';
 import { validateRolebinding } from './validate_rolebinding';
 import { ConfigMap } from './gen/io.k8s.api.core.v1';
@@ -26,7 +26,7 @@ const FUNC_CONFIG: ConfigMap = new ConfigMap({
 });
 
 describe(validateRolebinding.name, () => {
-  it('passes empty input', RUNNER.assertCallback(undefined, undefined, TypeError));
+  it('passes empty input', RUNNER.assertCallback(undefined, undefined, ConfigError));
 
   it(
     'passes valid RoleBindings',

ts/demo-functions/src/validate_rolebinding_test.ts

@@ -46,7 +46,7 @@ describe('writeYaml', () => {
   });
 
   it('test dir', async () => {
-    const input = readIntermediate();
+    const input = await readIntermediate();
     functionConfig.data![SINK_DIR] = tmpDir;
     const configs = new kpt.Configs(input.getAll(), functionConfig);
 
@@ -57,7 +57,7 @@ describe('writeYaml', () => {
 
   it("throws if --overwrite isn't passed for non-empty directory", async () => {
     fs.copySync(SINK_DIR_EXPECTED, tmpDir);
-    const input = readIntermediate();
+    const input = await readIntermediate();
     functionConfig.data![SINK_DIR] = tmpDir;
     const configs = new kpt.Configs(input.getAll(), functionConfig);
 
@@ -66,7 +66,7 @@ describe('writeYaml', () => {
 
   it("silently makes output directory if it doesn't exist", async () => {
     const sinkDir = path.resolve(tmpDir, 'foo');
-    const input = readIntermediate();
+    const input = await readIntermediate();
     functionConfig.data![SINK_DIR] = sinkDir;
     const configs = new kpt.Configs(input.getAll(), functionConfig);
 
@@ -88,7 +88,7 @@ describe('writeYaml', () => {
       path.resolve(tmpDir, 'foo-corp-1.0.0', 'podsecuritypolicy_psp.yaml'),
       path.resolve(tmpDir, 'foo-corp-1.0.0', 'other.yaml'),
     );
-    const input = readIntermediate();
+    const input = await readIntermediate();
     functionConfig.data![SINK_DIR] = tmpDir;
     functionConfig.data![OVERWRITE] = 'true';
     const configs = new kpt.Configs(input.getAll(), functionConfig);
@@ -160,7 +160,7 @@ describe('buildSourcePath', () => {
   });
 });
 
-function readIntermediate(): kpt.Configs {
+function readIntermediate(): Promise<kpt.Configs> {
   return kpt.readConfigs(INTERMEDIATE_FILE, kpt.FileFormat.YAML);
 }
 

ts/demo-functions/src/write_yaml_test.ts

@@ -15,7 +15,7 @@
     "kpt:type-create": "kpt type-create"
   },
   "dependencies": {
-    "@googlecontainertools/kpt-functions": "^0.11.0"
+    "@googlecontainertools/kpt-functions": "^0.12.0"
   },
   "devDependencies": {
     "@googlecontainertools/create-kpt-functions": "^0.15.0",

ts/hello-world/package-lock.json

@@ -14,7 +14,7 @@
  * limitations under the License.
  */
 
-import { Configs, TestRunner } from '@googlecontainertools/kpt-functions';
+import { Configs, TestRunner, ConfigError } from '@googlecontainertools/kpt-functions';
 import { labelNamespace, LABEL_NAME, LABEL_VALUE } from './label_namespace';
 import { Namespace, ConfigMap } from './gen/io.k8s.api.core.v1';
 
@@ -32,7 +32,7 @@ describe('labelNamespace', () => {
 
   it('empty input ok', RUNNER.assertCallback(new Configs(undefined, functionConfig)));
 
-  it('requires functionConfig', RUNNER.assertCallback(undefined, undefined, TypeError));
+  it('requires functionConfig', RUNNER.assertCallback(undefined, undefined, ConfigError));
 
   it('adds label namespace when metadata.labels is undefined', async () => {
     const actual = new Configs(undefined, functionConfig);