diff --git a/test/samlp.signedresponse.tests.js b/test/samlp.signedresponse.tests.js index eab58cb..73862e5 100644 --- a/test/samlp.signedresponse.tests.js +++ b/test/samlp.signedresponse.tests.js @@ -3,6 +3,7 @@ var server = require('./fixture/server'); var request = require('request'); var cheerio = require('cheerio'); var xmlhelper = require('./xmlhelper'); +var xmldom = require('xmldom'); describe('samlp signed response', function () { before(function (done) { @@ -35,7 +36,6 @@ describe('samlp signed response', function () { }); it('should contain a valid signed response', function(){ - console.log(signedResponse); var isValid = xmlhelper.verifySignature( signedResponse, server.credentials.cert); @@ -57,5 +57,12 @@ describe('samlp signed response', function () { expect(destination).to.equal('http://destination'); }); + it('should have signature after issuer', function(){ + var doc = new xmldom.DOMParser().parseFromString(signedResponse); + + var signature = doc.documentElement.getElementsByTagName('Signature'); + + expect(signature[0].previousSibling.nodeName).to.equal('saml:Issuer'); + }); }); -}); \ No newline at end of file +}); diff --git a/test/samlp.tests.js b/test/samlp.tests.js index a287d6d..37c6f90 100644 --- a/test/samlp.tests.js +++ b/test/samlp.tests.js @@ -58,7 +58,7 @@ describe('samlp', function () { var signature = doc.documentElement.getElementsByTagName('Signature'); - expect('saml:Issuer', signature[0].previousSibling.nodeName); + expect(signature[0].previousSibling.nodeName).to.equal('saml:Issuer'); }); it('should use sha256 as default signature algorithm', function(){