commit 684e7a64e1badab0cafbab32ff97d613473caf0a
author Luciano Pacheco <lucmult@chromium.org> Wed Jan 24 01:31:01 2024
committer Chromium LUCI CQ <chromium-scoped@luci-project-accounts.iam.gserviceaccount.com> Wed Jan 24 01:31:01 2024
tree 8af8ed1dbec0827ed0deadd55de5683801a24852
parent a895002554d8a4d80b202c7f78576aebcc6fabf0

Files app: Change function registration in the integration tests

Before this CL the test functions were registered directly to the
namespace `testcase`. However this had 2 drawbacks:
1. It required to import all files in background.js, which could cause
circular imports, because some helper functions live inside
background.js and are imported by the test files.
2. When converting to TS, the type system doesn't allow to easily
append functions to the `testcase` namespace, it requires to use the
syntax: `testcase['testName'] = async () => {}`.

This CL changes the registration to happen directly inside the
testcase.ts, so the test files just need to export their test functions.

Convert android_photos.js to TS as example for further conversions.

Bug: b:319189127
Change-Id: I829d5d464571eae035c7ab2cf63d07a7dbb02b0b
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/5221804
Reviewed-by: Wenbo Jie <wenbojie@chromium.org>
Commit-Queue: Luciano Pacheco <lucmult@chromium.org>
Cr-Commit-Position: refs/heads/main@{#1251169}