Support newInstanceNamed with empty name (#1194)

Co-authored-by: Moritz <[email protected]>

Author: mattrberry

pkgs/code_builder/CHANGELOG.md

@@ -2,6 +2,7 @@
 
 * Upgrade `dart_style` and `source_gen` to remove `package:macros` dependency.
 * Require Dart `^3.6.0` due to the upgrades.
+* Support `Expression.newInstanceNamed` with empty name
 
 ## 4.10.1
 

pkgs/code_builder/lib/src/specs/expression.dart

@@ -578,10 +578,10 @@ abstract mixin class ExpressionEmitter
     final out = output ??= StringBuffer();
     return _writeConstExpression(out, expression.isConst, () {
       expression.target.accept(this, out);
-      if (expression.name != null) {
+      if (expression.name case final name? when name.isNotEmpty) {
         out
           ..write('.')
-          ..write(expression.name);
+          ..write(name);
       }
       if (expression.typeArguments.isNotEmpty) {
         out.write('<');

pkgs/code_builder/test/specs/code/expression_test.dart

@@ -251,6 +251,13 @@ void main() {
     );
   });
 
+  test('should emit invoking unnamed constructor when name is empty', () {
+    expect(
+      refer('Foo').newInstanceNamed('', []),
+      equalsDart('Foo()'),
+    );
+  });
+
   test('should emit invoking const Type()', () {
     expect(
       refer('Object').constInstance([]),