diff options
author | gjoranv <gv@yahooinc.com> | 2023-05-23 10:43:37 +0200 |
---|---|---|
committer | gjoranv <gv@yahooinc.com> | 2023-05-23 10:43:37 +0200 |
commit | c8815fbb7230e237757ab4c410296d18cd53d20c (patch) | |
tree | 53186ab3960fcb73e8b45ce36348fee56d70eb40 /bundle-plugin | |
parent | 8e40763559f4d32d51a65f5f53d8334299420969 (diff) |
Add comment to add test, none of the combine() methods are tested
Diffstat (limited to 'bundle-plugin')
-rw-r--r-- | bundle-plugin/src/main/java/com/yahoo/container/plugin/classanalysis/PackageTally.java | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/bundle-plugin/src/main/java/com/yahoo/container/plugin/classanalysis/PackageTally.java b/bundle-plugin/src/main/java/com/yahoo/container/plugin/classanalysis/PackageTally.java index 6e8fc89c605..e2d2b96d194 100644 --- a/bundle-plugin/src/main/java/com/yahoo/container/plugin/classanalysis/PackageTally.java +++ b/bundle-plugin/src/main/java/com/yahoo/container/plugin/classanalysis/PackageTally.java @@ -59,7 +59,7 @@ public class PackageTally { * Represents the classes for two package tallies that are deployed as a single unit. * <p> * ExportPackageAnnotations from this has precedence over the other. - * TODO: try using Map.merge (as in the functions below). Can't see how Maps.combine is any different. + * TODO: Add unit test and try using Map.merge (as in the functions below). Can't see how Maps.combine is any different. */ public PackageTally combine(PackageTally other) { var definedPkgs = Maps.combine(this.definedPackages, other.definedPackages, PackageInfo::hasExportPackageOrElse); |