tequilaOS/platform_build
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

Metadata tool correction: Output empty proto in case of no input

Browse source 
Change-Id: I85ca22574433ab164c303c1f6b0d29191e704cbb
This commit is contained in:
Aditya Choudhary 2024-01-11 11:12:44 +00:00
parent f28fa571d0
commit f0670e89f4
2 changed files with 19 additions and 11 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

29
tools/metadata/generator.go
View file

@ -77,9 +77,18 @@ func readFileToString(filePath string) string {
return string(data) return string(data)
} }
func writeNewlineToOutputFile(outputFile string) { func writeEmptyOutputProto(outputFile string, metadataRule string) {
file, err := os.Create(outputFile) file, err := os.Create(outputFile)
data := "\n" if err != nil {
log.Fatal(err)
}
var message proto.Message
if metadataRule == "test_spec" {
message = &test_spec_proto.TestSpec{}
} else if metadataRule == "code_metadata" {
message = &code_metadata_proto.CodeMetadata{}
}
data, err := proto.Marshal(message)
if err != nil { if err != nil {
log.Fatal(err) log.Fatal(err)
} }
@ -92,8 +101,8 @@ func writeNewlineToOutputFile(outputFile string) {
} }
func processTestSpecProtobuf( func processTestSpecProtobuf(
filePath string, ownershipMetadataMap *sync.Map, keyLocks *keyToLocksMap, filePath string, ownershipMetadataMap *sync.Map, keyLocks *keyToLocksMap,
errCh chan error, wg *sync.WaitGroup, errCh chan error, wg *sync.WaitGroup,
) { ) {
defer wg.Done() defer wg.Done()
@ -121,7 +130,7 @@ func processTestSpecProtobuf(
if metadata.GetTrendyTeamId() != existing.GetTrendyTeamId() { if metadata.GetTrendyTeamId() != existing.GetTrendyTeamId() {
errCh <- fmt.Errorf( errCh <- fmt.Errorf(
"Conflicting trendy team IDs found for %s at:\n%s with teamId"+ "Conflicting trendy team IDs found for %s at:\n%s with teamId"+
": %s,\n%s with teamId: %s", ": %s,\n%s with teamId: %s",
key, key,
metadata.GetPath(), metadata.GetTrendyTeamId(), existing.GetPath(), metadata.GetPath(), metadata.GetTrendyTeamId(), existing.GetPath(),
existing.GetTrendyTeamId(), existing.GetTrendyTeamId(),
@ -147,8 +156,8 @@ func processTestSpecProtobuf(
// processCodeMetadataProtobuf processes CodeMetadata protobuf files // processCodeMetadataProtobuf processes CodeMetadata protobuf files
func processCodeMetadataProtobuf( func processCodeMetadataProtobuf(
filePath string, ownershipMetadataMap *sync.Map, sourceFileMetadataMap *sync.Map, keyLocks *keyToLocksMap, filePath string, ownershipMetadataMap *sync.Map, sourceFileMetadataMap *sync.Map, keyLocks *keyToLocksMap,
errCh chan error, wg *sync.WaitGroup, errCh chan error, wg *sync.WaitGroup,
) { ) {
defer wg.Done() defer wg.Done()
@ -182,8 +191,8 @@ func processCodeMetadataProtobuf(
if attributes.TeamID != existing.TeamID && (!attributes.MultiOwnership || !existing.MultiOwnership) { if attributes.TeamID != existing.TeamID && (!attributes.MultiOwnership || !existing.MultiOwnership) {
errCh <- fmt.Errorf( errCh <- fmt.Errorf(
"Conflict found for source file %s covered at %s with team ID: %s. Existing team ID: %s and path: %s."+ "Conflict found for source file %s covered at %s with team ID: %s. Existing team ID: %s and path: %s."+
" If multi-ownership is required, multiOwnership should be set to true in all test_spec modules using this target. "+ " If multi-ownership is required, multiOwnership should be set to true in all test_spec modules using this target. "+
"Multiple-ownership in general is discouraged though as it make infrastructure around android relying on this information pick up a random value when it needs only one.", "Multiple-ownership in general is discouraged though as it make infrastructure around android relying on this information pick up a random value when it needs only one.",
srcFile, internalMetadata.GetPath(), attributes.TeamID, existing.TeamID, existing.Path, srcFile, internalMetadata.GetPath(), attributes.TeamID, existing.TeamID, existing.Path,
) )
srcFileLock.Unlock() srcFileLock.Unlock()
@ -235,7 +244,7 @@ func main() {
inputFileData := strings.TrimRight(readFileToString(*inputFile), "\n") inputFileData := strings.TrimRight(readFileToString(*inputFile), "\n")
filePaths := strings.Split(inputFileData, " ") filePaths := strings.Split(inputFileData, " ")
if len(filePaths) == 1 && filePaths[0] == "" { if len(filePaths) == 1 && filePaths[0] == "" {
writeNewlineToOutputFile(*outputFile) writeEmptyOutputProto(*outputFile, *rule)
return return
} }
ownershipMetadataMap := &sync.Map{} ownershipMetadataMap := &sync.Map{}

1
tools/metadata/testdata/generatedEmptyOutputFile.txt vendored
View file

@ -1 +0,0 @@