App tab – upload APK/IPA files directly to the server's fileshare

[luprochazka-cen63872]

Hi, I’m aware of the App tab and I use it to sign the wda.ipa file. However, I’m considering another use case. In our Robot Framework scripts, we install applications using absolute paths.

The Automation FileName feature might be tricky for us — I’d prefer to retain the original filename. This approach could save time by avoiding the need to access the server manually to download or copy the correct app file.

[luprochazka-cen63872]

up

[luprochazka-cen63872]

@saikrishna321 I believe it could be very useful, especially for test automation scripts, when you need the application installation files on the target machine. A random filename complicates the situation.

[saikrishna321]

@luprochazka-cen63872 you suggesting to use the App Name instead of AutomationFileName?

[luprochazka-cen63872]

Yes, I think the original name can remain the same in the folder, allowing users to upload files like App_21.2.ipa. However, for wda-resign.ipa, it won't work unless you manage to overwrite the file, maybe a dialog for overwriting? And perhaps users could specify a folder for uploading these files (as a startup argument of Device Farm)?

[twisteer]

Currently, the findAppPath function searches the db only when the fileName starts with file.

Adding an additional condition to check whether the fileName exists in the table could solve the issue with random file names.

async function findAppPath(caps: any) {
  const mergedCaps = Object.assign({}, caps.firstMatch ? caps.firstMatch[0] : {}, caps.alwaysMatch);
  const fileName = mergedCaps['appium:app'];
  if (fileName?.startsWith('file')) {
    const appInfo: any = await prisma.appInformation.findFirst({
      where: { uploadedFileName: fileName as string },
    });
    return `${DevicePlugin.serverUrl}${appInfo?.path}`;
  } else if (fileName?.match(/^[A-Za-z]+[A-Za-z0-9_\-\.]+\.(apk|ipa)$/g)) {
    const appInfo: any = await prisma.appInformation.findFirst({
     where: { fileName: fileName as string },
     orderBy: { createdAt: 'desc' },
    });
    return appInfo ? `${DevicePlugin.serverUrl}${appInfo?.path}` : fileName;
  } else {
    return fileName;
  }
}

[luprochazka-cen63872]

Any update for this issue?

[twisteer]

Merged - #2028