feat(device_info_plus): LinuxDeviceInfo toString method

[defuncart]

Description

Adds toString for LinuxDeviceInfo. This is useful when debugging apps on Linux distros.

Related Issues

N/A

Checklist

• I read the Contributor Guide and followed the process outlined there for submitting PRs.
• I titled the PR using Conventional Commits.
• I did not modify the CHANGELOG.md nor the plugin version in pubspec.yaml files.
• All existing and new tests are passing.
• The analyzer (flutter analyze) does not report any problems on my PR.

Breaking Change

Does your PR require plugin users to manually update their apps to accommodate your change?

• Yes, this is a breaking change (please indicate that with a ! in the title as explained in Conventional Commits).
• No, this is not a breaking change.

[miquelbeltran]

Thanks for the contribution, and including a test as well!

I'd prefer if the long string could be split into several lines or a multiline string, as it will be easier to extend and review changes in the future.

Can you take a look at the Flutter analyze issues as well?

[defuncart]

I'd prefer if the long string could be split into several lines or a multiline string, as it will be easier to extend and review changes in the future.

The method was generated by dart-data-class-generator and uses a standard format. How would you like it to be multi-line? Code of the form

    return '''LinuxDeviceInfo(
name: $name,
version: $version,
id: $id, 
idLike: $idLike, 
versionCodename: $versionCodename, 
versionId: $versionId, 
prettyName: $prettyName, 
buildId: $buildId, 
variant: $variant, 
variantId: $variantId, 
machineId: $machineId
)''';

would return a multi-line string, not a single line as expected in toString. To improve extensibility, StringBuffer could be considered, or alternatively data could be used as follows

String toString() => 'LinuxDeviceInfo(${data.toString().replaceAll('}', '').replaceAll('{', '')})';

Can you take a look at the Flutter analyze issues as well?

Done.

[miquelbeltran]

That's right, I was thinking on a different approach, but since we already do similar toString methods in other parts of the projects, I guess that would be fine as well:

plus_plugins/packages/package_info_plus/package_info_plus/lib/package_info_plus.dart

Line 119 in c943583

String toString() {

[defuncart]

@miquelbeltran Seems the pipeline needs to be re-run, android integration test is failing & I cannot manually re-run. Probably a flaky issue as PR does not touch any android code.

[miquelbeltran]

That's fine, no worries, the android_integration_test (22) is indeed flaky