From 8cf06ad2fffb5143ef031214e089dd0be1c061b5 Mon Sep 17 00:00:00 2001
From: Charles Wahome <thewahome.cw@gmail.com>
Date: Tue, 28 Feb 2023 12:26:53 +0300
Subject: [PATCH 1/3] check whether a query should run based on exceptions

---
 .../query-runner/query-input/QueryInput.tsx   |  6 +++-
 .../sidebar/sample-queries/SampleQueries.tsx  |  8 +++---
 .../sample-queries/sample-query-utils.ts      | 28 ++++++++++++++++---
 3 files changed, 33 insertions(+), 9 deletions(-)

diff --git a/src/app/views/query-runner/query-input/QueryInput.tsx b/src/app/views/query-runner/query-input/QueryInput.tsx
index 2a46fe861..f98ce01cb 100644
--- a/src/app/views/query-runner/query-input/QueryInput.tsx
+++ b/src/app/views/query-runner/query-input/QueryInput.tsx
@@ -10,6 +10,7 @@ import { getStyleFor } from '../../../utils/http-methods.utils';
 import { parseSampleUrl } from '../../../utils/sample-url-generation';
 import { translateMessage } from '../../../utils/translate-messages';
 import SubmitButton from '../../../views/common/submit-button/SubmitButton';
+import { shouldRunQuery } from '../../sidebar/sample-queries/sample-query-utils';
 import { queryRunnerStyles } from '../QueryRunner.styles';
 import { AutoComplete } from './auto-complete';
 import { ShareQuery } from './share-query';
@@ -36,7 +37,10 @@ const QueryInput = (props: IQueryInputProps) => {
   const authenticated = !!authToken.token;
   const { mobileScreen } = sidebarProperties;
 
-  const showError = !authenticated && sampleQuery.selectedVerb !== 'GET';
+  const showError = shouldRunQuery({
+    method: sampleQuery.selectedVerb, authenticated,
+    url: sampleQuery.sampleUrl
+  });
   const { queryButtonStyles, verbSelector, shareQueryButtonStyles } = queryRunnerStyles();
   verbSelector.title = {
     ...verbSelector.title,
diff --git a/src/app/views/sidebar/sample-queries/SampleQueries.tsx b/src/app/views/sidebar/sample-queries/SampleQueries.tsx
index 0de028dd1..f53fb1b00 100644
--- a/src/app/views/sidebar/sample-queries/SampleQueries.tsx
+++ b/src/app/views/sidebar/sample-queries/SampleQueries.tsx
@@ -25,7 +25,7 @@ import { classNames } from '../../classnames';
 import { NoResultsFound } from '../sidebar-utils/SearchResult';
 import { sidebarStyles } from '../Sidebar.styles';
 import {
-  isJsonString, performSearch, trackDocumentLinkClickedEvent,
+  isJsonString, performSearch, shouldRunQuery, trackDocumentLinkClickedEvent,
   trackSampleQueryClickEvent
 } from './sample-query-utils';
 
@@ -167,7 +167,7 @@ const UnstyledSampleQueries = (sampleProps?: ISampleQueriesProps): JSX.Element =
       onRender: (item: ISampleQuery) => {
         const signInText = translateMessage('Sign In to try this sample');
 
-        if (item.method === 'GET' || tokenPresent) {
+        if (!shouldRunQuery({ method: item.method, authenticated: tokenPresent, url: item.requestUrl })) {
           return <div aria-hidden='true' />;
         }
 
@@ -270,7 +270,8 @@ const UnstyledSampleQueries = (sampleProps?: ISampleQueriesProps): JSX.Element =
     }
 
     if (props) {
-      if (!tokenPresent && props.item.method !== 'GET') {
+      const query: ISampleQuery = props.item!;
+      if (shouldRunQuery({ method: query.method, authenticated: tokenPresent, url: query.requestUrl })) {
         selectionDisabled = true;
       }
       return (
@@ -280,7 +281,6 @@ const UnstyledSampleQueries = (sampleProps?: ISampleQueriesProps): JSX.Element =
             styles={customStyles}
             onClick={() => {
               if (!selectionDisabled) {
-                const query: ISampleQuery = props.item!;
                 querySelected(query);
               }
               setSelectedQuery(props.item)
diff --git a/src/app/views/sidebar/sample-queries/sample-query-utils.ts b/src/app/views/sidebar/sample-queries/sample-query-utils.ts
index f3fc31fde..58e24ac53 100644
--- a/src/app/views/sidebar/sample-queries/sample-query-utils.ts
+++ b/src/app/views/sidebar/sample-queries/sample-query-utils.ts
@@ -4,7 +4,7 @@ import { GRAPH_URL } from '../../../services/graph-constants';
 import { validateExternalLink } from '../../../utils/external-link-validation';
 import { sanitizeQueryUrl } from '../../../utils/query-url-sanitization';
 
-export function isJsonString(str: string): boolean {
+function isJsonString(str: string): boolean {
   try {
     JSON.parse(str);
     return true;
@@ -13,7 +13,7 @@ export function isJsonString(str: string): boolean {
   }
 }
 
-export function performSearch(queries: ISampleQuery[], value: string): ISampleQuery[] {
+function performSearch(queries: ISampleQuery[], value: string): ISampleQuery[] {
   const keyword = value.toLowerCase();
   return queries.filter((sample: any) => {
     const name = sample.humanName.toLowerCase();
@@ -22,7 +22,7 @@ export function performSearch(queries: ISampleQuery[], value: string): ISampleQu
   });
 }
 
-export const trackSampleQueryClickEvent = (query: ISampleQuery) => {
+const trackSampleQueryClickEvent = (query: ISampleQuery) => {
   const sanitizedUrl = sanitizeQueryUrl(GRAPH_URL + query.requestUrl);
   telemetry.trackEvent(
     eventTypes.LISTITEM_CLICK_EVENT,
@@ -35,7 +35,7 @@ export const trackSampleQueryClickEvent = (query: ISampleQuery) => {
     });
 }
 
-export const trackDocumentLinkClickedEvent = async (item: ISampleQuery): Promise<void> => {
+const trackDocumentLinkClickedEvent = async (item: ISampleQuery): Promise<void> => {
   const properties: { [key: string]: any } = {
     ComponentName: componentNames.DOCUMENTATION_LINK,
     SampleId: item.id,
@@ -48,3 +48,23 @@ export const trackDocumentLinkClickedEvent = async (item: ISampleQuery): Promise
   // Check if link throws error
   validateExternalLink(item.docLink || '', componentNames.DOCUMENTATION_LINK, item.id);
 }
+
+interface ShouldRunQueryArgs {
+  method: string;
+  url: string;
+  authenticated: boolean;
+}
+
+const shouldRunQuery = ({ method, url, authenticated }: ShouldRunQueryArgs): boolean => {
+  const isPOSTException = method === 'POST' && url.contains('search/query');
+  const shouldRun = method === 'GET' || authenticated || isPOSTException;
+  return !shouldRun;
+}
+
+export {
+  trackDocumentLinkClickedEvent,
+  trackSampleQueryClickEvent,
+  performSearch,
+  isJsonString,
+  shouldRunQuery
+}

From edd57b8da165b05fe4d5b7de658ccd7db7904e40 Mon Sep 17 00:00:00 2001
From: Charles Wahome <thewahome.cw@gmail.com>
Date: Thu, 2 Mar 2023 21:46:12 +0300
Subject: [PATCH 2/3] make shouldRunQuery affirmative

---
 src/app/views/query-runner/query-input/QueryInput.tsx      | 2 +-
 src/app/views/sidebar/sample-queries/SampleQueries.tsx     | 4 ++--
 src/app/views/sidebar/sample-queries/sample-query-utils.ts | 2 +-
 3 files changed, 4 insertions(+), 4 deletions(-)

diff --git a/src/app/views/query-runner/query-input/QueryInput.tsx b/src/app/views/query-runner/query-input/QueryInput.tsx
index f98ce01cb..556b791bb 100644
--- a/src/app/views/query-runner/query-input/QueryInput.tsx
+++ b/src/app/views/query-runner/query-input/QueryInput.tsx
@@ -37,7 +37,7 @@ const QueryInput = (props: IQueryInputProps) => {
   const authenticated = !!authToken.token;
   const { mobileScreen } = sidebarProperties;
 
-  const showError = shouldRunQuery({
+  const showError = !shouldRunQuery({
     method: sampleQuery.selectedVerb, authenticated,
     url: sampleQuery.sampleUrl
   });
diff --git a/src/app/views/sidebar/sample-queries/SampleQueries.tsx b/src/app/views/sidebar/sample-queries/SampleQueries.tsx
index f53fb1b00..e14d5eeba 100644
--- a/src/app/views/sidebar/sample-queries/SampleQueries.tsx
+++ b/src/app/views/sidebar/sample-queries/SampleQueries.tsx
@@ -167,7 +167,7 @@ const UnstyledSampleQueries = (sampleProps?: ISampleQueriesProps): JSX.Element =
       onRender: (item: ISampleQuery) => {
         const signInText = translateMessage('Sign In to try this sample');
 
-        if (!shouldRunQuery({ method: item.method, authenticated: tokenPresent, url: item.requestUrl })) {
+        if (shouldRunQuery({ method: item.method, authenticated: tokenPresent, url: item.requestUrl })) {
           return <div aria-hidden='true' />;
         }
 
@@ -271,7 +271,7 @@ const UnstyledSampleQueries = (sampleProps?: ISampleQueriesProps): JSX.Element =
 
     if (props) {
       const query: ISampleQuery = props.item!;
-      if (shouldRunQuery({ method: query.method, authenticated: tokenPresent, url: query.requestUrl })) {
+      if (!shouldRunQuery({ method: query.method, authenticated: tokenPresent, url: query.requestUrl })) {
         selectionDisabled = true;
       }
       return (
diff --git a/src/app/views/sidebar/sample-queries/sample-query-utils.ts b/src/app/views/sidebar/sample-queries/sample-query-utils.ts
index 58e24ac53..f1c45c2fe 100644
--- a/src/app/views/sidebar/sample-queries/sample-query-utils.ts
+++ b/src/app/views/sidebar/sample-queries/sample-query-utils.ts
@@ -58,7 +58,7 @@ interface ShouldRunQueryArgs {
 const shouldRunQuery = ({ method, url, authenticated }: ShouldRunQueryArgs): boolean => {
   const isPOSTException = method === 'POST' && url.contains('search/query');
   const shouldRun = method === 'GET' || authenticated || isPOSTException;
-  return !shouldRun;
+  return shouldRun;
 }
 
 export {

From cee7633e5cd25b50f47a738b27b5b686bc5fc91f Mon Sep 17 00:00:00 2001
From: Onokaev <evansonoka05.j@gmail.com>
Date: Mon, 31 Jul 2023 11:11:32 +0300
Subject: [PATCH 3/3] Add request body

---
 src/app/services/actions/query-action-creator-util.ts | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/app/services/actions/query-action-creator-util.ts b/src/app/services/actions/query-action-creator-util.ts
index e87f17890..807d1dcd5 100644
--- a/src/app/services/actions/query-action-creator-util.ts
+++ b/src/app/services/actions/query-action-creator-util.ts
@@ -70,7 +70,8 @@ export function createAnonymousRequest(query: IQuery, proxyUrl: string, queryRun
 
   const options: IRequestOptions = {
     method: query.selectedVerb,
-    headers
+    headers,
+    body: query.sampleBody ? JSON.stringify(query.sampleBody) : undefined
   };
   return { graphUrl, options };
 }