Merge pull request #115 from splitio/fix_typos

Fix typos

Author: EmilianoSanchez

src/__tests__/browserSuites/events.spec.js

@@ -69,7 +69,7 @@ export function withoutBindingTT(fetchMock, assert) {
   assert.ok(client.track, 'client.track should be defined.');
   assert.equal(typeof client.track, 'function', 'client.track should be a function.');
 
-  // Key binded as with getTreatment.
+  // Key bound as with getTreatment.
   tsStart = Date.now();
   assert.ok(client.track('sometraffictype', 'someEvent', 10), 'client.track returns true if an event is added to the queue.');
   assert.ok(client.track('othertraffictype', 'genericEvent', 25), 'client.track returns true if event value is null and is added to the queue.');
@@ -96,7 +96,7 @@ export function withoutBindingTT(fetchMock, assert) {
 // Not for JS Browser SDK, because it doesn't let bind traffic types to clients
 export function bindingTT(fetchMock, assert) {
   const localSettings = Object.assign({}, baseSettings);
-  localSettings.core.trafficType = 'binded_tt';
+  localSettings.core.trafficType = 'bound_tt';
   const splitio = SplitFactory(localSettings);
   const client = splitio.client();
 
@@ -113,20 +113,20 @@ export function bindingTT(fetchMock, assert) {
 
     assert.equal(firstEvent.key, 'facundo@split.io', 'Key should match received value.');
     assert.equal(firstEvent.eventTypeId, 'someEvent', 'EventTypeId should match received value.');
-    assert.equal(firstEvent.trafficTypeName, 'binded_tt', 'TrafficTypeName should match the binded value.');
+    assert.equal(firstEvent.trafficTypeName, 'bound_tt', 'TrafficTypeName should match the bound value.');
     assert.equal(firstEvent.value, 10, 'Value should match the value received on the .track() function.');
     assert.equal(typeof firstEvent.timestamp, 'number', 'The timestamp should be a number.');
 
     assert.equal(midEvent.key, 'facundo@split.io', 'Key should match received value.');
     assert.equal(midEvent.eventTypeId, 'genericEvent', 'EventTypeId should match received value.');
-    assert.equal(midEvent.trafficTypeName, 'binded_tt', 'TrafficTypeName should match received value in lowercases.');
+    assert.equal(midEvent.trafficTypeName, 'bound_tt', 'TrafficTypeName should match received value in lowercases.');
     assert.equal(midEvent.value, 24, 'Value should match the value received on the .track() function.');
     assert.true(midEvent.timestamp >= tsStart && midEvent.timestamp <= tsEnd, 'The timestamp should be a number with the right value.');
     assert.deepEqual(midEvent.properties, { prop1: true, prop2: 'a', prop3: 2, prop4: null, willBeNulled: null }, 'The properties should be correct.');
 
     assert.equal(lastEvent.key, 'facundo@split.io', 'Key should match received value.');
     assert.equal(lastEvent.eventTypeId, 'my.checkout.event', 'EventTypeId should match received value.');
-    assert.equal(lastEvent.trafficTypeName, 'binded_tt', 'TrafficTypeName should match the binded value.');
+    assert.equal(lastEvent.trafficTypeName, 'bound_tt', 'TrafficTypeName should match the bound value.');
     assert.equal(lastEvent.value, null, 'Should have null as value.');
     assert.equal(typeof lastEvent.timestamp, 'number', 'The timestamp should be a number.');
 
@@ -139,7 +139,7 @@ export function bindingTT(fetchMock, assert) {
   assert.ok(client.track, 'client.track should be defined.');
   assert.equal(typeof client.track, 'function', 'client.track should be a function.');
 
-  // Key binded as with getTreatment.
+  // Key bound as with getTreatment.
   tsStart = Date.now();
   assert.ok(client.track('someEvent', 10), 'client.track returns true if an event is added to the queue.');
   assert.ok(client.track('genericEvent', 25), 'client.track returns true if an event is added to the queue');

src/__tests__/browserSuites/push-initialization-retries.spec.js

@@ -61,7 +61,7 @@ export function testPushRetriesDueToAuthErrors(fetchMock, assert) {
     if (!opts.headers['Authorization']) assert.fail('`/v2/auth` request must include `Authorization` header');
     const lapse = Date.now() - start;
     const expected = (settings.scheduler.pushRetryBackoffBase * Math.pow(2, 0) + settings.scheduler.pushRetryBackoffBase * Math.pow(2, 1));
-    assert.true(nearlyEqual(lapse, expected), 'third auth attempt (aproximately in 0.3 seconds from first attempt)');
+    assert.true(nearlyEqual(lapse, expected), 'third auth attempt (approximately in 0.3 seconds from first attempt)');
     return { status: 200, body: authPushDisabled };
   });
   fetchMock.get({ url: url(settings, '/mySegments/nicolas%40split.io'), repeat: 4 }, { status: 200, body: mySegmentsNicolasMock });
@@ -124,7 +124,7 @@ export function testPushRetriesDueToSseErrors(fetchMock, assert) {
     } else {
       const lapse = Date.now() - start;
 
-      assert.true(nearlyEqual(lapse, expectedTimeToSSEsuccess), 'third auth attempt (aproximately in 0.3 seconds from first attempt)');
+      assert.true(nearlyEqual(lapse, expectedTimeToSSEsuccess), 'third auth attempt (approximately in 0.3 seconds from first attempt)');
       eventSourceInstance.emitOpen();
     }
     sseattempts++;

src/__tests__/browserSuites/shared-instantiation.spec.js

@@ -107,9 +107,9 @@ export default function sharedInstantiationSuite(startWithTT, sdkIgnoresTT, fetc
       const events = JSON.parse(opts.body);
 
       assert.equal(events.length, sdkIgnoresTT ? 2 : 3, 'Tracked only valid events');
-      assert.equal(events[0].trafficTypeName, `${startWithTT && !sdkIgnoresTT ? 'start' : 'main'}_tt`, 'matching traffic types both binded and provided through client.track()');
-      assert.equal(events[1].trafficTypeName, 'marcio_tt', 'matching traffic types both binded and provided through client.track()');
-      if (!sdkIgnoresTT) assert.equal(events[2].trafficTypeName, 'nico_tt', 'matching traffic types both binded and provided through client.track()');
+      assert.equal(events[0].trafficTypeName, `${startWithTT && !sdkIgnoresTT ? 'start' : 'main'}_tt`, 'matching traffic type provided through client.track()');
+      assert.equal(events[1].trafficTypeName, 'marcio_tt', 'matching traffic type provided through client.track()');
+      if (!sdkIgnoresTT) assert.equal(events[2].trafficTypeName, 'nico_tt', 'matching traffic type provided through client.track()');
 
       finished.next();
 
@@ -124,14 +124,14 @@ export default function sharedInstantiationSuite(startWithTT, sdkIgnoresTT, fetc
     }
 
     // Shared instance without TT on instantiation
-    assert.false(marcioClient.track('marcioEvent'), 'If a shared client was created passing only key, no traffic type is binded so we need to provide one for client.track()');
-    assert.true(marcioClient.track('marcio_tt', 'marcioEvent', 10), 'If a shared client was created passing only key, no traffic type is binded so we need to provide one for client.track()');
+    assert.false(marcioClient.track('marcioEvent'), 'If a shared client was created passing only key, no traffic type is bound so we need to provide one for client.track()');
+    assert.true(marcioClient.track('marcio_tt', 'marcioEvent', 10), 'If a shared client was created passing only key, no traffic type is bound so we need to provide one for client.track()');
 
     // Shared instance with TT on instantiation
     if (sdkIgnoresTT) {
-      assert.false(nicolasClient.track('nicoEvent', 10), 'If a shared client was created passing both key and TT but the SDK ignores TT, the latter doesn\'t get binded to it so it is necessary to provide the traffic type to client.track()');
+      assert.false(nicolasClient.track('nicoEvent', 10), 'If a shared client was created passing both key and TT but the SDK ignores TT, the latter doesn\'t get bound to it so it is necessary to provide the traffic type to client.track()');
     } else {
-      assert.true(nicolasClient.track('nicoEvent', 10), 'If a shared client was created passing both key and TT, the latter gets binded to it so it is not necessary to provide the traffic type to client.track()');
+      assert.true(nicolasClient.track('nicoEvent', 10), 'If a shared client was created passing both key and TT, the latter gets bound to it so it is not necessary to provide the traffic type to client.track()');
     }
   };
 

src/__tests__/browserSuites/use-beacon-api.debug.spec.js

@@ -37,7 +37,7 @@ const assertImpressionSent = (assert, impression) => {
 const assertEventSent = (assert, event) => {
   assert.equal(event.key, 'facundo@split.io', 'Key should match received value.');
   assert.equal(event.eventTypeId, 'someEvent', 'EventTypeId should match received value.');
-  assert.equal(event.trafficTypeName, 'sometraffictype', 'TrafficTypeName should match the binded value.');
+  assert.equal(event.trafficTypeName, 'sometraffictype', 'TrafficTypeName should match the bound value.');
 };
 
 const assertCallsToBeaconAPI = (assert) => {

src/__tests__/browserSuites/use-beacon-api.spec.js

@@ -34,7 +34,7 @@ const assertImpressionSent = (assert, impression) => {
 const assertEventSent = (assert, event) => {
   assert.equal(event.key, 'facundo@split.io', 'Key should match received value.');
   assert.equal(event.eventTypeId, 'someEvent', 'EventTypeId should match received value.');
-  assert.equal(event.trafficTypeName, 'sometraffictype', 'TrafficTypeName should match the binded value.');
+  assert.equal(event.trafficTypeName, 'sometraffictype', 'TrafficTypeName should match the bound value.');
 };
 
 const assertImpressionsCountSent = (assert, impressionCount) => {

src/__tests__/online/browser.spec.js

@@ -108,7 +108,7 @@ tape('## E2E CI Tests ##', function (assert) {
   /* Check events */
   assert.test('E2E / Events', withoutBindingTT.bind(null, fetchMock));
   // Next test only applies for the isomorphic JS SDK, that support clients with bound TT
-  // assert.test('E2E / Events with TT binded', bindingTT.bind(null, fetchMock));
+  // assert.test('E2E / Events with TT bound', bindingTT.bind(null, fetchMock));
   /* Check shared clients */
   assert.test('E2E / Shared instances', sharedInstantiationSuite.bind(null, false, true, fetchMock));
   // Next test only applies for the isomorphic JS SDK, that support clients with bound TT