Browse Source

style(js): Minor formatting cleanup in jest titles (#33112)

Evan Purkhiser 2 years ago
parent
commit
531c11e072

+ 1 - 1
tests/js/spec/components/search/sources/formSource.spec.jsx

@@ -69,7 +69,7 @@ describe('FormSource', function () {
     );
   });
 
-  it('does not find any form field ', async function () {
+  it('does not find any form field', async function () {
     const mock = jest.fn().mockReturnValue(null);
     wrapper = mountWithTheme(<FormSource query="invalid">{mock}</FormSource>);
 

+ 1 - 1
tests/js/spec/components/search/sources/routeSource.spec.jsx

@@ -29,7 +29,7 @@ describe('RouteSource', function () {
     });
   });
 
-  it('does not find any form field ', async function () {
+  it('does not find any form field', async function () {
     const mock = jest.fn().mockReturnValue(null);
     const {organization, project} = initializeOrg();
     wrapper = mountWithTheme(

+ 3 - 3
tests/js/spec/utils/discover/eventView.spec.jsx

@@ -3386,7 +3386,7 @@ describe('isAPIPayloadSimilar', function () {
       expect(results).toBe(false);
     });
 
-    it('it is similar if column order changes', function () {
+    it('is similar if column order changes', function () {
       const thisEventView = new EventView(state);
       const location = {};
       const thisAPIPayload = thisEventView.getEventsAPIPayload(location);
@@ -3401,7 +3401,7 @@ describe('isAPIPayloadSimilar', function () {
       expect(results).toBe(true);
     });
 
-    it('it is similar if equation order relatively same', function () {
+    it('is similar if equation order relatively same', function () {
       const equationField = {field: 'equation|failure_count() / count()'};
       const otherEquationField = {field: 'equation|failure_count() / 2'};
       state.fields = [
@@ -3429,7 +3429,7 @@ describe('isAPIPayloadSimilar', function () {
       expect(results).toBe(true);
     });
 
-    it('it is not similar if equation order changes', function () {
+    it('is not similar if equation order changes', function () {
       const equationField = {field: 'equation|failure_count() / count()'};
       const otherEquationField = {field: 'equation|failure_count() / 2'};
       state.fields = [

+ 1 - 1
tests/js/spec/utils/useApi.spec.tsx

@@ -4,7 +4,7 @@ import {Client} from 'sentry/api';
 import useApi from 'sentry/utils/useApi';
 
 describe('useApi', function () {
-  it('provides an api client ', function () {
+  it('provides an api client', function () {
     const {result} = reactHooks.renderHook(() => useApi());
 
     expect(result.current).toBeInstanceOf(Client);

+ 1 - 1
tests/js/spec/views/dashboardsV2/widgetBuilder/widgetBuilder.spec.tsx

@@ -753,7 +753,7 @@ describe('WidgetBuilder', function () {
     expect(handleSave).toHaveBeenCalledTimes(1);
   });
 
-  it('should properly query for table fields ', async function () {
+  it('should properly query for table fields', async function () {
     const defaultWidgetQuery = {
       name: '',
       fields: ['title', 'count()'],

+ 1 - 1
tests/js/spec/views/organizationIntegrations/sentryAppDetailedView.spec.jsx

@@ -133,7 +133,7 @@ describe('SentryAppDetailedView', function () {
       expect(wrapper.find('InstallButton').props().children).toEqual('Accept & Install');
     });
 
-    describe('onClick: ', function () {
+    describe('onClick:', function () {
       let wrapperState;
 
       it('installs app', async function () {

+ 1 - 1
tests/js/spec/views/performance/landing/queryBatcher.spec.tsx

@@ -199,7 +199,7 @@ describe('Performance > Widgets > Query Batching', function () {
     );
   });
 
-  it('Multiple EventsRequest based component merge queries with provider ', async function () {
+  it('Multiple EventsRequest based component merge queries with provider', async function () {
     const data = initializeData();
 
     render(

+ 1 - 1
tests/js/spec/views/projectInstall/issueAlertOptions.spec.jsx

@@ -81,7 +81,7 @@ describe('IssueAlertOptions', function () {
     expect(wrapper.find('RadioLineItem')).toHaveLength(3);
   });
 
-  it('should ignore conditions that are not `sentry.rules.conditions.event_frequency.EventFrequencyCondition` and `sentry.rules.conditions.event_frequency.EventUniqueUserFrequencyCondition` ', () => {
+  it('should ignore conditions that are not `sentry.rules.conditions.event_frequency.EventFrequencyCondition` and `sentry.rules.conditions.event_frequency.EventUniqueUserFrequencyCondition`', () => {
     MockApiClient.addMockResponse({
       url: URL,
       body: TestStubs.MOCK_RESP_ONLY_IGNORED_CONDITIONS_INVALID,

+ 3 - 3
tests/js/spec/views/settings/organizationDeveloperSettings/sentryApplicationDetails.spec.jsx

@@ -45,7 +45,7 @@ describe('Sentry Application Details', function () {
       expect(wrapper.exists(redirectUrlInput)).toBeTruthy();
     });
 
-    it('it shows empty scopes and no credentials', function () {
+    it('shows empty scopes and no credentials', function () {
       // new app starts off with no scopes selected
       expect(wrapper.find('PermissionsObserver').prop('scopes')).toEqual([]);
       expect(
@@ -172,7 +172,7 @@ describe('Sentry Application Details', function () {
       expect(wrapper.exists(redirectUrlInput)).toBeTruthy();
     });
 
-    it('it shows application data', function () {
+    it('shows application data', function () {
       // data should be filled out
       expect(wrapper.find('PermissionsObserver').prop('scopes')).toEqual([
         'project:read',
@@ -360,7 +360,7 @@ describe('Sentry Application Details', function () {
       );
     });
 
-    it('it updates app with correct data', function () {
+    it('updates app with correct data', function () {
       wrapper
         .find(redirectUrlInput)
         .simulate('change', {target: {value: 'https://hello.com/'}});