[#1082] Fix for jQuery.contains as params should be dom elements and …

…not jQuery collections
ckan · Jul 4, 2013 · f9e43bb · f9e43bb
1 parent 05096e9
commit f9e43bb
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/ckan/public/base/test/spec/modules/custom-fields.spec.js b/ckan/public/base/test/spec/modules/custom-fields.spec.js
@@ -66,7 +66,7 @@ describe('ckan.module.CustomFieldsModule()', function () {
 
       this.module.newField();
 
-      assert.ok(jQuery.contains(this.module.el, element));
+      assert.ok(jQuery.contains(this.module.el[0], element));
     });
   });
 

diff --git a/ckan/public/base/test/spec/modules/resource-upload-field.spec.js b/ckan/public/base/test/spec/modules/resource-upload-field.spec.js
@@ -30,7 +30,7 @@ describe('ckan.modules.ResourceUploadFieldModule()', function () {
     it('should append the upload field to the module element', function () {
       this.module.initialize();
 
-      assert.ok(jQuery.contains(this.el, this.module.upload));
+      assert.ok(jQuery.contains(this.el[0], this.module.upload[0]));
     });
 
     it('should call .setupFileUpload()', function () {