From 17f4954646ea918a2ae2bd378c0c29c6a2a18ab2 Mon Sep 17 00:00:00 2001
From: Jerry Zhang <thejerryzhang@gmail.com>
Date: Mon, 17 Apr 2017 18:10:53 -0700
Subject: [PATCH] fix: Fix tests + comments after changing scmUri to pipelineId
 in templates

---
 README.md                        |  2 +-
 lib/templateFactory.js           |  8 ++++----
 test/lib/template.test.js        |  2 +-
 test/lib/templateFactory.test.js | 14 +++++++-------
 4 files changed, 13 insertions(+), 13 deletions(-)

diff --git a/README.md b/README.md
index e2277fe5..b154e502 100644
--- a/README.md
+++ b/README.md
@@ -744,7 +744,7 @@ factory.create(config).then(model => {
 | config.description | String | Yes | Description of the template |
 | config.maintainer | Array | Yes | Maintainer's email |
 | config.config | Object | Yes | Config of the screwdriver-template.yaml |
-| config.scmUri | String | Yes | ScmUri of the template pipeline |
+| config.pipelineId | String | Yes | pipelineId of the template |
 | config.labels | Array | No | Labels attached to the template |
 
 #### Get
diff --git a/lib/templateFactory.js b/lib/templateFactory.js
index 173ddb6e..3ac779cb 100644
--- a/lib/templateFactory.js
+++ b/lib/templateFactory.js
@@ -32,8 +32,8 @@ class TemplateFactory extends BaseFactory {
      * @param  {String}     config.description   Description of the template
      * @param  {String}     config.maintainer    Maintainer's email
      * @param  {Object}     config.config        Config of the screwdriver-template.yaml
-     * @param  {String}     config.scmUri        ScmUri of the template pipeline
-     * @param  {Array}      [config.labels]        Labels attached to the template
+     * @param  {String}     config.pipelineId    pipelineId of the template
+     * @param  {Array}      [config.labels]      Labels attached to the template
      * @return {Job}
      */
     createClass(config) {
@@ -50,8 +50,8 @@ class TemplateFactory extends BaseFactory {
      * @param  {String}     config.description   Description of the template
      * @param  {String}     config.maintainer    Maintainer's email
      * @param  {Object}     config.config        Config of the screwdriver-template.yaml
-     * @param  {String}     config.scmUri        ScmUri of the template pipeline
-     * @param  {Array}      [config.labels]        Labels attached to the template
+     * @param  {String}     config.pipelineId    pipelineId of the template
+     * @param  {Array}      [config.labels]      Labels attached to the template
      * @return {Promise}
      */
     create(config) {
diff --git a/test/lib/template.test.js b/test/lib/template.test.js
index c4ffc40d..263025ba 100644
--- a/test/lib/template.test.js
+++ b/test/lib/template.test.js
@@ -41,7 +41,7 @@ describe('Template Model', () => {
             description: 'this is a template',
             labels: ['test', 'beta'],
             config: { image: 'node:6' },
-            scmUri: 'github.com:123:master'
+            pipelineId: '123'
         };
         template = new TemplateModel(createConfig);
     });
diff --git a/test/lib/templateFactory.test.js b/test/lib/templateFactory.test.js
index 6c208b43..2dd4298a 100644
--- a/test/lib/templateFactory.test.js
+++ b/test/lib/templateFactory.test.js
@@ -13,7 +13,7 @@ describe('Template Factory', () => {
     const description = 'this is a template';
     const labels = ['test', 'beta'];
     const templateConfig = { image: 'node:6' };
-    const scmUri = 'github.com:123:master';
+    const pipelineId = '123';
     const metaData = {
         name,
         version,
@@ -21,7 +21,7 @@ describe('Template Factory', () => {
         description,
         labels,
         config: templateConfig,
-        scmUri
+        pipelineId
     };
     let TemplateFactory;
     let datastore;
@@ -80,7 +80,7 @@ describe('Template Factory', () => {
                 description,
                 labels,
                 config: templateConfig,
-                scmUri,
+                pipelineId,
                 id: generatedId
             };
         });
@@ -98,7 +98,7 @@ describe('Template Factory', () => {
                 description,
                 labels,
                 config: templateConfig,
-                scmUri
+                pipelineId
             }).then((model) => {
                 assert.instanceOf(model, Template);
                 Object.keys(expected).forEach((key) => {
@@ -120,7 +120,7 @@ describe('Template Factory', () => {
                 description,
                 labels,
                 config: templateConfig,
-                scmUri
+                pipelineId
             }).then((model) => {
                 assert.instanceOf(model, Template);
                 Object.keys(expected).forEach((key) => {
@@ -137,7 +137,7 @@ describe('Template Factory', () => {
                 description,
                 labels,
                 config: templateConfig,
-                scmUri,
+                pipelineId,
                 id: generatedId
             };
 
@@ -153,7 +153,7 @@ describe('Template Factory', () => {
                 description,
                 labels,
                 config: templateConfig,
-                scmUri
+                pipelineId
             }).then((model) => {
                 assert.instanceOf(model, Template);
                 Object.keys(expected).forEach((key) => {