Add car and foldable types to target types.

PiperOrigin-RevId: 641248360
This commit is contained in:
Anay Wadhera
2024-06-07 07:50:45 -07:00
committed by Copybara-Service
parent 2d845718f2
commit c3924eb8df
5 changed files with 39 additions and 11 deletions
+3
View File
@@ -92,12 +92,15 @@ uint8_t ConvertHasDeviceName(bool hasDeviceName) {
int ParseVersion(uint8_t b) { return (b >> 5) & kVersionBitmask; }
// The values are in ShareTargetType.
bool IsKnownDeviceValue(int32_t value) {
switch (value) {
case 0:
case 1:
case 2:
case 3:
case 4:
case 5:
return true;
default:
return false;
+12
View File
@@ -89,6 +89,18 @@ INSTANTIATE_TEST_SUITE_P(
Advertisement::kMetadataEncryptionKeyHashByteSize),
.target_type = ShareTargetType::kTablet,
.target_name = std::nullopt,
.vendor_id = 0},
TestParameters{.salt = std::vector<uint8_t>(Advertisement::kSaltSize),
.encrypted_metadata_key = std::vector<uint8_t>(
Advertisement::kMetadataEncryptionKeyHashByteSize),
.target_type = ShareTargetType::kCar,
.target_name = std::nullopt,
.vendor_id = 0},
TestParameters{.salt = std::vector<uint8_t>(Advertisement::kSaltSize),
.encrypted_metadata_key = std::vector<uint8_t>(
Advertisement::kMetadataEncryptionKeyHashByteSize),
.target_type = ShareTargetType::kFoldable,
.target_name = std::nullopt,
.vendor_id = 0}));
INSTANTIATE_TEST_SUITE_P(
VendorIds, AdvertisementTest,
+12
View File
@@ -43,6 +43,7 @@ enum class DeviceNameValidationResult {
// be kept in sync with Android implementation.
// These values are persisted to logs. Entries should not be renumbered and
// numeric values should never be reused.
// LINT.IfChange()
enum class ShareTargetType {
// Unknown device type.
kUnknown = 0,
@@ -52,7 +53,18 @@ enum class ShareTargetType {
kTablet = 2,
// A laptop.
kLaptop = 3,
// A car.
kCar = 4,
// A foldable.
kFoldable = 5,
};
// LINT.ThenChange(
// //depot/google3/location/nearby/cpp/sharing/clients/proto/share_target.proto:ShareTarget.ShareTargetType,
// //depot/google3/location/nearby/sharing/sdk/quick_share/protos/share_target.proto:ShareTarget,
// //depot/google3/third_party/nearby/sharing/advertisement.cc:IsKnownDeviceValue,
// //depot/google3/java/com/google/android/gmscore/integ/client/nearby/src/com/google/android/gms/nearby/sharing/ShareTarget.java:Type,
// //depot/google3/location/nearby/cpp/sharing/clients/dart/platform/lib/types/share_target.dart:ShareTargetType
//)
// This enum combines both text and file share attachment types into a single
// enum that more directly maps to what is shown to the user for preview.
+1
View File
@@ -67,6 +67,7 @@ std::string ShareTarget::ToString() const {
std::vector<std::string> fmt;
fmt.push_back(absl::StrFormat("id: %" PRId64, id));
fmt.push_back(absl::StrFormat("type: %d", type));
fmt.push_back(absl::StrFormat("device_name: %s", device_name));
if (full_name) {
fmt.push_back(absl::StrFormat("full_name: %s", *full_name));
+11 -11
View File
@@ -43,17 +43,17 @@ std::vector<ShareTargetToStringTestData> GetTestData() {
share_target1.id = 1;
share_target2.id = 2;
static std::vector<
ShareTargetToStringTestData>* kShareTargetToStringTestData =
new std::vector<ShareTargetToStringTestData>({
{share_target1,
"ShareTarget<id: 1, device_name: , is_known: 0, is_incoming: 0, "
"for_self_share: 0, vendor_id: 0>"},
{share_target2,
"ShareTarget<id: 2, device_name: test_name, full_name: "
"test_full_name, image_url: ://:0, device_id: test_device_id, "
"is_known: 0, is_incoming: 1, for_self_share: 1, vendor_id: 0>"},
});
static std::vector<ShareTargetToStringTestData>*
kShareTargetToStringTestData =
new std::vector<ShareTargetToStringTestData>({
{share_target1,
"ShareTarget<id: 1, type: 0, device_name: , is_known: 0, "
"is_incoming: 0, for_self_share: 0, vendor_id: 0>"},
{share_target2,
"ShareTarget<id: 2, type: 1, device_name: test_name, full_name: "
"test_full_name, image_url: ://:0, device_id: test_device_id, "
"is_known: 0, is_incoming: 1, for_self_share: 1, vendor_id: 0>"},
});
return *kShareTargetToStringTestData;
}