Skip to content

Commit 74d7d08

Browse files
committed
Suggested fixes around readability
1 parent a3c9065 commit 74d7d08

File tree

2 files changed

+30
-18
lines changed

2 files changed

+30
-18
lines changed

src/lib/local.ts

Lines changed: 14 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -81,13 +81,22 @@ export async function ensureLocalBinarySetup(localIdentifier?: string): Promise<
8181
const localBinary = new Local();
8282
await killExistingBrowserStackLocalProcesses();
8383

84+
const requestBody: {
85+
key: string;
86+
username: string;
87+
localIdentifier?: string;
88+
} = {
89+
key: config.browserstackAccessKey,
90+
username: config.browserstackUsername
91+
};
92+
93+
if (localIdentifier) {
94+
requestBody.localIdentifier = localIdentifier;
95+
}
96+
8497
return await new Promise((resolve, reject) => {
8598
localBinary.start(
86-
{
87-
key: config.browserstackAccessKey,
88-
username: config.browserstackUsername,
89-
...(localIdentifier ? { localIdentifier } : {}),
90-
},
99+
requestBody,
91100
(error?: Error) => {
92101
if (error) {
93102
logger.error(

src/tools/accessiblity-utils/scanner.ts

Lines changed: 16 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,8 @@ export class AccessibilityScanner {
3333
// Check if any URL is local
3434
const hasLocal = urlList.some(isLocalURL);
3535
const localIdentifier = crypto.randomUUID();
36-
const localHosts = new Set(["127.0.0.1", "localhost"]);
36+
const LOCAL_IP = "127.0.0.1";
37+
const BS_LOCAL_DOMAIN = "bs-local.com";
3738

3839
if (hasLocal) {
3940
await ensureLocalBinarySetup(localIdentifier);
@@ -44,8 +45,8 @@ export class AccessibilityScanner {
4445
const transformedUrlList = urlList.map((url) => {
4546
try {
4647
const parsed = new URL(url);
47-
if (localHosts.has(parsed.hostname)) {
48-
parsed.hostname = "bs-local.com";
48+
if (parsed.hostname === LOCAL_IP) {
49+
parsed.hostname = BS_LOCAL_DOMAIN;
4950
return parsed.toString();
5051
}
5152
return url;
@@ -55,21 +56,23 @@ export class AccessibilityScanner {
5556
}
5657
});
5758

58-
const requestBody = {
59+
const baseRequestBody = {
5960
name,
6061
urlList: transformedUrlList,
6162
recurring: false,
62-
...(hasLocal
63-
? {
64-
local: true,
65-
localTestingInfo: {
66-
localIdentifier: localIdentifier,
67-
localEnabled: true,
68-
},
69-
}
70-
: {}),
7163
};
7264

65+
let requestBody = baseRequestBody;
66+
if (hasLocal) {
67+
const localConfig = {
68+
localTestingInfo: {
69+
localIdentifier,
70+
localEnabled: true,
71+
}
72+
};
73+
requestBody = { ...baseRequestBody, ...localConfig };
74+
}
75+
7376
try {
7477
const { data } = await axios.post<AccessibilityScanResponse>(
7578
"https://api-accessibility.browserstack.com/api/website-scanner/v1/scans",

0 commit comments

Comments
 (0)