Merge pull request #777 from prettier-solidity/null-node-printer

Adding test for null node in printer

Author: fvictorio

.c8rc

@@ -1,9 +1,9 @@
 {
   "check-coverage": true,
   "branches": 99,
-  "lines": 99,
+  "lines": 100,
   "functions": 100,
-  "statements": 99,
+  "statements": 100,
   "exclude": ["/node_modules/", "/scripts/", "/src/prettier-comments/"],
   "include": ["src/**/*.js", "!src/prettier-comments/**/*.js"],
   "reporter": ["lcov", "text"],

tests/unit/binary-operation.test.js

@@ -1,6 +1,6 @@
 const genericPrint = require('../../src/printer');
 
-test('given an unknown module type then genericPrint function should throw', () => {
+test('given an unknown operator then the BinaryOperation print function should throw', () => {
   const mockPath = {
     getValue: () => ({ type: 'BinaryOperation', operator: '?' })
   };

tests/unit/printer.test.js

@@ -8,3 +8,11 @@ test('given an unknown module type then genericPrint function should throw', ()
     genericPrint(mockPath);
   }).toThrow(`Unknown type: ${JSON.stringify(node.type)}`);
 });
+
+test('if the AST contains a null node, print an empty string', () => {
+  // Prettier V3 avoids returning null when traversing the AST, but V2 does not.
+  // By mocking this, we ensure both cases are covered.
+  const mockPath = { getValue: () => null };
+
+  expect(genericPrint(mockPath)).toEqual('');
+});