Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: headers are appended to existing one (open-telemetry#2335) #2357

Merged
merged 9 commits into from
Jul 21, 2021
Prev Previous commit
Next Next commit
fix: review lint
niko-achilles committed Jul 21, 2021
commit 9217290832f5d34466edb47b571a8e476ab3c708
Original file line number Diff line number Diff line change
@@ -14,14 +14,14 @@
* limitations under the License.
*/

import * as sinon from "sinon";
import { sendWithXhr } from "../../src/platform/browser/util";
import { ensureHeadersContain } from "../helper";
import * as sinon from 'sinon';
import { sendWithXhr } from '../../src/platform/browser/util';
import { ensureHeadersContain } from '../helper';

describe('util - browser', () => {
let server: any;
const body = "";
const url = "";
const body = '';
const url = '';

let onSuccessStub: sinon.SinonStub;
let onErrorStub: sinon.SinonStub;
@@ -96,7 +96,7 @@ describe('util - browser', () => {
describe('and custom headers are set', () => {
let customHeaders: Record<string,string>;
beforeEach(()=>{
customHeaders = { aHeader: "aValue", bHeader: "bValue" };
customHeaders = { aHeader: 'aValue', bHeader: 'bValue' };
sendWithXhr(body, url, customHeaders, onSuccessStub, onErrorStub);
});
it('Request Headers should contain "Content-Type" header', done => {