Merge pull request #154 from screwdriver-cd/fix

fix: list need an extra layer of params
screwdriver-cd · Mar 24, 2017 · a4d9a28 · a4d9a28
2 parents 132b9e7 + a03e3ea
commit a4d9a28
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/lib/templateFactory.js b/lib/templateFactory.js
@@ -90,7 +90,7 @@ class TemplateFactory extends BaseFactory {
      */
     getTemplate(config) {
         // get all templates with the same name
-        return super.list({ name: config.name })
+        return super.list({ params: { name: config.name } })
             .then((templates) => {
                 // get templates that have version prefix as config.version
                 const filtered = templates.filter(template =>

diff --git a/test/lib/templateFactory.test.js b/test/lib/templateFactory.test.js
@@ -231,6 +231,7 @@ describe('Template Factory', () => {
             datastore.scan.resolves(returnValue);
 
             return factory.getTemplate(config).then((model) => {
+                assert.calledWithMatch(datastore.scan, { params: { name: config.name } });
                 assert.instanceOf(model, Template);
                 Object.keys(expected).forEach((key) => {
                     assert.strictEqual(model[key], expected[key]);