employer
/
liza
1
0
Fork 0
Code Releases Activity

XhrHttpImpl#request invokes XHR send 

This should have always been the case; I am not sure what I was thinking,
considering that the object is encapsulated.
master
Mike Gerwitz 2015-05-28 09:03:39 -04:00
parent 76280b0cac
commit 1669f9eeda
2 changed files with 4 additions and 15 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/dapi/http/XhrHttpImpl.js
View File

@ -1,7 +1,7 @@
/** /**
* XMLHttpRequest HTTP protocol implementation * XMLHttpRequest HTTP protocol implementation
* *
* Copyright (C) 2015 LoVullo Associates, Inc. * Copyright (C) 2014, 2015 LoVullo Associates, Inc.
* *
* This file is part of the Liza Data Collection Framework * This file is part of the Liza Data Collection Framework
* *
@ -69,6 +69,8 @@ module.exports = Class( 'XhrHttpImpl' )
{ {
callback( err, resp ); callback( err, resp );
} ); } );
req.send( data );
} }
catch ( e ) catch ( e )
{ {

15
test/dapi/http/XhrHttpImplTest.js
View File

@ -1,7 +1,7 @@
/** /**
* Test case for XMLHttpRequest HTTP protocol implementation * Test case for XMLHttpRequest HTTP protocol implementation
* *
* Copyright (C) 2014 LoVullo Associates, Inc. * Copyright (C) 2014, 2015 LoVullo Associates, Inc.
* *
* This file is part of the Liza Data Collection Framework * This file is part of the Liza Data Collection Framework
* *
@ -127,9 +127,6 @@ describe( 'XhrHttpImpl', function()
expect( resp ).to.equal( retdata ); expect( resp ).to.equal( retdata );
done(); done();
} ); } );
// invoke callback
StubXhr.inst.send( src );
} ); } );
@ -154,8 +151,6 @@ describe( 'XhrHttpImpl', function()
done(); done();
} ); } );
StubXhr.inst.send( '' );
} ); } );
@ -175,8 +170,6 @@ describe( 'XhrHttpImpl', function()
expect( resp.data ).to.equal( reply ); expect( resp.data ).to.equal( reply );
done(); done();
} ); } );
StubXhr.inst.send( '' );
} ); } );
} ); } );
@ -192,8 +185,6 @@ describe( 'XhrHttpImpl', function()
expect( err ).to.equal( null ); expect( err ).to.equal( null );
done(); done();
} ); } );
StubXhr.inst.send( '' );
} ); } );
@ -217,8 +208,6 @@ describe( 'XhrHttpImpl', function()
expect( err ).to.equal( null ); expect( err ).to.equal( null );
done(); done();
} ); } );
StubXhr.inst.send( '' );
} ); } );
@ -249,8 +238,6 @@ describe( 'XhrHttpImpl', function()
done(); done();
} ); } );
StubXhr.inst.send( '' );
} ); } );