[GitHub] [trafficcontrol] dpham692 commented on a change in pull request #6139: Fix roles hard code for TP integration test

[GitBox <gi...@apache.org>]

dpham692 commented on a change in pull request #6139:
URL: https://github.com/apache/trafficcontrol/pull/6139#discussion_r696866125



##########
File path: traffic_portal/test/integration/CommonUtils/API.ts
##########
@@ -259,9 +259,11 @@ export class API {
             return null;
         }
         for (const request of data.getRequest) {
-            const query = `?${encodeURIComponent(request.queryKey)}=${encodeURIComponent(request.queryValue)}${randomize}`;
+            let query = `?${encodeURIComponent(request.queryKey)}=${encodeURIComponent(request.queryValue)}${randomize}`;
+            if (`?${encodeURIComponent(request.queryKey)}` === 'admin' || `?${encodeURIComponent(request.queryKey)}` === 'operations' || `?${encodeURIComponent(request.queryKey)}` === 'read-only'){

Review comment:
       Yeah, I was like why the test failed? and I console.log it then I saw the typo




-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@trafficcontrol.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org