chromite: Split long strings on lines > 80 chars.
Black will only split strings at spaces.
We can tweak its `SPLIT_SAFE_CHARS` to split elsewhere -
https://github.com/psf/black/blob/main/src/black/trans.py#L76
Allowing splits at "-","/", and "." lets it deal with a few more
long strings in chromite automatically, so there will be fewer
suppressions when the line-too-long lint is introduced.
BUG=b:233893248
TEST=cros lint + CQ
Change-Id: Ic092be1d0a80a4c13fac30d19756db715b42b8dd
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/chromite/+/4535497
Commit-Queue: Trent Apted <tapted@chromium.org>
Reviewed-by: Jack Rosenthal <jrosenth@chromium.org>
Tested-by: Trent Apted <tapted@chromium.org>
diff --git a/scripts/upload_prebuilts_unittest.py b/scripts/upload_prebuilts_unittest.py
index e88cd87..d79d9bd 100644
--- a/scripts/upload_prebuilts_unittest.py
+++ b/scripts/upload_prebuilts_unittest.py
@@ -764,16 +764,18 @@
"""Make sure processing of board-specific overlay tarballs works."""
to_tarballs = (
(
- "i686-pc-linux-gnu:"
- "/some/path/built-sdk-overlay-toolchains-i686-pc-linux-gnu.tar.xz"
+ "i686-pc-linux-gnu:/some/path/built-sdk-overlay-toolchains-"
+ "i686-pc-linux-gnu.tar.xz"
),
(
- "armv7a-cros-linux-gnueabi-arm-none-eabi:"
- "/some/path/built-sdk-overlay-toolchains-armv7a-cros-linux-gnueabi-"
- "arm-none-eabi"
+ "armv7a-cros-linux-gnueabi-arm-none-eabi:/some/path/built-sdk-"
+ "overlay-toolchains-armv7a-cros-linux-gnueabi-arm-none-eabi"
),
)
- to_upload_path = "1994/04/cros-sdk-overlay-toolchains-%(toolchains)s-1994.04.02.tar.xz"
+ to_upload_path = (
+ "1994/04/cros-sdk-overlay-toolchains-%(toolchains)s-1994.04.02."
+ "tar.xz"
+ )
self.testSdkUpload(
to_tarballs=to_tarballs, to_upload_path=to_upload_path
)